Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 220210
Summary: | Review Request: krename - Powerful batch file renamer | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Michał Bentkowski <mr.ecik> |
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | rdieter |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-12-27 19:41:49 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Michał Bentkowski
2006-12-19 18:28:01 UTC
I will review this later (In reply to comment #0) > Note that package owns whole %{_datadir}/icons/locolor directory This package _should_ own this directory First review of this package. A. From http://fedoraproject.org/wiki/Packaging/Guidelines : * BuildRequires ----------------------------------------------------------- Requires: hicolor-icon-theme ----------------------------------------------------------- - Usually this is regarded as "not needed to be written" because hicolor-icon-theme is generally considered as a type of rpms like "filesystem" and many GUI packages depend on this package directly/indirectly. * Documentation The following documentations include non-UTF8 characters. Consider them to UTF-8 characters. ----------------------------------------------------------- /usr/share/doc/krename-3.0.13/ChangeLog: ISO-8859 English text /usr/share/doc/krename-3.0.13/TODO: Non-ISO extended-ASCII English text, with very long lines ----------------------------------------------------------- * Desktop files ----------------------------------------------------------- Categories=Application;Utility;Qt;KDE; ----------------------------------------------------------- Category "Application" is deprecated and so this should be removed. ----------------------------------------------------------- [tasaka1@dhcp158 krename]$ desktop-file-validate /usr/share/applications/fedora-krename.desktop /usr/share/applications/fedora-krename.desktop: warning: The 'Application' category is not defined by the desktop entry specification. Please use one of "AudioVideo", "Audio", "Video", "Development", "Education", "Game", "Graphics", "Network", "Office", "Settings", "System", "Utility" instead ----------------------------------------------------------- * Scriptlets requirements ----------------------------------------------------------- %{_datadir}/icons/hicolor/*/apps/%{name}.png ----------------------------------------------------------- This requires updating of GTK+ icon cache (desribed in the following). http://fedoraproject.org/wiki/Packaging/ScriptletSnippets Actually I cannot see icons on KRename menu entry. * File and Directory Ownership - On my system, the following directories are not owned by any packages. ------------------------------------------------------------ /usr/share/apps/konqueror/ /usr/share/apps/konqueror/servicemenus/ ------------------------------------------------------------ ... because I am a GNOME user and I don't have kdebase installed. I think this package should own these directories as this package can be used for non-KDE users, too. B. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines : (= this is okay, except for things written in A) Spec URL: http://mutebox.net/~ecik/krename/krename.spec SRPM URL: http://mutebox.net/~ecik/krename/krename-3.0.13-2.src.rpm * Thu Dec 21 2006 Michał Bentkowski <mr.ecik at gmail.com> - 3.0.13-2 - Fix encoding of ChangeLog and TODO files - Fix desktop file issue - Add %%post and %%postun sections - Make %%{_datadir}/apps/konqueror owned by this package I didn't change hicolor-icon-theme dependency, because in my opinion it is good if it's present. Well, for TODO file, some characters are fixed, however there are still some garbage characters which are not correctly seen by "less" command. I cannot figure out in what coding these characters are encoded.... Maybe it may be better that we leave TODO file as it is. Other things are okay. --------------------------------------------------------- This package (krename) is APPROVED by me. Please rebuild this... I would really like to, but currently I'm encountering problems with my internet connection, so I'm simply unable to upload it. I hope I'll be within a week. I managed to build this package today. Closing This package has not had a proper review, and wont be branched until it has. Though I am not willing, I will attach a detail for sumbitter's benefit. >From http://fedoraproject.org/wiki/Packaging/Guidelines = Naming of this package is good = License documentation is included = License is OSI approved = License documentation is actually consistent with the ones actually used in source files. = No shareware data is included = No patents issue is found = This is not a emulators = This is not a binary firmware = No libexecdir files is needed as no wrapper scripts are needed = rpmlint is silent = Changelog entry is proper = Tag is correctly used = Build root tag is okay = Generally "Requires: hicolor-icon-theme" description is not needed, however, I don't object to this. = Dependencies other than libraries' dependencies automatically added by rpmbuild is not necessary = BuildRequires is enough: mockbuild is okay for FC-devel = No redundant BuildRequires is described = Summary and description is okay = Documentation Encodings are fixed (according to my suggestion) = Needed documentation - AUTHORS - COPYING - ChangeLog - README - TODO --- all included (in main package) = Mock build log says that fedora specific compilation flags are correctly passed (checked by grep -v FORTIFY MOCK-krename.log ) = No static libraries nor .la files = There is no libraries duplicate of system libraries = /usr/lib/rpm/check-rpath-worker `rpm -ql krename` does not complain = No conf file = Desktop description is okay desktop-file-varidate does not report any error = desktop-file-install correctly used = Macros correctly used = No mixed use of %{buildroot} <-> $RPM_BUILD_ROOT = %makeinstall not used = Locale files are handled by %find_lang = Timestamps are correctly kept for - xml/html - gettext mo files - png file (checked by `rpm -qilvv --changelog --scripts krename) = Parallel build okay = For scriplets -> According to http://fedoraproject.org/wiki/Packaging/ScriptletSnippets = No shared libraries, ldconfig not needed = No services = No GConf = No Texinfo = No Scrollkeeper = mime type is not needed nor described in desktop file (desktop update is not needed, proper) = mimetype xml is not included = files are installed under %{_datadir}/icons/hicolor -> GTK+ icon cache updating is needed --- correctly handled!! = No fonts = No conditional dependencies = Mockbuild is okay, this means that non-root users' rebuild should work = No content which cannot be accepted in FE is not included = Unowned directory - /usr -> filesystem - /usr/bin -> filesystem - /usr/share -> filesystem - /usr/share/applications -> filesystem - /usr/share/apps -> kdelibs - /usr/share/doc -> filesystem - /usr/share/doc/HTML -> kdelibs - /usr/share/doc/HTML/en -> kdelibs - /usr/share/icons -> redhat-artwork - /usr/share/icons/hicolor -> hicolor-icon-theme - /usr/share/icons/hicolor/??x?? -> hicolor-icon-theme - /usr/share/icons/hicolor/??x??/apps -> hicolor-icon-theme = all okay = Owned directory - /usr/share/apps/konqueror - /usr/share/apps/krename - /usr/share/doc/HTML/en/krename - /usr/share/icons/lcolor = all are not owned by other packages needed by this package ( as for /usr/share/apps/konqueror, this is owned by kdebase, however, this package can be used for NON-KDE user so owning this directory is okay and recommended) = This is no web app and /var/www is not used Then from http://fedoraproject.org/wiki/Packaging/ReviewGuidelines = rpmlint for source is silent = rpmlint for binary rpm is silent = rpmlint for installed rpm is silent = Naming is okay (described above) = Consistency for package guideline is checked above = License is okay (described above) = License documentation included (described above) = Actually I don't know the deferrence between American/British/Other English in detail...... = I can read this spec file with ease = Downloading all sources (one) from described URLs succeeded = md5sum values are same = mockbuild is okay for FC-devel i386 = BuildRequires is okay (described above) = locale handling okay (described above) = ldconfig not needed (described above) = relocable description is not used = Directory ownership is okay (described above) = permission is okay - checked by rpmlint and rpm -qilvv krename = %clean section handled properly = macro usage is okay (described above) = code/content issue is no problem (described above) = No large documentation is included in source tarball and -doc subpackage is not needed = -devel subpackage is not needed = .la files/static archives are not included (described above) = desktop file is correctly installed (described above) = directory ownership is handled correctly (described above) Well, other thing I have noticed = mock build log is okay = file `rpm -ql krename` is no problem = rpm -qilvv --changelog --scripts krename is okay = ( for f in `rpm -ql krename` ; do if file $f | grep -q text ; then echo $f ; done ) | xargs less is okay = It seems that this app works and no segv happened for now = ( for f in `rpm -ql krename` ; do if file $f | grep -q image l then echo $f ; done ) | xargs display is okay = w3m /usr/share/doc/HTML/en/krename/index.html is okay APPROVED |