[Bug 440676] New: Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/show_bug.cgi?id=440676

Summary: Review Request: lua-filesystem - File System Library for
                    Lua
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://fedorapeople.org/~timn/luastuff/lua-filesystem.spec
SRPM URL: http://fedorapeople.org/~timn/luastuff/lua-filesystem-1.4.0-1.fc8.src.rpm
Description:
LuaFileSystem is a Lua library developed to complement the set of functions
related to file systems offered by the standard Lua distribution.

LuaFileSystem offers a portable way to access the underlying directory
structure and file attributes.

Website: http://www.keplerproject.org/luafilesystem/

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |440681
              nThis|                            |

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?

------- Additional Comments From [e-mail]  2008-04-04 18:43 EST -------
Builds fine and rpmlint is silent.

The compiler is not called with the proper flags.  This leads to a broken
debuginfo package, among other issues.  It looks like you'll need to patch the
"config" file as it overwrites the passed CFLAGS.

* source files match upstream:
   226db0b2903e2a04de0ad0a71e53a0f629683a61172c228a3e7d416c226101ca  
   luafilesystem-1.4.0.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
X compiler flags are not correct.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
X debuginfo package is not complete.
* rpmlint is silent.
* final provides and requires are sane:
   lfs.so()(64bit)
   lua-filesystem = 1.4.0-1.fc9
  =
   lua >= 5.1
* %check is not present; no test suite upstream.  I do not know how to test this 
  software.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

------- Additional Comments From [e-mail]  2008-04-04 18:54 EST -------
I'm setting the CFLAGS to %{optflags} now (I didn't pass anything yet). I have
to add an additional "-fPIC" or otherwise I get the following compiler error (on
x86_64):
/usr/bin/ld: src/lfs.o: relocation R_X86_64_32 against `a local symbol' can not
be used when making a shared object; recompile with -fPIC
src/lfs.o: could not read symbols: Bad value

Does that satisfy debuginfo generation?

New SRPM is at
http://fedorapeople.org/~timn/luastuff/lua-filesystem-1.4.0-2.fc8.src.rpm

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

------- Additional Comments From [e-mail]  2008-04-04 19:10 EST -------
Yes, debuginfo is OK now.

Do note that I see the following unpleasant compiler warnings:

src/lfs.c: In function 'get_dir':
src/lfs.c:112: warning: implicit declaration of function 'free'
src/lfs.c:112: warning: incompatible implicit declaration of built-in function
'free'

which will probably cause a build failure if we ever turn on
-Werror-implicit-function-declaration by default.

Do note also that your passing of CFLAGS disables the default -I arguments that
the makefile passes.  It turns out that it's only /usr/include so you should be OK.

Anyway, everything looks acceptable now.
APPROVED

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

------- Additional Comments From [e-mail]  2008-04-04 19:23 EST -------
You are right, I have created a trivial patch to fix the problem and included it
in http://fedorapeople.org/~timn/luastuff/lua-filesystem-1.4.0-3.fc8.src.rpm and
sent it upstream.

Are you willing to take over the other reviews of my Lua packages (440677 to
440681) as well?

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2008-04-04 19:25 EST -------
New Package CVS Request
=======================
Package Name: lua-filesystem
Short Description: File System Library for Lua
Owners: timn
Branches: F-7 F-8
InitialCC: 
Cvsextras Commits: yes

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

------- Additional Comments From [e-mail]  2008-04-04 19:28 EST -------
I'm working on lua-posix at the moment.  These packages are mostly trivial so I
should be able to work through them pretty quickly.

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2008-04-05 12:49 EST -------
cvs done.

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lua-filesystem - File System Library for Lua

https://bugzilla.redhat.com/show_bug.cgi?id=440676

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=440676

Michel Alexandre Salim  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
               Flag|fedora-cvs+                 |fedora-cvs?

--- Comment #8 from Michel Alexandre Salim   2009-10-01 18:05:10 EDT ---
Discussed with Tim; we'd want to eventually have the entire Lua stack on EPEL.

Package Change Request
======================
Package Name: lua-filesystem
New Branches: EL-4 EL-5
Owners: timn salimma

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=440676

Kevin Fenzi  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

--- Comment #9 from Kevin Fenzi   2009-10-03 17:34:22 EDT ---
cvs done.

[Bug 440676] Review Request: lua-filesystem - File System Library for Lua

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=440676

--- Comment #10 from Fedora Update System   2009-10-03 20:07:06 EDT ---
lua-filesystem-1.4.2-1.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/lua-filesystem-1.4.2-1.el5