Re: [patch #6448] [MSVC 7/7] Add MSVC Support

[Ralf Wildenhues]

Hi Peter,

* Peter Rosin wrote on Wed, Aug 13, 2008 at 12:41:04PM CEST:
> 2008-08-13  Peter Rosin  <address@hidden>
>
>       * libltdl/m4/libtool.m4 (_LT_LINKER_SHLIBS)
>       [cygwin, mingw, pw32, cegcc] <cl*>: Indicate that reloadable
>       objects does not work.
>       * tests/duplicate_conv.at: Skip last test if reloadable
>       objects does not work.

Please add "Set to @code{false} on systems that cannot create
reloadable objects" to the reload_cmds documentation in libtool.texi.

OK with nit below addressed.

Thanks,
Ralf

PS: yes, all the other new tag variables need documenting in the manual,
too, before the branch can be merged into master.

> --- a/libltdl/m4/libtool.m4
> +++ b/libltdl/m4/libtool.m4
> @@ -4821,6 +4821,7 @@ _LT_EOF
>           mt -manifest "@address@hidden" -outputresource:"@address@hidden";
>           $RM "@address@hidden";
>         fi'
> +     reload_cmds=false

Why is this not set in _LT_CMD_RELOAD?  That's where reload_cmds is set
(exceptions not invalidating the rule ;-)

>       ;;
>        *)
>       # Assume MSVC wrapper