Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: gettext https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225791 ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-03-12 07:49 EST ------- (In reply to comment #3) > > * Source2 > > - SOURCE files should not have executable permission. > > Not sure how to fix this: the python script is executable in cvs. Well, it seems that to fix this you have to once remove this script from cvs and readd?? If so, leave this as it is for now. > > - Would you explain if the status of > > unversioned-explicit-provides devel(libintl) > > is proper (and why this is required)? > > No idea. Probably some historical artefact. Perhaps it can just be removed? Well, on my system 'devel(libintl)' is not required by any package. > > - There are many 'spurious-executable-perm' complaint. > > - And somes are 'wrong-script-interpreter' > > - Check if some zero-length files are needed. > > To be honest I am tempted just to remove all the examples completely from > gettext-devel or at least move them to another subpackage. Well, these files doesn't seem to be needed... Other points: * It may be better that "./ChangeLog" is added to %doc. Now I am checking all sources... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review