GNU Libtool - Patches: patch #6448, [MSVC 7/7] Add MSVC Support
You are not allowed to post comments on this tracker with your current authentication level.
patch #6448: [MSVC 7/7] Add MSVC Support
Submitter: | Peter Rosin <pekberg> | ||
Submitted: | Tue 04 Mar 2008 10:04:53 AM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | None | Privacy: | Public |
Assigned to: | None | Open/Closed: | Open |
( Jump to the original submission )
Wed 09 Sep 2009 06:31:45 AM UTC, comment #8: |
Matthias Pleh <nocide> |
Tue 08 Sep 2009 07:01:47 PM UTC, comment #7: Hi Matthias!
Ok, I have tried that cccl script, but it still fails miserably to even build libtool. So naturally I have no testsuite results. The failure looks the same (on a cursory look) on master and on the pr-msvc-support branch so I do not appear to have made things worse...
This cccl script is not targeting MSYS (it uses single forward slashes) and can therefore only be used from cygwin. It is also a fact that this cccl script uses cygpath to convert incoming posix paths to windows paths, which libtool is also doing when cccl is detected on cygwin. That looks like a major incompatibilty to me. The provided ccclibtool wrapper isn't improving things for me.
I still need someone to show me a recipe for how to build libtool with (some incarnation of) cccl.
Cheers,
|
Peter Rosin <pekberg>![]() |
Tue 08 Sep 2009 01:53:34 PM UTC, comment #6: Peter, you have mentioned, that you wanted to test this path with cccl, but doesn't worked for you. I don't know if this is still an issue but there exists a new patch for cccl mentioned on this site: http://folti.blogs.balabit.com/2009/08/compiling-autoconfmake-using-sources.html
I hope this helps to improve the MSVC support, because for now this is a must have to introduce the use of open source libraries in our company!
Cheers,
|
Matthias Pleh <nocide> |
Sun 09 Mar 2008 06:29:17 PM UTC, comment #5: Thanks! But don't hesitate to report back if you don't have any issues... :-)
Cheers,
|
Peter Rosin <pekberg>![]() |
Sun 09 Mar 2008 05:49:21 PM UTC, comment #4: Peter, I've installed that patched libtool system-wide on gentoo/x86_64 laptop on which I'm doing my current gstreamer development, I'll be re-libtool everything with it and will report any issues. |
Edward Hervey <bilboed> |
Sat 08 Mar 2008 11:19:10 PM UTC, comment #3: Thanks very much for the feedback,
I'm glad to hear about the success! Previously there has been requests to test how this patch series behaves on other systems (which are not supposed to be affected). So, you can help by checking for regressions on whatever other systems you have available.
In particular, testing for regressions using the cccl script was requested, but I failed to get cccl to do anything useful in the libtool context. So, any cccl user out there could also help with testing.
I think the lib-as-archiver patches (i.e. 1/7 and 6/7) are the most intrusive in the series and they should perhaps be held back. IIRC, the patches work well with AR=ar as well, so that would not hinder using MSVC as the compiler/linker. You can test that as well, if you like. Leaving out AR=lib should be equivalent to saying AR=ar.
Cheers,
|
Peter Rosin <pekberg>![]() |
Sat 08 Mar 2008 06:09:26 PM UTC, comment #2: Hi, I'd like to give my feedback on these 7 patches (applied against trunk svn).
They work amazingly well, I have been compiling gstreamer and dependencies under msys using msvc... and I had to remove all the 'hacks' I previously added to the makefiles to make it compile properly.
Apart from being libtool-ized, they don't require ANY makefile/configure modification to be compiled with msvc under msys (thanks for fixing all the cygwin vs msys issues also).
Awesome work ! I don't know what more is needed to push these patches in, if you need more feedback, logs, etc... please ask.
|
Edward Hervey <bilboed> |
Tue 04 Mar 2008 10:23:47 AM UTC, comment #1: I have no problems with this patch series on either mingw, nor
I have not found a functioning cccl to test with. I have tried
Niether cccl can even build libtool. I have tried to patch
(I used cygwin and ../configure CC=cccl CXX=cccl LD=cccl F77=no FC=no GCJ=no)
Annotated testsuite with these patches:
On MSYS and MSVC 12.00.8804 (MSVC 6) with:
The problems with the exported variables can be solved with the
PASS: tests/link.test
|
Peter Rosin <pekberg>![]() |
Tue 04 Mar 2008 10:04:53 AM UTC, original submission:
Add MSVC support. Some of this stuff would probably benefit
|
Peter Rosin <pekberg>![]() |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Ok, I see. So we need a working version of cccl or a replacement. I try to make a deeper look into this issue.