libtool-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Gary V. Vaughan
Subject: Re: 276-gary-remove-unusable-deprecated-ltdl-apis.diff
Date: Sun, 18 Sep 2005 11:02:58 +0100
User-agent: Mozilla Thunderbird 1.0.6 (Macintosh/20050716)

Ralf Wildenhues wrote:
> Hi Gary,

Hallo Ralf!

> * Gary V. Vaughan wrote on Sun, Sep 18, 2005 at 12:31:56AM CEST:
> 
>>Okay to apply?
>>
>>Since we are explicity no longer supporting applications that don't
>>update to the new API with libtool-2.0, I've removed the deprecated
>>functions.
> 
> I have not seen a decision to that extent.  Only to kill support for
> lt_dlcaller_register.

Good point.  I was extrapolating.

The section at the end of ltdl.h labeled, "Binary compatibility with
old libltdl" seems redundant -- we've just bumped the revisions such
that 2.0 libltdl will never be runtime loaded into an app linked
against 1.5.x libltdl.

> So, no, I don't see any reason to apply this.

Okay, we have 2 paths:  Remove all deprecated functions and get our
clients to clean up their code, instead of wasting cycles on setting
unused malloc pointers, and seeing a deprecation warning on every
run of a program that is still trying to use the original non-working
mutex APIs.  Recall, that on the transition to 2.0, clients will
already need to fix their code a little and recompile freshly with
libltdl.

Alternatively, we can add more functions that answer to the names
of APIs that were removed in the transition to 2.0 for reasons
explained in my earlier mails, and bug the users with additional
deprecation warnings.

Currently, we do a bit of both which is plain wrong.  Since 2.0 is
plainly a major upgrade, I think the first option makes more sense.

> And don't remove lt_ptr just for fun!

Oh, *blush*.  Yes that was a little rash.  Removing all trace of it from
the documentation is still the right thing IMHO.

>>+     * libltdl/ltdl.h (lt_ptr, 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.

Cheers,
        Gary.
-- 
Gary V. Vaughan      ())_.  address@hidden,gnu.org}
Research Scientist   ( '/   http://tkd.kicks-ass.net
GNU Hacker           / )=   http://www.gnu.org/software/libtool
Technical Author   `(_~)_   http://sources.redhat.com/autobook

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]