Bug 233242 - Review Request: ragel - finite state machine compiler
Review Request: ragel - finite state machine compiler
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jochen Schmitt
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-03-21 02:22 EDT by Jeremy Hinegardner
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-03-24 14:49:49 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
jochen: fedora‑review+
wtogami: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Jeremy Hinegardner 2007-03-21 02:22:44 EDT
Spec URL: http://www.hinegardner.org/fedora-extras/ragel.spec
SRPM URL: http://www.hinegardner.org/fedora-extras/ragel-5.19-1.src.rpm
Description: Ragel compiles finite state machines from regular languages into executable C,C++, Objective-C, or D code.

7939ed87968dad75cc7932b032f5f108  ragel-5.19.tar.gz
Comment 1 Jochen Schmitt 2007-03-21 13:01:37 EDT
Good:
+ Naming seems ok.
+ Tar ball matches with upstream.
+ Local build works fine.
* Rpmlint quite on source rpm.
+ Rpmlint quite on binary rpm.
+ Rpmlint quite on debugin rpm.
+ Local install and uninstallworks fine.
+ Start of programm works without crash.
+ Mock build works fine.

Bad:
- Why you have a BuildRequires to autoconf, when you not call autoconf in the
build stanza?
Comment 2 Jeremy Hinegardner 2007-03-21 18:06:58 EDT
This is also my first package and I'm seeking a sponsor.
Comment 3 Jeremy Hinegardner 2007-03-22 01:56:19 EDT
I've updated the spec to not have BuildRequires: autoconf.  Not sure why I had
that, bit of a mistake.

New Spec and src rpm available.

Spec URL: http://www.hinegardner.org/fedora-extras/ragel.spec
SRPM URL: http://www.hinegardner.org/fedora-extras/ragel-5.19-1.src.rpm
Comment 4 Jochen Schmitt 2007-03-22 13:00:16 EDT
Note: Please increase the release number if you create a new rpm package.
Comment 5 Jochen Schmitt 2007-03-22 13:10:52 EDT
Good:
+ License OK.
+ Autoconf was removed from BR.

** APPROVED **
Comment 6 Jeremy Hinegardner 2007-03-22 17:37:12 EDT
A new upload with the appropriately incremented relase number is available

Spec URL: http://www.hinegardner.org/fedora-extras/ragel.spec
SRPM URL: http://www.hinegardner.org/fedora-extras/ragel-5.19-2.src.rpm
Comment 7 Kevin Fenzi 2007-03-23 17:22:27 EDT
Removing FE-NEEDSPONSOR. Submitter was sponsored in 
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233522
Comment 8 Jeremy Hinegardner 2007-03-23 19:16:35 EDT
New Package CVS Request
=======================
Package Name: ragel
Short Description: Finite state machine compiler
Owners: jeremy@hinegardner.org
Branches: FC-5 FC-6
InitialCC: 

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