Bug 476712 - Package Review: lua-rex - Regular expression handling library for Lua
Summary: Package Review: lua-rex - Regular expression handling library for Lua
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: David Cantrell
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-12-16 17:27 UTC by Lubomir Rintel
Modified: 2013-01-10 04:58 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-02-06 00:11:35 UTC
Type: ---
Embargoed:
jkeating: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Lubomir Rintel 2008-12-16 17:27:48 UTC
SPEC: http://netbsd.sk/~lkundrak/SPECS/lua-rex.spec
SRPM: http://netbsd.sk/~lkundrak/SRPMS/lua-rex-2.4.0-1.fc10.src.rpm

Description:

Lrexlib are bindings of three regular expression library APIs (POSIX, PCRE
and Oniguruma) to Lua.

Comment 1 Lubomir Rintel 2008-12-16 17:30:59 UTC
Builds in fedora-10-i386 mock, rpmlint silent and happy.

Comment 2 Jesse Keating 2008-12-16 23:26:09 UTC
Taking review.

Comment 3 Jesse Keating 2008-12-16 23:38:31 UTC
Should the .so files go in a -devel subpackage?

License seems good

Why isn't doc/ packaged in %doc ?

Why aren't smp_flags used?

What is the purpose of the manual lrexlib Provides?  What will be Requiring that, other than the other auto-generated Provides:

rex_onig.so.2.4  
rex_pcre.so.2.4  
rex_posix.so.2.4  
lua-rex = 2.4.0-1.fc11
lua-rex(x86-32) = 2.4.0-1.fc11

Comment 4 Lubomir Rintel 2008-12-17 12:04:14 UTC
(In reply to comment #3)
> Should the .so files go in a -devel subpackage?

No. They are not meant for use by linker, lua interpreter looks for .so files specifically.

I'm thinking whether it makes sense to ship the .2.4 files, but it's that just because upstream does so. The .so files don't have a SONAME and are not used by dynamic linker, just dlopened by lua.

> License seems good
> 
> Why isn't doc/ packaged in %doc ?

I just forgot. I'll be including it in new package I'll post shortly.

> Why aren't smp_flags used?

There's a comment. Or should I be more specific than just "doesn't work"?

> What is the purpose of the manual lrexlib Provides?  What will be Requiring
> that, other than the other auto-generated Provides:
> 
> rex_onig.so.2.4  
> rex_pcre.so.2.4  
> rex_posix.so.2.4  
> lua-rex = 2.4.0-1.fc11
> lua-rex(x86-32) = 2.4.0-1.fc11

That's an upstream name. While it is desirable to name this package lua-rex, to denote it's a lua rex module there's a chance people will try to install it by doing "yum install lrexlib". If I remember correctly this is a recommended practice for perl modules, and I assumed it would be the same for lua.

Comment 5 Lubomir Rintel 2008-12-17 18:18:50 UTC
SPEC: http://netbsd.sk/~lkundrak/SPECS/lua-rex.spec
SRPM: http://netbsd.sk/~lkundrak/SRPMS/lua-rex-2.4.0-2.el5.src.rpm

* Wed Dec 17 2008 Lubomir Rintel <lkundrak> - 2.4.0-2
- Add doc directory to documentation
- Allow parallel make runs

Comment 6 Jesse Keating 2008-12-17 21:47:56 UTC
Looks great now, approving.

At some point you should probably bring a lua packaging guideline suggestion to the FPC since there are a few lua packages in now, and could be more coming.  It'd speed up reviews.

Comment 7 Lubomir Rintel 2008-12-18 07:11:55 UTC
New Package CVS Request
=======================
Package Name: lua-rex
Short Description: Regular expression handling library for Lua
Owners: lkundrak
Branches: F-10

Comment 8 Kevin Fenzi 2008-12-21 04:43:26 UTC
cvs done.


Note You need to log in before you can comment on or make changes to this bug.