Hi Michel. The dependencies have just been approved and you can start reviewing
this package and #440681 now.

Thanks for the heads-up!

Same problem as luadoc:
- rpmlint: strange permission for tarball: 0600
- package is noarch but installs files in %{_libdir}
- does not seem to work on x86_64:

> require "logging.console"
> local logger = logging.console()
> =logger
nil

Same as luadoc, re-uploaded the very same SRPM with fixed source tarball
permissions. For this package there is nothing in %{_libdir} as well, tricked by
lualibdir define? Please otherwise attach "rpm -qpl" output of resulting package.

For the test you did: it cannot work the way you did it! The Lua shell compiles
every line as its own chunk. If you use "local logger" then it's only defined
for that very line and cannot be used in the next line. You can use either:
> local logger = logging.console(); logger:info("Test")
(have it both on one line such that logger is defined) or just don't have the
local such that the logger is global and thus available in the following lines.
This has nothing to do with x86_64 (I'm testing on this platform myself) - after
all this package is true noarch.

Aha! So that's the scope of local. I was expecting the context to be the entire
top-level, not just the single compilation unit. Thanks.

• rpmlint: OK
• package name: OK
• spec file name: OK
• package guideline-compliant: OK
• license complies with guidelines: OK
• license field accurate: OK
• license file not deleted: OK
• spec in US English: OK
• spec legible: OK
• source matches upstream: OK
• builds under >= 1 archs, others excluded: noarch
• build dependencies complete: OK
• locales handled using %find_lang, no %{_datadir}/locale: N/A
• library -> ldconfig: N/A
• relocatable: give reason: N/A
• own all directories: OK
• no dupes in %files: OK
• permission: OK
• %clean RPM_BUILD_ROOT: OK
• macros used consistently: OK
• Package contains code: OK
• large docs => -doc: N/A
• doc not runtime dependent: N/A
• headers in -devel: N/A
• static in -static: N/A
• if contains *.pc, req pkgconfig: N/A
• if libfiles are suffixed, the non-suffixed goes to devel: N/A
• devel requires versioned base package: N/A
• desktop file uses desktop-file-install: N/A
• clean buildroot before install: OK
• filenames UTF-8: OK

SHOULD
• if license text missing, ask upstream to include it: N/A
• desc and summary contain translations if available: No
• package build in mock on all architectures: OK
• package functioned as described: OK
• scriplets are sane: OK
• other subpackages should require versioned base: N/A
• if main pkg is development-wise, pkgconfig can go in main package: N/A
• require package not files: OK

APPROVED

New Package CVS Request
=======================
Package Name: lua-logging
Short Description: A simple API to use logging features in Lua
Owners: timn
Branches: F-7 F-8
InitialCC: 
Cvsextras Commits: yes

cvs done.

Package Change Request
======================
Package Name: lua-logging
New Branches: el5 el6
Owners: timn

Git done (by process-git-requests).