[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [patch #6448] [MSVC 7/7] Add MSVC Support
From: |
Ralf Wildenhues |
Subject: |
Re: [patch #6448] [MSVC 7/7] Add MSVC Support |
Date: |
Mon, 11 Aug 2008 20:04:22 +0200 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
* Peter Rosin wrote on Sat, Aug 09, 2008 at 11:03:50AM CEST:
> Ralf Wildenhues skrev:
>> Does this BTW mean that the manifest file ends up being part of the .exe
>> and thus needs not be explicitly installed and uninstalled? That would
>> be nice.
>
> Yes, the mt command embeds the manifest as a resource, so nothing extra to
> install.
Cool.
>>> 2008-08-07 Peter Rosin <address@hidden>
>>>
>>> * libltdl/m4/libtool.m4 (_LT_LINKER_SHLIBS) <postlink_cmds>:
>>> New tag variable.
>>> [ cygwin, mingw, cegcc ] <cl*>: Embed the manifest file as a
>>> resource.
>>> (_LT_LANG_CXX_CONFIG): Likewise.
>>> * libltdl/config/ltmain.m4sh (func_mode_link): After linking
>>> a program, replace @OUTPUT@ in $postlink_cmds and then execute
>>> the commands.
>>
>> Not so bad; can we rely on 'mt' being present on all systems that have
>> cl? Have you tried this patch on a setup where the program is relinked
>> at install time (does this ever happen with cl)?
> No, the manifest was introduced in MSVC 7 or 8 or something. MSVC 6 doesn't
> generate any manifests. That's one reason for the postlink_cmds to have
> if test -f @address@hidden;
Alright.
> That way, mt is only tried if there is a manifest file to embed, which also
> covers the case when the manifest file isn't needed (and thus not
> generated). If MSVC has generated a manifest, it better also have mt.
OK.
> It might be a problem if you build inside the src tree and a
> foo.exe.manifest is part of the source tree, and the .exe isn't generated
> in the .libs subdir. I guess you just shouldn't do that. Doctor, it hurts
> when I etc...
Good enough for me ATM. Please go ahead and apply this on the branch,
thanks.
Cheers,
Ralf
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, (continued)
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/12
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/07
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/07
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/09
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/09
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support,
Ralf Wildenhues <=
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/13
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/14
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/15
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/16
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/17
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Ralf Wildenhues, 2008/08/19
- Re: [patch #6448] [MSVC 7/7] Add MSVC Support, Peter Rosin, 2008/08/19
- RE: [patch #6448] [MSVC 7/7] Add MSVC Support, Markus Duft, 2008/08/20