[Bug 225791] Merge Review: gettext

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 02:25 EST -------
Created an attachment (id=149807)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=149807&action=view)
rpmlint log for gettext 0.16.1-5 on FC-devel i386

Well, for 0.16.1-5:
Some cleanup seems to be needed.

A spec file
* Summary:
  - Please don't end with dots.

* Prereq
  - Deprecated. Use Requires(post) and Requires(preun)

* Source2
  - SOURCE files should not have executable permission.

? BuildRoot tag
  - This BuildRoot does not meet current Fedora policy.

* Parallel make
  - Please check if parallel make is possible.

* %{makeinstall} macro
  - Avoid this if possible.

* Timestamps
---------------------------------------------
install -m 755 %SOURCE2 ${RPM_BUILD_ROOT}/%{_bindir}/msghack
---------------------------------------------
  - Change to "install -p -m 755'. This is a script is keeping timestamp
    is recommended.
  - Well, perhaps also for this package
---------------------------------------------
make install INSTALL="%{__install} -p" .....
---------------------------------------------
    works well to keep timestamps on some files.

* gettext mo files
---------------------------------------------
rm -f $RPM_BUILD_DIR/%{name}-%{version}/trans.list
pushd %{buildroot}/%{_datadir}/locale
for foo in `find . -maxdepth 1 -mindepth 1 -type d` ; do
  lang=`echo $foo | cut -c 3-`
  echo "%lang($lang) %{_datadir}/locale/$foo/*/*" >> \
    $RPM_BUILD_DIR/%{name}-%{version}/trans.list
done
popd
---------------------------------------------
  Cannot this be treated by %find_lang macro?

* One line command in %postun etc...
  - Please use "-p" option.

B. File check
  - Directory ownership issue
----------------------------------------------
[tasaka1@localhost gettext]$ ( for f in `rpm -ql gettext gettext-devel | sort`
; do if [ -d $f ] ; then for g in `rpm -qf $f` ; do echo -e "$f\t$g" ; done ;
fi ; done ) | sed -e '/^[^ \t][^ \t]*\tgettext.*/d' 
/usr/share/locale/en@boldquot	filesystem-2.4.2-1.fc7
/usr/share/locale/en@boldquot/LC_MESSAGES	filesystem-2.4.2-1.fc7
/usr/share/locale/en@quot	filesystem-2.4.2-1.fc7
/usr/share/locale/en@quot/LC_MESSAGES	filesystem-2.4.2-1.fc7
----------------------------------------------
   - Static archive
     * Would you explain why static archive is needed, or just split
       these to -static subpackage?

C. rpmlint issue
   ------ This is attached --------
   NOTES:
   - Please use %% in %changelog for macros.
   - Would you explain if the status of
---------------------------------------------
     unversioned-explicit-provides devel(libintl)
---------------------------------------------
     is proper (and why this is required)?
   - Would you explain the status of /usr/lib/preloadable_libintl.so?
     * Currently this has 0644 permission
     * This is not stripped, debuginfo rpm does not contain the debug
       information for this file.
   - There are many 'spurious-executable-perm' complaint.
   - And somes are 'wrong-script-interpreter'
   - Check if some zero-length files are needed.

D. NOTES:
   - /usr/share/aclocal is not owned by this package and
     -devel package has some files under this directory.

     Generally this status is not allowed, however, it is still under
     discussion about
     * should all packages which have some files under /usr/share/aclocal
       require "automake" to satisfy the directory ownership requirement?
     * or should all packages own this directory?
     * or should filesystem  own this directory? 
     So for now I leave this at it is.

-- 
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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]