Bug 173080 - Review Request: fdupes
Summary: Review Request: fdupes
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Adrian Reber
QA Contact: David Lawrence
URL: http://www.thoughtpolice.co.uk/packag...
Whiteboard:
: 172870 (view as bug list)
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-11-13 20:43 UTC by Simon B
Modified: 2014-03-18 12:10 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-01-09 17:15:18 UTC
Type: ---
Embargoed:
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Simon B 2005-11-13 20:43:07 UTC
Spec Name or Url: http://www.thoughtpolice.co.uk/packages/fedora/fc4/fdupes/fdupes.spec
SRPM Name or Url: http://www.thoughtpolice.co.uk/packages/fedora/fc4/fdupes/fdupes-1.40-1.src.rpm
Description: fdupes is a program for identifying or deleting duplicate files.
Such files are found by comparing file sizes and MD5 signatures, followed by a
byte-by-byte comparison.


Possible problems:

* The Makefile uses /usr/local paths. The package includes a patch to fix that, but perhaps there is a better, recommended way of doing this.

* Upstream includes the license in README, I have asked if this can be separated into a separate file for a later release.

Comment 1 José Matos 2005-11-13 21:57:27 UTC
Notice that fdupes was already submitted to Extras and is waiting 
for review. (What an irony to have a duplicate submission for 
a package whose purpose is to avoid duplicated files). ;-) 
 
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=172870 

Comment 2 Simon B 2005-11-13 22:06:42 UTC
Damn. That's annoying, I use fdupes a lot, and was hoping to be the maintainer.
I asked the author if he was okay with me packaging for extras, and he was. Then
I checked the list of packages, and fdupes wasn't on it. I guess my timing was bad.

I'll withdraw my submission if you want to be the maintainer.

Comment 3 José Matos 2005-11-13 22:16:00 UTC
I did not say that I was the maintainer, I was simply checking my email  
while in the interval of "The Lord of the Rings: The Two Towers", (oops  
another duplication ;-) ).  
  
You should contact Bastien Nocera who submitted the other package. 
That was why I did not closed this bug as duplicated. 
 
And now back to the rest of the movie. ;-) 

Comment 4 Bastien Nocera 2005-11-14 08:59:26 UTC
Damn Channel 4 adverts ;)
Simon, if you're interested in maintaining fdupes, feel free. But please make
sure that you take into account the patch from Adrian in the other bug.

Comment 5 Simon B 2005-11-14 14:42:51 UTC
Great, thanks.

I've uploaded the new spec file and srpm. Same URLs as before.

I noticed that Adrian uses Applications/Files as a Group, which rpmlint doesn't
like. I've assumed rpmlint was wrong, and changed from Applications/System.

Comment 6 Paul Howarth 2005-11-14 16:58:53 UTC
(In reply to comment #5)
> I noticed that Adrian uses Applications/Files as a Group, which rpmlint doesn't
> like. I've assumed rpmlint was wrong, and changed from Applications/System.

The group should be selected from one of the ones listed in the file
/usr/share/doc/rpm-*/GROUPS

Applications/File would seem to be the best candidate (same as for findutils).



Comment 7 Simon B 2005-11-14 17:27:43 UTC
(In reply to comment #6)
> The group should be selected from one of the ones listed in the file
> /usr/share/doc/rpm-*/GROUPS
> 
> Applications/File would seem to be the best candidate (same as for findutils).
> 

Thanks for that. Very useful.
I've put a request in for rpmlint to recognise more groups (bug 173149).

Comment 8 Simon B 2005-11-14 22:06:06 UTC
This is my first package for Extras, and I am seeking a sponsor.

Comment 9 Adrian Reber 2005-11-15 06:54:48 UTC
(In reply to comment #5)
> I noticed that Adrian uses Applications/Files as a Group, which rpmlint doesn't
> like.

That was a typo and should have been Applications/File. So rpmlint was right.

So who is submitting this package now? Bastien or Simon?

Comment 10 Simon B 2005-11-15 16:49:19 UTC
(In reply to comment #9)
> That was a typo and should have been Applications/File. So rpmlint was right.
Fixed.

> So who is submitting this package now? Bastien or Simon?
I am. Is that an offer to sponsor me? :)



Comment 11 Adrian Reber 2005-11-17 18:38:49 UTC
You shouldn't repeat the name of the package in the summary.

Instead of patching the Makefile you could just use install to copy the files to
the correct location without using "make install". This would make the spec a
bit easier to read because the makefile tweaking wouldn't be necessary.

The Group has still to be changed from Applications/Files to Applications/File

Comment 12 Bastien Nocera 2005-11-18 09:29:57 UTC
*** Bug 172870 has been marked as a duplicate of this bug. ***

Comment 13 Simon B 2005-11-22 18:16:29 UTC
Fixed. Latest version on site.

Comment 14 Adrian Reber 2005-11-24 15:47:17 UTC
Please post the link to the updated SRPM. It is not hard to find, but would be
nicer if I don't have to guess.

Looks pretty good so far the only thing which should be added back to the spec
is this line "%{__sed} -i -e "s/-Wall/$RPM_OPT_FLAGS/" Makefile" with which the
RPM_OPT_FLAGS are used to build the binary.

Comment 16 Adrian Reber 2005-11-25 09:13:25 UTC
* builds in mock (FC4 and development)
* rpmlint is happy
* spec looks good
* source matches upstream
* clean installation and removal
* works as expected

APPROVED

I can sponsor you. Do you already exist in the account system?

Comment 17 Simon B 2005-11-25 09:43:13 UTC
Great! Thanks!

Yes, I have an account.

Comment 18 Adrian Reber 2005-12-06 19:56:18 UTC
I cannot find this in the development tree. Have you not requested a build in
the devel tree? If this has been built successfully then please close this bug
according to the documentation in the wiki.

Comment 19 Simon B 2005-12-06 22:55:06 UTC
Hi Adrian,
Thanks for the heads up. It's in now.

Comment 20 Christian Iseli 2006-01-09 13:24:15 UTC
(In reply to comment #19)
> It's in now.

Could you then set this ticket to CLOSED / NEXTRELEASE ?

Thanks.


Comment 21 Björn 'besser82' Esser 2014-03-18 12:00:37 UTC
Package Change Request
======================
Package Name: fdupes
New Branches: el5 el6 epel7
Owners: besser82 hobbes1069

Want to build on EPEL-branches, too.

Comment 22 Gwyn Ciesla 2014-03-18 12:10:07 UTC
Git done (by process-git-requests).


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