217776 - Build fails with zip error

Status: RESOLVED FIXED

(Other Applications Graveyard :: ChatZilla, defect)

Description

[Ari Pollak]

When compiling Mozilla 1.5b on Solaris, the build fails with the following error:
+++ making chrome /tmp/mozilla/extensions/irc  =>
../../dist/bin/chrome/chatzilla.jar
+++ updating chrome ../../dist/bin/chrome/installed-chrome.txt
+++     content,install,url,jar:resource:/chrome/chatzilla.jar!/content/chatzilla/
+++ updating chrome ../../dist/bin/chrome/installed-chrome.txt
+++    
locale,install,url,jar:resource:/chrome/chatzilla.jar!/locale/en-US/chatzilla/
+++ updating chrome ../../dist/bin/chrome/installed-chrome.txt
+++     skin,install,url,jar:resource:/chrome/chatzilla.jar!/skin/modern/chatzilla/
zip warning: name in zip file repeated: content/chatzilla/lib/js/command-manager.js
zip warning:   first full name: content/chatzilla/lib/js/command-manager.js
zip warning:  second full name: content/chatzilla/lib/js/command-manager.js
 
zip error: Invalid command arguments (cannot repeat names in zip file)
Error invoking zip: 16 at ../../config/make-jars.pl line 156, <STDIN> line 83.
gmake[3]: *** [libs] Error 29
gmake[3]: Leaving directory `/tmp/mozilla/extensions/irc'


I'm using zip 2.3.

Comment 1

[Boris Zbarsky [:bzbarsky]]

To chatzilla.

Comment 2

[Samuel Sieb]

Attachment: remove the extra line

Comment 3

[Robert Ginda]

Comment on attachment 130801 [details] [diff] [review]
remove the extra line

r=rginda, I'm going away on vacation for a few days, please feel free to check
this in.

Comment 4

[Samuel Sieb]

Comment on attachment 130801 [details] [diff] [review]
remove the extra line

I think the bug is probably on the 1.4 branch as well.	Can anyone confirm
that?

Comment 5

[Asa Dotzler [:asa]]

Comment on attachment 130801 [details] [diff] [review]
remove the extra line

a=asa (on behalf of drivers) for checkin to Mozilla 1.5 and Mozilla 1.4.1

Comment 6

[Samuel Sieb]

Comment on attachment 130801 [details] [diff] [review]
remove the extra line

This bug does not exist on the 1.4 branch.

Comment 7

[Samuel Sieb]

checked in on trunk.