Re: bdw-gc dependency

--0000000000005ec38805f3e7d06e
Content-Type: text/plain; charset="UTF-8"

On Wed, 1 Feb 2023 at 14:24, Bruno Haible <[email protected]> wrote:

>
> There are three problems here:
>
> 1) Since I don't want to install pkg-config on my build machines, I opt
> for the environment variables approach. I have already set BDW_GC_CFLAGS
> and BDW_GC_LIBS for the sake of GNU guile and GNU poke. Now GNU a2ps
> comes along and wants me to set *other* environment variables for the
> *same* purpose.
>
> A code search on codesearch.debian.net shows that
>   - 2 GNU packages (guile and poke) use
>       PKG_CHECK_MODULES([BDW_GC], ...
>       or PKG_CHECK_MODULES(BDW_GC, ...
>   - 3 non-GNU packages (mmv, ocp, libsidplayfp) use
>       PKG_CHECK_MODULES([LIBGC], ...
>       or PKG_CHECK_MODULES(LIBGC, ...
>
> Please join the camp of guile and poke, and thus avoid unnecessary work
> for the one who installs the package!
>

Done. As it happens, I'm also upstream for mmv, so I've made the change to
its sources too!

(It was a fairly narrow thing in terms of historical precedence, though:
GNU Zile used LIBGC from 2011 (until its recent rewrite in Vala), and was
just beaten by Guile, which was used BDW_GC since 2010!)

3) The pkg.m4 that you used for creating the tarball mishandles empty
> environment variables. When I set
>   export LIBGC_CFLAGS=
>   export LIBGC_LIBS="-lgc"
> the configure test bails out and aborts. Whereas when I set
>   export LIBGC_CFLAGS="-Wall"
>   export LIBGC_LIBS="-lgc"
> the configure test succeeds and I can build the package.
>
> Suggested fix: Pull the pkg.m4 from GNU poke 3.0 (which you find on
> https://ftp.gnu.org/gnu/poke/). AFAIK version 0.29.2 is the newest one.
>

Oops, I had an old version installed in my user's home directory. I've
cleaned it out, and the next release should use my system-installed 0.29.2.

-- 
https://rrt.sc3d.org

--0000000000005ec38805f3e7d06e
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail=
_attr">On Wed, 1 Feb 2023 at 14:24, Bruno Haible &lt;<a href=3D"mailto:brun=
[email protected]">[email protected]</a>&gt; wrote:<br></div><blockquote class=3D"g=
mail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204=
,204,204);padding-left:1ex">
<br>
There are three problems here:<br>
<br>
1) Since I don&#39;t want to install pkg-config on my build machines, I opt=
<br>
for the environment variables approach. I have already set BDW_GC_CFLAGS<br=
>
and BDW_GC_LIBS for the sake of GNU guile and GNU poke. Now GNU a2ps<br>
comes along and wants me to set *other* environment variables for the<br>
*same* purpose.<br>
<br>
A code search on <a href=3D"http://codesearch.debian.net" rel=3D"noreferrer=
" target=3D"_blank">codesearch.debian.net</a> shows that<br>
=C2=A0 - 2 GNU packages (guile and poke) use<br>
=C2=A0 =C2=A0 =C2=A0 PKG_CHECK_MODULES([BDW_GC], ...<br>
=C2=A0 =C2=A0 =C2=A0 or PKG_CHECK_MODULES(BDW_GC, ...<br>
=C2=A0 - 3 non-GNU packages (mmv, ocp, libsidplayfp) use<br>
=C2=A0 =C2=A0 =C2=A0 PKG_CHECK_MODULES([LIBGC], ...<br>
=C2=A0 =C2=A0 =C2=A0 or PKG_CHECK_MODULES(LIBGC, ...<br>
<br>
Please join the camp of guile and poke, and thus avoid unnecessary work<br>
for the one who installs the package!<br></blockquote><div><br></div><div><=
div style=3D"font-family:arial,helvetica,sans-serif;font-size:small" class=
=3D"gmail_default">Done. As it happens, I&#39;m also upstream for mmv, so I=
&#39;ve made the change to its sources too!</div><div style=3D"font-family:=
arial,helvetica,sans-serif;font-size:small" class=3D"gmail_default"><br></d=
iv><div style=3D"font-family:arial,helvetica,sans-serif;font-size:small" cl=
ass=3D"gmail_default">(It was a fairly narrow thing in terms of historical =
precedence, though: GNU Zile used LIBGC from 2011 (until its recent rewrite=
 in Vala), and was just beaten by Guile, which was used BDW_GC since 2010!)=
<br></div><br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0=
px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
3) The pkg.m4 that you used for creating the tarball mishandles empty<br>
environment variables. When I set<br>
=C2=A0 export LIBGC_CFLAGS=3D<br>
=C2=A0 export LIBGC_LIBS=3D&quot;-lgc&quot;<br>
the configure test bails out and aborts. Whereas when I set<br>
=C2=A0 export LIBGC_CFLAGS=3D&quot;-Wall&quot;<br>
=C2=A0 export LIBGC_LIBS=3D&quot;-lgc&quot;<br>
the configure test succeeds and I can build the package.<br>
<br>
Suggested fix: Pull the pkg.m4 from GNU poke 3.0 (which you find on<br>
<a href=3D"https://ftp.gnu.org/gnu/poke/" rel=3D"noreferrer" target=3D"_bla=
nk">https://ftp.gnu.org/gnu/poke/</a>). AFAIK version 0.29.2 is the newest =
one.<br>
</blockquote></div><div><br></div><div style=3D"font-family:arial,helvetica=
,sans-serif;font-size:small" class=3D"gmail_default">Oops, I had an old ver=
sion installed in my user&#39;s home directory. I&#39;ve cleaned it out, an=
d the next release should use my system-installed 0.29.2.</div><br>-- <br><=
div dir=3D"ltr" class=3D"gmail_signature"><div dir=3D"ltr"><div><div dir=3D=
"ltr"><a href=3D"https://rrt.sc3d.org" target=3D"_blank">https://rrt.sc3d.o=
rg</a></div></div></div></div></div>

--0000000000005ec38805f3e7d06e--