Re: texinfo-6.0.91 pretest

[Ken Brown]

On 1/2/2016 3:07 AM, Eli Zaretskii wrote:
> > Cc: address@hidden
> > From: Ken Brown <address@hidden>
> > Date: Fri, 1 Jan 2016 17:45:28 -0500
> >
> > There is a similar linking problem on Cygwin, but the following patch fixes it:
> >
> > Index: tp/Texinfo/Convert/XSParagraph/Makefile.am
> > ===================================================================
> > --- tp/Texinfo/Convert/XSParagraph/Makefile.am  (revision 6910)
> > +++ tp/Texinfo/Convert/XSParagraph/Makefile.am  (working copy)
> > @@ -74,7 +74,7 @@
> >
> >   XSParagraph_la_CFLAGS += -DVERSION=\"$(VERSION)\" 
> > -DXS_VERSION=\"$(XSPARAGRAPH_INTERFACE_VERSION)\"  "-I$(PERL_INC)"
> >
> > -XSParagraph_la_LDFLAGS=-no-undefined -avoid-version -module 
> > $(PERL_CONF_cccdlflags)
> > +XSParagraph_la_LDFLAGS=-no-undefined -avoid-version -module 
> > $(PERL_CONF_cccdlflags) -L$(PERL_INC) -lperl
>
> I don't think it's correct to use literally "-L$(PERL_INC) -lperl",
> although it might be in many installations.  The correct values should
> obtained from "perl -V:<variable>", and I already provided the
> variables that should be used here.  E.g., if the user has Active Perl
> installed, like I do, the "-lperl" part should be replaced with
> "-lperl520" (for Perl 5.20.X).

On Cygwin, "perl -V:libperl" returns "libperl='cygperl5_22.dll';", but 
you wouldn't guess from this that you should use "-lperl" for linking 
(i.e., the actual import library is libperl.dll.a).

> Moreover, on Posix hosts I'm not sure it's TRT to link against
> libperl.a, as that's a static library, and Unix doesn't need import
> libraries to link against shared libraries at run time.

Are you sure that using -lperl on Posix hosts would link against 
libperl.a rather than libperl.so?  I thought the whole point of the '-l' 
was so that the linker would choose the appropriate library, depending 
on the context.

Ken