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=771123 Mark McKinstry <mmckinst@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(mmckinst@nexcess. | |net) | --- Comment #3 from Mark McKinstry <mmckinst@xxxxxxxxxxx> 2012-01-09 19:01:32 EST --- Mattia, Thank you for the review. I have: 1. Added a %clean section since I do plan on putting this in EPEL too. 2. Fixed the spelling issue. "tokenizeing" is now spelled "tokenizing". rpmlint is still unhappy with the word but tokenizing is commonly used including the wikipedia article on lexical analysis. 3. Added a Group to the spec file. It wasn't in the original spec and it looks like rpmlint doesn't check for it. Updated spec and src.rpm: http://mmckinst.fedorapeople.org/packages/fex/fex.spec http://mmckinst.fedorapeople.org/packages/fex/fex-1.20100416.2814-2.fc16.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review