Re: [PATCH] getopt.m4sh generated libtool option parser, and XSI improvements.

2010-06-23 Thread Gary V. Vaughan
I forgot to commit the following chunk before extracting the patch from git: diff --git a/libltdl/config/getopt.m4sh b/libltdl/config/getopt.m4sh index 1487755..76f9d35 100644 --- a/libltdl/config/getopt.m4sh +++ b/libltdl/config/getopt.m4sh @@ -49,6 +49,13 @@ m4_pattern_forbid([^_?m4go_]) ## 1.

Re: MSVC: MSVC needs a hint to force it to compile either as C or C++.

2010-06-23 Thread Peter Rosin
Hi Gary, Den 2010-06-22 14:53 skrev Gary V. Vaughan: In this case, Peter, would you point me at the followup patches that need this preliminary prep work? I had a poke around in your git branch but couldn't (quickly) figure out where this is going... and without that it's hard to do a sane

Re: MSVC: MSVC needs a hint to force it to compile either as C or C++.

2010-06-23 Thread Gary V. Vaughan
On 23 Jun 2010, at 13:56, Peter Rosin wrote: Hi Gary, Hi Peter, Den 2010-06-22 14:53 skrev Gary V. Vaughan: In this case, Peter, would you point me at the followup patches that need this preliminary prep work? I had a poke around in your git branch but couldn't (quickly) figure out where

Re: MSVC: MSVC needs a hint to force it to compile either as C or C++.

2010-06-23 Thread Peter Rosin
Hi Ralf, Den 2010-06-22 22:16 skrev Ralf Wildenhues: * Peter Rosin wrote on Tue, Jun 22, 2010 at 03:07:30PM CEST: So, it's not a bit deal for me to have this one dangling. But that said, it is nice to fire and forget... This one is not fire and forget. I can live with fire, but not with

libtool does not recognize /usr/lib64 as default location for libraries

2010-06-23 Thread Jirka Hladky
Hello, we have run into a problem where package developed on Debian and packaged for Fedora had rpath included in binaries. This is bad and forbidden by rpm rules (rpmlint will mark it as an error). After long discussion with upstream and Fedora packagers mailing list we have found a reason

Re: MSVC: MSVC needs a hint to force it to compile either as C or C++.

2010-06-23 Thread Peter Rosin
Den 2010-06-23 07:39 skrev Gary V. Vaughan: What happened 4 years ago? (pr-msvc-support branched on 2008-08-02) I started working in this problem space ca 2003-2004, and proposed a very rough patch 2005. Ralf help immensely and it all got into a much better shape. 2006 was the first time my

Re: MSVC: MSVC needs a hint to force it to compile either as C or C++.

2010-06-23 Thread Peter Rosin
Den 2010-06-22 22:16 skrev Ralf Wildenhues: This one is not fire and forget. I can live with fire, but not with forget, because the right place to fix this issue is not within Libtool. So, let's skip this for libtool and assume that it is fixed elsewhere, at least for now. It's not vital and

Re: MSVC: MSVC needs a hint to force it to compile either as C or C++.

2010-06-23 Thread Charles Wilson
On 6/23/2010 3:57 AM, Peter Rosin wrote: For those that didn't know, we have these cl options to play with: /Tpsource file compile file as .cpp /Tcsource file compile file as .c /TP compile all files as .cpp /TC compile all files as .c (but using - instead of / to not go insane

Re: libtool does not recognize /usr/lib64 as default location for libraries

2010-06-23 Thread Olly Betts
On 2010-06-23, Jirka Hladky jhla...@redhat.com wrote: we have run into a problem where package developed on Debian and packaged for Fedora had rpath included in binaries. This is bad and forbidden by rpm rules (rpmlint will mark it as an error). After long discussion with upstream and

Re: libtool does not recognize /usr/lib64 as default location for libraries

2010-06-23 Thread Peter O'Gorman
On 06/23/2010 09:12 AM, Olly Betts wrote: I posted just such a patch exactly 3 years ago today (coincidentally) - here's the thread: http://thread.gmane.org/gmane.comp.gnu.libtool.general/8339/focus=8345 Back then Ralf said Olly's solution should not be forgotten, though, but it seems it has

MSVC: Add MSVC support.

2010-06-23 Thread Peter Rosin
Hi! This is basically Add MSVC support. 9c848f1e8acdf435d019aa80f3f0b0aeb45613f7 but with some things removed, which makes it depend on the proposed changes to the compile script in automake. This patch should be a NOP unless you are using the Microsoft C/C++ Compiler (aka MSVC, aka cl) and

MSVC: For MSVC, embed the manifest as a resource in the executable.

2010-06-23 Thread Peter Rosin
Hi! This is For MSVC, embed the manifest as a resource in the executable. 9f550cb81d4dfe4fc8962f23a7eccb1152e5c4a5 and the relevant part of patch msvc-documentation.patch 06cfce005204bb8ca212aadab38b38c0202ea04e with @var changed to @code This patch depends on the just posted Add MSVC

Re: MSVC: For MSVC, embed the manifest as a resource in the executable.

2010-06-23 Thread Peter Rosin
Hi! Den 2010-06-23 16:47 skrev Peter Rosin: I'm intend to reply to this message with testsuite results for the Add MSVC support patch plus this patch for various systems. While I didn't see any regressions, I did notice that it didn't go too well with MSVC either. So, I dug around and found a

Re: MSVC: MSVC needs a hint to force it to compile either as C or C++.

2010-06-23 Thread Ralf Wildenhues
* Peter Rosin wrote on Wed, Jun 23, 2010 at 09:57:54AM CEST: For those that didn't know, we have these cl options to play with: One reference: http://msdn.microsoft.com/en-us/library/19z1t1wy.aspx My patches went with -TP/-TC. IIUC AC_FC_SRCEXT outputs FCFLAGS_ext which can be used as this:

Re: MSVC: Add MSVC support.

2010-06-23 Thread Ralf Wildenhues
* Peter Rosin wrote on Wed, Jun 23, 2010 at 04:26:22PM CEST: For this patch I'm looking for an ok to commit, but I'm going to hold the actual commit until I have done a couple of more things: * I have a similar ok for one more patch (.manifest related,

Re: MSVC: For MSVC, embed the manifest as a resource in the executable.

2010-06-23 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Wed, Jun 23, 2010 at 04:47:45PM CEST: For MSVC, embed the manifest as a resource in the executable. 9f550cb81d4dfe4fc8962f23a7eccb1152e5c4a5 and the relevant part of patch msvc-documentation.patch 06cfce005204bb8ca212aadab38b38c0202ea04e This patch

Re: MSVC: For MSVC, embed the manifest as a resource in the executable.

2010-06-23 Thread Ralf Wildenhues
* Peter Rosin wrote on Wed, Jun 23, 2010 at 06:12:43PM CEST: While I didn't see any regressions, I did notice that it didn't go too well with MSVC either. So, I dug around and found a forgotten gem [1], and updated it to apply on top of the previous patch. Result attached. So, I'm asking if

Re: MSVC: Add MSVC support.

2010-06-23 Thread Peter Rosin
Den 2010-06-23 20:29 skrev Ralf Wildenhues: * Peter Rosin wrote on Wed, Jun 23, 2010 at 04:26:22PM CEST: For this patch I'm looking for an ok to commit, but I'm going to hold the actual commit until I have done a couple of more things: * I have a similar ok for one more patch (.manifest