Last Comment Bug 327469 - "argument list too long" when compiling javaxpcom
: "argument list too long" when compiling javaxpcom
Status: RESOLVED FIXED
[nvn-dl]
: fixed1.8.0.2, fixed1.8.1
Product: Core Graveyard
Classification: Graveyard
Component: Java to XPCOM Bridge (show other bugs)
: Trunk
: x86 All
: -- normal (vote)
: mozilla1.8.1
Assigned To: Benjamin Smedberg AWAY UNTIL 2-AUG-2016 [:bsmedberg]
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-02-16 07:48 PST by Benjamin Smedberg AWAY UNTIL 2-AUG-2016 [:bsmedberg]
Modified: 2014-09-24 05:43 PDT (History)
2 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Use @java.files, rev. 1 (2.97 KB, patch)
2006-02-16 07:50 PST, Benjamin Smedberg AWAY UNTIL 2-AUG-2016 [:bsmedberg]
jhpedemonte: review+
Details | Diff | Splinter Review

Description Benjamin Smedberg AWAY UNTIL 2-AUG-2016 [:bsmedberg] 2006-02-16 07:48:07 PST
Patch coming up shortly to use an @file to specify the filenames to compile. I'm not sure why I'm seeing this now and not other times, it's perhaps related to what shell I'm making in, but this is a more reliable solution in general because many platforms have hard limits on the length of a command line.
Comment 1 Benjamin Smedberg AWAY UNTIL 2-AUG-2016 [:bsmedberg] 2006-02-16 07:50:21 PST
Created attachment 212115 [details] [diff] [review]
Use @java.files, rev. 1
Comment 2 Benjamin Smedberg AWAY UNTIL 2-AUG-2016 [:bsmedberg] 2006-02-16 07:50:48 PST
http://java.sun.com/j2se/1.4.2/docs/tooldocs/windows/javac.html#commandlineargfile
Comment 3 jhp (no longer active) 2006-02-16 08:06:25 PST
Comment on attachment 212115 [details] [diff] [review]
Use @java.files, rev. 1

Odd, I haven't had any problems with xargs on any of the 3 major platforms.  But this is an elegant solution.  Just need to add "java.files" to GARBAGE.  With that, r=jhpedemonte.

We should also get this on the branches.
Comment 4 jhp (no longer active) 2006-02-23 14:37:46 PST
bsmedberg checked this patch in to trunk and branches.

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