[Bug 225791] Merge Review: gettext

[bugzilla at redhat.com]

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 at ioa.s.u-tokyo.ac.jp  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.