broken pkg-config files

[Sven Joachim]

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.

Cheers,
       Sven

_______________________________________________
Bug-ncurses mailing list
Bug-ncurses@...
https://lists.gnu.org/mailman/listinfo/bug-ncurses

[Thomas Dickey-2]

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.

thanks - I'll repair it

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


_______________________________________________
Bug-ncurses mailing list
Bug-ncurses@...
https://lists.gnu.org/mailman/listinfo/bug-ncurses

[Thomas Dickey-2]

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).

What was the patch-date you were using previously?

I agree that they should be named tic.pc, tinfo.pc for the given options.

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


_______________________________________________
Bug-ncurses mailing list
Bug-ncurses@...
https://lists.gnu.org/mailman/listinfo/bug-ncurses

[Sven Joachim]

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.

> I agree that they should be named tic.pc, tinfo.pc for the given options.

And have -ltic, -ltinfo in the Libs: section.

Cheers,
       Sven

_______________________________________________
Bug-ncurses mailing list
Bug-ncurses@...
https://lists.gnu.org/mailman/listinfo/bug-ncurses

[Thomas Dickey-2]

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 <dickey@...>
http://invisible-island.net
ftp://invisible-island.net


_______________________________________________
Bug-ncurses mailing list
Bug-ncurses@...
https://lists.gnu.org/mailman/listinfo/bug-ncurses

[Sven Joachim]

On 2011-12-31 20:07 +0100, Thomas Dickey wrote:

> On Sat, Dec 31, 2011 at 07:55:19PM +0100, Sven Joachim wrote:
>> On 2011-12-31 19:31 +0100, Thomas Dickey wrote:
>> > 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).

Thanks, it's working in the 20111231 patchlevel.  One minor nitpick:
With the given option, tic.pc has

Requires.private: ncursesw, tinfo

Shouldn't it list tinfo only, considering your explanation in
http://lists.gnu.org/archive/html/bug-ncurses/2011-10/msg00010.html ?

Cheers,
       Sven

_______________________________________________
Bug-ncurses mailing list
Bug-ncurses@...
https://lists.gnu.org/mailman/listinfo/bug-ncurses

[Thomas Dickey-2]

On Sun, Jan 01, 2012 at 06:12:01PM +0100, Sven Joachim wrote:

> On 2011-12-31 20:07 +0100, Thomas Dickey wrote:
>
> > On Sat, Dec 31, 2011 at 07:55:19PM +0100, Sven Joachim wrote:
> >> On 2011-12-31 19:31 +0100, Thomas Dickey wrote:
> >> > 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).
>
> Thanks, it's working in the 20111231 patchlevel.  One minor nitpick:
> With the given option, tic.pc has
>
> Requires.private: ncursesw, tinfo
>
> Shouldn't it list tinfo only, considering your explanation in
> http://lists.gnu.org/archive/html/bug-ncurses/2011-10/msg00010.html ?

yes - I overlooked that yesterday when making fixes to the scripts.
(at the time, I was just fixing the naming inconsistencies).

I'll be revisiting this again anyway: it would be nice if I could configure
things so that termcap was supported with the libraries split up.  But in
that special case, libtic contains symbols needed by tinfo.  (I think it
might be possible to use weak symbols to do this, but only have it on a
to-do list so far - I'm reluctant to move things between libraries of
course).  Since Debian isn't making use of termcap, that's an academic
point...

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


_______________________________________________
Bug-ncurses mailing list
Bug-ncurses@...
https://lists.gnu.org/mailman/listinfo/bug-ncurses