Issue Details (XML | Word | Printable)

Key: HADOOP-3477
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Adam Heath
Reporter: Adam Heath
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Hadoop Common

release tar.gz contains duplicate files

Created: 01/Jun/08 11:09 PM   Updated: 22/Aug/08 07:48 PM
Return to search
Component/s: build
Affects Version/s: 0.16.4, 0.17.0
Fix Version/s: 0.17.1

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works hadoop-tar.patch 2008-06-01 11:11 PM Adam Heath 0.6 kB

Hadoop Flags: Reviewed
Resolution Date: 11/Jun/08 09:56 PM


 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Adam Heath added a comment - 01/Jun/08 11:10 PM
Simple patch, one line.

Adam Heath added a comment - 01/Jun/08 11:13 PM
More details:

==
adam@zoot:~/code/upstream/nutch$ tar tzvf hadoop-0.17.0.tar.gz |awk '{print $6}'|sort|uniq -c|grep -v '1 '
2 hadoop-0.17.0/contrib/hod/bin/VERSION
2 hadoop-0.17.0/contrib/hod/bin/checknodes
2 hadoop-0.17.0/contrib/hod/bin/hod
2 hadoop-0.17.0/contrib/hod/bin/hodcleanup
2 hadoop-0.17.0/contrib/hod/bin/hodring
2 hadoop-0.17.0/contrib/hod/bin/ringmaster
==

This occurs because those files are included twice in build.xml, because there is no explicit exclude on those dirs, like there is for the others. The attached patch(1-line) adds such an exclude.

This is keeping me from using the downloaded tarball as the orig.tar.gz for a debian package.


Doug Cutting added a comment - 02/Jun/08 06:13 PM
I just committed this. Thanks, Adam!

Adam Heath added a comment - 02/Jun/08 07:18 PM
Could you apply the fix to 0.16 as well(creating 0.16.5)? Nutch isn't using 0.17 yet.

Doug Cutting added a comment - 02/Jun/08 07:35 PM
I guess this is a regression, since it was introduced in 0.16.0 by HADOOP-2494. Unless there are objections, I will merge it into the 0.16 and 0.17 branches.

Hudson added a comment - 03/Jun/08 01:26 PM

Hudson added a comment - 19/Jun/08 12:35 PM