Bug 257181 - Review Request: komparator - kompare/sync files
Summary: Review Request: komparator - kompare/sync files
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Chitlesh GOORAH
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2007-08-27 17:20 UTC by Neal Becker
Modified: 2007-11-30 22:12 UTC (History)
2 users (show)

Clone Of:
Last Closed: 2007-08-29 11:07:06 UTC
chitlesh: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Neal Becker 2007-08-27 17:20:50 UTC
Spec URL: https://nbecker.dyndns.org/RPM/komparator.spec
SRPM URL: https://nbecker.dyndns.org/RPM/komparator-0.6-1.fc7.src.rpm
Description: Kompare and merge two folders.
They will be searched for duplicate files and empty folders.

Comment 1 Chitlesh GOORAH 2007-08-27 18:25:54 UTC
I'll take the review.

Comment 2 Chitlesh GOORAH 2007-08-27 18:48:34 UTC
#001: rpmlint issues
W: komparator dangling-relative-symlink /usr/share/doc/HTML/en/komparator/common
W: komparator summary-ended-with-dot Kompare and merge two folders.
E: komparator no-changelogname-tag
W: komparator invalid-license GPL

a) dangling-relative-symlink
this can be ignored

b) summary-ended-with-dot
 remove the "." at the end of 
Summary:        Kompare and merge two folders.

c) no-changelogname-tag
Use the format :

d) invalid-license GPL
in accordance to http://fedoraproject.org/wiki/Licensing, the license should be

#002:  invalid Desktop file when built for rawhide
+ popd
+ mkdir -p /var/tmp/komparator-0.6-1.fc8-root-kojibuilder/usr/share/applications
+ desktop-file-install --vendor fedora --dir
--add-category Development --add-category X-Fedora --delete-original
error: value "komparator %u %u" for key "Exec" in group "Desktop Entry" may
contain at most one
"4159892954", "0.000000" or "%U" field code
desktop-file-install created an invalid desktop file!
error: Bad exit status from /var/tmp/rpm-tmp.68670 (%install)

#003: build requires
kdebase-devel already requires kdelibs-devel.
Remove kdelibs-devel from the BuildRequires:

#004 Desktop file
a)    --add-category X-Fedora \ is not longer needed
b)    --add-category Development \ will add a duplicate menu entry on kmenu.
either use the catogory Utility or Development

#005: useless directory creation
You can remove 
mkdir -p $RPM_BUILD_ROOT%{_datadir}/applications
from the spec as desktop-file-creation does create the directory by itself.

#006 disabling rpath
add the following after the %configure macro
# disabling rpath
sed -i 's|^hardcode_libdir_flag_spec="\\${wl}--rpath
\\${wl}\\$libdir"|hardcode_libdir_flag_spec=""|g' libtool
sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool

Comment 3 Neal Becker 2007-08-28 10:48:23 UTC
#002: I don't know what to do with this one

#006: Is this needed?  I don't get any rpath complaint - and it was configured 
with --disable-rpath.

Please review https://nbecker.dyndns.org/RPM/komparator-0.6-2.fc7.src.rpm

Comment 4 Chitlesh GOORAH 2007-08-28 12:07:06 UTC
(In reply to comment #3)
> #002: I don't know what to do with this one

%{__sed} -i "s|\%u \%u|\%u|" src/%{name}.desktop
%setup -q

> #006: Is this needed?  I don't get any rpath complaint - and it was 
> with --disable-rpath.

no it's not needed. Forget it.

> Please review https://nbecker.dyndns.org/RPM/komparator-0.6-2.fc7.src.rpm

Next time do post the SPEC file as well together with the SRPM

Fix #002 as mentioned above, I'll approve it.

Comment 6 Chitlesh GOORAH 2007-08-28 15:27:43 UTC
The package komparator is approved by ME .

Comment 7 Neal Becker 2007-08-28 19:34:06 UTC
New Package CVS Request

Package Name: komparator
Short Description: kompare/sync files
Owners: nbecker
Branches: devel F-7
Cvsextras Commits: yes

Comment 8 Kevin Fenzi 2007-08-29 01:08:53 UTC
cvs done.

Note You need to log in before you can comment on or make changes to this bug.