Re: broken pkg-config files

[Thomas Dickey]

On Sat, Dec 31, 2011 at 07:55:19PM +0100, Sven Joachim wrote:
> On 2011-12-31 19:31 +0100, Thomas Dickey wrote:
> 
> > On Wed, Dec 28, 2011 at 12:33:50PM +0100, Sven Joachim wrote:
> >> When configuring ncurses with the following options
> >> 
> >> ./configure --enable-widec --enable-pc-files --with-ticlib=tic \
> >>   --with-termlib=tinfo
> >> 
> >> and running "sh misc/gen-pkgconfig", it prints the following:
> >> 
> >> ** creating ncursesw.pc
> >> ** creating panelw.pc
> >> ** creating menuw.pc
> >> ** creating formw.pc
> >> ** creating ncurses++w.pc
> >> ** creating tinfow.pc
> >> ** creating ticw.pc
> >> 
> >> And the ti{nfo,c}w.pc files are not only misnamed but also broken, since
> >> they have -ltinfow and -lticw in the Libs: section, despite the
> >> libraries being called tinfo and tic, respectively.
> >> 
> >> This seems to have been broken recently, probably with the
> >> 2011126/20111210 changes.
> >
> > I'm puzzled: testing with your configure options (to identify the point
> > at which the breakage occurred), I'm seeing only that the tic*.pc and
> > tinfo*.pc files appeared recently - testing for instance 5.9, 20110716,
> > 20100109).
> 
> Ah, that's quite possible.
> 
> > What was the patch-date you were using previously?
> 
> 20111112, and now I see that indeed this patchlevel does not generate
> any tic*.pc or tinfo*.pc with the given options.  That bug is concealed
> in the Debian build, because we also build a version without
> --enable-widec, and that build generates correct tic.pc and tinfo.pc
> files.
> 
> So it had always been broken, actually.

right.  There was something in that general area that I fixed this
year, but it was a regression which was for a different combination
of options:

20111210
        + modify configure script to add ".pc" files for tic- and
          tinfo-libraries, which were omitted in recent change (cf: 20111126).

I noticed it in _my_ build because those files weren't being updated.
But my build uses different options of course, since I'm setting up
libraries that complement the Debian packages.
 
> > I agree that they should be named tic.pc, tinfo.pc for the given options.
> 
> And have -ltic, -ltinfo in the Libs: section.

yes (I'm working on that).

-- 
Thomas E. Dickey <address@hidden>
http://invisible-island.net
ftp://invisible-island.net

signature.asc
Description: Digital signature