[Patch] translate response file names

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[Patch] translate response file names

Charles Wilson-8
Many of the GNU toolchain executables (gcc, binutils) accept "response
file" arguments:

        ar cru lib1.a @list-of-objects.txt

where the specified file is parsed, one line at a time, to generate
additional command line arguments. This way, the tools can work around
limitations in command line length.  However, the following currently
doesn't work, if 'ar' is the MinGW version, and the command is executed
in an MSYS shell:

$ /mingw/bin/ar cru lib1.a @/home/me/fnlist.txt
C:\MinGW\bin\ar.exe: @/home/me/fnlist.txt: No such file or directory

Because the native ar.exe has no idea where "/home/me/fnlist.txt" is.
The attached patch teaches msys to translate arguments that start with
'@' from unix to w32 when launching native applications (just like it
currently translates -L/a/unix/path)

2010.09.04  Charles Wilson  <...>

       * path.cc (msys_p2w): Support conversion of @file
       arguments.

OK to commit?

--
Chuck

------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:

Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Mingw-msys mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/mingw-msys

response-files.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [Patch] translate response file names

Cesar Strauss-2
> 2010.09.04  Charles Wilson<...>
>
>         * path.cc (msys_p2w): Support conversion of @file
>         arguments.

Patch looks good, seems to work fine.


> OK to commit?
>

OK, thanks.

It seems your CVS tree needs an update. The current revision for path.cc
is 1.53.

Thanks,
Cesar


------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:

Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Mingw-msys mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/mingw-msys
Reply | Threaded
Open this post in threaded view
|

Re: [Patch] translate response file names

Charles Wilson-8
On 9/6/2010 9:51 PM, Cesar Strauss wrote:
>> OK to commit?
>>
>
> OK, thanks.
>
> It seems your CVS tree needs an update. The current revision for path.cc
> is 1.53.

Updated, merged, and committed.

--
Chuck


------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:

Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Mingw-msys mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/mingw-msys