[Rpm-maint] [rpm-software-management/rpm] Fix rpm lua rpm_vercmp error message if second argument is broken (PR #2163)

AvoidMe notifications at github.com
Thu Aug 25 21:14:37 UTC 2022


This is a fix for small issue I encounter while was playing with lua in rpm spec:
```
➜  rpm --eval "%{lua:rpm.interactive()}"

RPM Interactive Lua 5.4 Interpreter
> print(rpm.vercmp('1', '1'))
0
> print(rpm.vercmp('', '1'))         
[string "<lua>"]:1: bad argument #1 to 'vercmp' (invalid version )
> print(rpm.vercmp('1', '')) 

> 
```
If first argument is wrong in `rpm.vercmp()` - meaningful error message as expected
if second argument is wrong - nothing happens and `nil` returned
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/2163

-- Commit Summary --

  * Fix rpm lua rpm_vercmp error message if second argument is broken

-- File Changes --

    M rpmio/rpmlua.c (2)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/2163.patch
https://github.com/rpm-software-management/rpm/pull/2163.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2163
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2163 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20220825/357935e2/attachment-0001.html>


More information about the Rpm-maint mailing list