This doesn't even compile against a build of Lua where the compat-stuff is 
actually disabled, due to missing port of luaL_openlib() in rpmio/rpmlua.c. 
What's worse, when using a version of lua that it does compile against, it 
causes any attempt to use rpm macros crash:
```
$ ./rpm --eval "%{_lib}"
PANIC: unprotected error in call to Lua API (attempt to index a nil value)
Aborted (core dumped)
```

....which in terms of the test-suite means:
```
## ------------- ##
## Test results. ##
## ------------- ##

ERROR: All 415 tests were run,
406 failed (4 expected failures).
```

Folks, if you're submitting untested code then at least have the courtesy of 
stating so.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/169#issuecomment-379735523
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to