Re: more bugs in branch-2-0/HEAD

[Gary V. Vaughan]

Hi Peter,

Peter O'Gorman wrote:
> Ralf Wildenhues wrote:
> | lt_dlcaller_register takes zero args in branch-1-5, 2 in HEAD.
> | This sucks.  CVS HEAD m4 wants two args, but does not test for this
> | (and thus loves to use an installed ltdl from branch-1-5 over its own
> | convenience one from HEAD).
> |
> | We need to kill this in order to avoid being backward incompatible.
> | I can only hope no packages other than m4 depend on this already.  :(
>
> Looked at this (as it is a showstopper) and agree. Added api (perhaps
> lt_dlcaller_set_iface(const char*,lt_dlhandle_interface *); or 
> something) to
> set the info and remove the arguments here.
>
> Gary?

I would deprecate the old api:

        lt_dlcaller_id lt_dlcaller_register (void);

And make the new call:

        lt_dlcaller_id lt_dlcaller_register_iface (const char *...

This should be a simple matter of tweaking the docs, renaming the 
current /lt_dlcaller_register/ to \&_iface, and then factoring out
the guts of the new \&_iface into a backwards compatible
lt_dlcaller_register.

I think the docs should relegate mention of the old style
lt_dlcaller_register to a footnote that says it is still present
but deprecated.

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

signature.asc
Description: OpenPGP digital signature