I would Cc: the cygwin list, but it always bounces my messages...

On Fri, Sep 15, 2000 at 01:55:16PM +0300, Paul Sokolovsky wrote:
> Hello Gary,

Hi!

> Unfortunately, there's no online
> archive of libtool-patches (why?), so I can't look there.

Wierd I added it to geocrawler *ages* ago, bt they haven't picked it
up yet.  I've just added it *again*.  Maybe this time, eh?

Also the fsf is moving the mailing lists to a new machine, and
changing to mailman.  If an archive isn't part of this arrangement,
I'll ask to have one made.

> GVV> Agreed in principle, however we need to be careful of 8.3 again =(O| I
> GVV> guess this means special casing DJGPP.
> 
>      But how that influence DJGPP or other DOS ports of gcc? That
> change applies only to win32 hosts (in fact, only to gnu-win32, I
> intentionally left hyphen-delimited naming for non-gcc's).

That's true.  Doh!

> >>   Note comented line with s/^lib/cyg/ - that's about my proposal for
> >> using cyg prefix for cygwin dlls.
> 
> GVV> I would like a patch that allows an extra libtool option to name the
> GVV> dll part of a win32 shared library as specified by the developer.
> 
>      But my original patch (hm, below) would allow to save sensibility
> for something more useful. Don't provide excess configurability when
> it's hardly needed.

But it is.  Chris Faylor has specifically denounced the cygfoo.dll
convention, but Chuck Wilson (and presumably yourself) would like an
option to use it for your own ports.  It is also more general, rather
than adding a bunch more special cases to libtool, porters can add
'-soname cygfoo.dll' to foo_LDFLAGS if they wish.

> (Automake won't generate that option specially for
> gnu-win32, will it?

Nope.  Nor should it.

> And when I would like DLL name to differ from
> standard, I can simply rename it after building.) YMMV

Only if you're prepared to rebuild the import library too.

>    if test "$ac_cv_prog_gcc" = yes; then
> -    library_names_spec='${libname}`echo ${release} | sed -e 
>'s/[.]/-/g'`${versuffix}.dll'
> +    case "$host_os" in
> +#     cygwin*) library_names_spec=`echo ${libname} | sed -e 
>'s/^lib/cyg/'`'${release}${versuffix}.so $libname.dll.a' ;;
> +      pw32*)   library_names_spec='${libname}${release}${versuffix}.so 
>$libname.dll.a' ;;
> +      *)       library_names_spec='${libname}${release}${versuffix}.dll 
>$libname.dll.a' ;;
> +    esac
>    else
>      library_names_spec='${libname}`echo ${release} | sed -e 
>'s/[.]/-/g'`${versuffix}.dll $libname.lib'
>    fi

Looks good to me.  Could you regenerate your patch against the current
HEAD release and repost please?

Cheers,
        Gary.
-- 
  ___              _   ___   __              _         mailto: [EMAIL PROTECTED]
 / __|__ _ _ ___ _| | / / | / /_ _ _  _ __ _| |_  __ _ ___       [EMAIL PROTECTED] 
| (_ / _` | '_|// / |/ /| |/ / _` | || / _` | ' \/ _` | _ \
 \___\__,_|_|\_, /|___(_)___/\__,_|\_,_\__, |_||_\__,_|//_/
home page:  /___/                      /___/                  gpg public key:
http://www.oranda.demon.co.uk           http://www.oranda.demon.co.uk/key.asc

Reply via email to