libtool-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take


From: Peter Rosin
Subject: Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)
Date: Thu, 26 Aug 2010 08:26:51 +0200
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.8) Gecko/20100802 Thunderbird/3.1.2

Hi,

[Full quoting since I added address@hidden

Den 2010-08-25 22:28 skrev Charles Wilson:
> [Added Peter]
> 
> On 8/25/2010 4:17 PM, Ralf Wildenhues wrote:
>> * Charles Wilson wrote on Wed, Aug 25, 2010 at 10:15:55PM CEST:
>>> On 8/25/2010 3:50 PM, Ralf Wildenhues wrote:
>>>> just to let you know that I am working on reviewing your patch.
>>>> But it is not quick to digest ...
>>>
>>> I know, it is a beast.  Fortunately, it has been picked apart in the
>>> past, so if you go back and review the threads I posted (which,
>>> itself, will take some time to do) that may help make the actual
>>> review of the current iteration a little less arduous.
>>>
>>> I knew it would be a bear, which is why I didn't want to drop it on
>>> everybody while we were still discussing the sysroot stuff.
>>> Unfortunately, with the 2.2.next deadline approaching, and the
>>> sysroot stuff still unmerged...I was running out of time. Since I
>>> didn't want to hold Peter's MSVC stuff up any longer, AND I know
>>> I'll need a follow-on docu patch or two...I went ahead this week.
>>> Sorry :-(
>>
>> No problem at all, that's what I get from postponing forever.  Do you
>> know how many other non-doc patches are pending from you and Peter?
> 
> I have no idea how many patches Peter has still pending.  In my queue
> there are only a few -- none of which are critical for 2.2.next.
> 
> Two have already been presented, but with objections:
> 
> 1) passing -{static|shared}-{libgcc|libstdc++|...} flags thru libtool
>    to the compiler
> 2) handling of win32 manifest files
> 
> In both cases, a long discussion was spawned which never reached a
> conclusion about what the right thing to do is.  So, I'd just punt on
> both until after the next release. I'll just forward port my existing
> patches, and package custom cygwin|mingw releases as I've been doing
> for the last few years.
> 
> 
> For #1, I suspect TRTTD is a big scary change: for g++ (and probably,
> other gcc language drivers), don't use ld to link, and don't save
> pre/post deps.  Instead, for GNU tools, use the language drivers
> to link.  Then, there's no problem allowing these flags thru.  (The
> issue is, if you configure libtool *without* the flag, but use it
> *with* the flag -- or vice versa -- then all your pre/post deps are
> wrong.)
> 
> For #2, I'd prefer to wait until after Peter's MSVC stuff is
> completely merged, and then see if the GNU toolchains for win32 can
> piggy back off of any of his machinery.
> 
> Other than that, there are the following (not yet written) patches:
> 3) docu for this patch
> 4) tests for my earlier monster patch merged a few months ago ("allow
> -dlpreopen with -disable-shared", mostly concerned with getting the
> correct symbol exports when -{enable|disable}-{static|shared}).

This is my current queue of libtool patches. They need more work.
In particular, I don't know if 0008-Slashify-instead-of-backslashify
is even remotely acceptable. The subject has been up before and I
think Chuck had some issue with it, but don't remember what. I Can do
some searching in my mailbox if that's important...

Further, my patches regresses library searching, I think due to paths
being converted from posix form to win32 form too early and then
something fails to find dependent libraries. Possibly other problems
too?

Perhaps most interesting are the patches
0002-Add-path-conversion-from-build-to-toolchain
0005-Convert-file-names-to-toolchain-format-in-NM-and-AR
which *should* fix stresstest.at on MSYS (not confirmed, due to the above
problems).

Maybe 0001-Move-path-conversion-functions-earlier-in-the-libtoo
is needed for 0002 to apply cleanly, but it should not be needed
in principle.

Anyway, here they are, just so that you can see where I'm at...

Cheers,
Peter

Attachment: 0001-Move-path-conversion-functions-earlier-in-the-libtoo.patch.txt
Description: Text document

Attachment: 0002-Add-path-conversion-from-build-to-toolchain.patch.txt
Description: Text document

Attachment: 0003-Convert-file-names-to-toolchain-format-when-linking.patch.txt
Description: Text document

Attachment: 0004-Translate-dependent-libraries-using-func_to_tool_pat.patch.txt
Description: Text document

Attachment: 0005-Convert-file-names-to-toolchain-format-in-NM-and-AR-.patch.txt
Description: Text document

Attachment: 0006-Use-func_to_tool_path-instead-of-fix_srcfile_path.patch.txt
Description: Text document

Attachment: 0007-Convert-POSIX-paths-to-toolchain-format-for-MSVC.patch.txt
Description: Text document

Attachment: 0008-Slashify-instead-of-backslashify.patch.txt
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]