Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.bugs.dist > #1198233

Bug#1071581: dialog: stop using libtool-bin

From Helmut Grohne <helmut@subdivi.de>
Newsgroups linux.debian.bugs.dist
Subject Bug#1071581: dialog: stop using libtool-bin
Date 2024-05-22 08:20 +0200
Message-ID <IGNuq-f0HM-3@gated-at.bofh.it> (permalink)
References <IGylH-eRBY-7@gated-at.bofh.it> <IGylH-eRBY-7@gated-at.bofh.it> <IGDbH-eUt6-1@gated-at.bofh.it> <IGylH-eRBY-7@gated-at.bofh.it> <IGDbH-eUt6-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Thomas,

On Tue, May 21, 2024 at 03:06:00PM -0400, Thomas Dickey wrote:
> hmm - there are two sets of changes - I don't see a reason for the
> change to the curses function checks.

Thank you for reviewing my patch. The curses function check change does
have a reason. It can be solved differently in principle.

When I ran autoheader, config.hin would lack all the defines that should
have come from CF_CURSES_FUNCS while the relevant HAVE_* defines would
still show up in config.log after running configure and therefore the
resulting dlg_config.h would also lack them. That meant that dialog
would perceive a very dysfunctional curses and its shim would fail to
compile. Quite clearly, we shouldn't assume a crippled curses and
config.hin should contain the relevant templates. As it turns out,
autoheader interprets the m4 files and collects the AC_DEFINE and
AC_DEFINE_UNQUOTED invocations, well some of them actually. The
AC_CHECK_FUNCS would be collected whereas CF_CURSES_FUNCS not, even
though both seemed quite similar. The subtle difference is that
AC_CHECK_FUNCS uses AS_FOR (a loop that is evaluated using m4) whereas
CF_CURSES_FUNCS uses a shell for loop. Thus, autoheader would only see a
single, bogus AC_DEFINE_UNQUOTED for all of CF_CURSES_FUNCS and ignore
that. Avoiding this shell loop is key here and I went for manually
unrolling it, because AS_FOR didn't work out initially and unrolling
seemed workable to me. The crucial bit here is that you cannot use shell
for control flow here. If you prefer AS_FOR or some other working
mechanism, that's fine. Just do something about it to avoid dialog
failing to build when we remove libtool-bin from Debian.

Helmut

Back to linux.debian.bugs.dist | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

Bug#1071581: dialog: stop using libtool-bin Helmut Grohne <helmut@subdivi.de> - 2024-05-21 16:10 +0200
  Bug#1071581: dialog: stop using libtool-bin Thomas Dickey <dickey@his.com> - 2024-05-21 21:20 +0200
    Bug#1071581: dialog: stop using libtool-bin Helmut Grohne <helmut@subdivi.de> - 2024-05-22 08:20 +0200
      Bug#1071581: dialog: stop using libtool-bin Thomas Dickey <dickey@his.com> - 2024-05-22 10:10 +0200
  Bug#1071581: dialog: stop using libtool-bin Helmut Grohne <helmut@subdivi.de> - 2024-05-22 15:30 +0200
    Bug#1071581: dialog: stop using libtool-bin Thomas Dickey <dickey@his.com> - 2024-05-23 10:40 +0200

csiph-web