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=755602 Parag AN(पराग) <panemade@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #5 from Parag AN(पराग) <panemade@xxxxxxxxx> 2011-11-22 10:09:14 EST --- (In reply to comment #4) > (In reply to comment #3) > > Suggestion: > > 1) you should write a wrapper script in /usr/bin instead of linking > > json_diff.py in /usr/bin/json_diff > > > > see examples files createrepo or system-config-printer in /usr/bin > > Why? And how does it break Packaging Guidelines? Fine. That was just a suggestion. APPROVED. -- 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