Re: 276-gary-remove-unusable-deprecated-ltdl-apis.diff

[Ralf Wildenhues]

Hi Gary,

* Gary V. Vaughan wrote on Mon, Sep 19, 2005 at 01:22:38AM CEST:
> Repost.  Easing back on the lt_ptr removal.

OK, I guess.  We'll hurt a couple of people in any case.
I noticed some packages use some of the deprecated APIs
in their AC_CHECK_LIB tests, so they'll fail to find
libltdl then.  I haven't checked whether they also _need_
those deprecated APIs.  In that case the failure would do
the right thing.  Shows the need to make this prominently
visible to users!

Maybe we should add a couple of public #define's for ltdl API?
Like this:
  #define LTDL_MAJOR  2
  #define LTDL_MINOR  0
  #define LTDL_REVISION  0
  #define LTDL_RELEASE ((LTDL_MAJOR << 16) | (LTDL_MINOR << 8) | LTDL_REVISION)

so packages that have to maintain source-compatibility with both pre-
and post-2.0 can do so without a configure check?  (I haven't thought
much about this, the numbering should probably be done according to
VERSION_INFO).

Thoughts?

Cheers, and thank you!
Ralf

> Index: libtool--devo--1.0/ChangeLog
> ===================================================================
> --- libtool--devo--1.0.orig/ChangeLog
> +++ libtool--devo--1.0/ChangeLog
> @@ -1,5 +1,11 @@
>  2005-09-17  Gary V. Vaughan  <address@hidden>
>  
> +     * libltdl/ltdl.h (lt_dlmutex_register, lt_dlmutex_lock)
> +     (lt_dlmutex_unlock, lt_dlmutex_seterror, lt_dlmutex_geterror)
> +     (lt_dlmalloc, lt_dlrealloc, lt_dlfree): Remove deprecated APIs.
> +     * doc/libtool.tex: Updated.
> +     * NEWS. Updated.
> +
>       * libltdl/ltdl.c (lt_dlcaller_register): Renamed to avoid problems
>       with module visibilty when linked with programs written for the
>       old API.
> Index: libtool--devo--1.0/NEWS
> ===================================================================
> --- libtool--devo--1.0.orig/NEWS
> +++ libtool--devo--1.0/NEWS
> @@ -1,9 +1,16 @@
>  NEWS - list of user-visible changes between releases of GNU Libtool
>  
> -New in 2.1b: 2005-??-??; CVS version 2.1a, Libtool team:
> +New in 1.9h: 2005-??-??; CVS version 2.1a, Libtool team:
>  * New tests for support of Automake subdir-objects.
> -* Support for Portland Group compiler on GNU/Linux.
>  * Fix libltdl on static platforms.
> +* New lt_dlinterface_register, lt_dlinterface_set_data and
> +  lt_dlinterface_get_data libltdl API calls to maintain separation of
> +  concerns between modules loaded by different libraries.
> +* Removed deprecated APIs from libltdl: lt_dlcaller_register,
> +  lt_dlcaller_get_data, lt_dlcaller_set_data, lt_dlmutex_register,
> +  lt_dlmutex_lock, lt_dlmutex_unlock, lt_dlmutex_seterror,
> +  lt_dlmutex_geterror, lt_dlmalloc, lt_dlrealloc, lt_dlfree.
> +* Support for Portland Group compiler on GNU/Linux.
>  * Support for linux-dietlibc (`diet' as well as `diet-dyn', separately).
>  * Shell optimizations which break use of the stdin file descriptor in 
> libtool.
>  * `libtoolize --install' now also installs `install-sh'.
> @@ -13,8 +20,6 @@ New in 2.1b: 2005-??-??; CVS version 2.1
>  * Detection of compiler wrappers like distcc/ccache and $host_alias prefix.
>  * Initial Support for FC (modern Fortran).
>  * Fixed a regression that prevented use of libltdl without autotools.
> -
> -New in 1.9h: 2004-??-??; CVS version 1.9g, Libtool team:
>  * Bug fixes.
>  
>  New in 1.9f: 2004-10-23; CVS version 1.9e, Libtool team:
*snip*