* "guile-config link" reports LDFLAGS it has been compiled with
@ 2007-12-16 14:57 Marijn Schouten (hkBst)
2008-02-16 23:01 ` Ludovic Courtès
0 siblings, 1 reply; 7+ messages in thread
From: Marijn Schouten (hkBst) @ 2007-12-16 14:57 UTC (permalink / raw)
To: guile-devel
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi list,
"guile-config link" reports LDFLAGS it has been compiled with. Usage says:
guile-config link - print libraries to link with
so one of these is inaccurate. To resolve this issue in favor of the description:
- --- guile-1.8.3.orig/configure.in
+++ guile-1.8.3/configure.in
@@ -1333,7 +1333,7 @@ AC_SUBST(LIBGUILE_SRFI_SRFI_60_INTERFACE
#######################################################################
dnl Tell guile-config what flags guile users should compile and link with.
- -GUILE_LIBS="$LDFLAGS $LIBS"
+GUILE_LIBS="$LIBS"
GUILE_CFLAGS="$PTHREAD_CFLAGS"
AC_SUBST(GUILE_LIBS)
AC_SUBST(GUILE_CFLAGS)
as reported in http://bugs.gentoo.org/show_bug.cgi?id=202205
thank you,
Marijn
- --
Marijn Schouten (hkBst), Gentoo Lisp project, Gentoo ML
<http://www.gentoo.org/proj/en/lisp/>, #gentoo-{lisp,ml} on FreeNode
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.7 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFHZTzqp/VmCx0OL2wRAqnQAKCBwiDJsJqXbMooyy4XJw7Mh+ggRACeNFAM
vdvgjK+1OM8aQTdJNczmvsQ=
=c1ip
-----END PGP SIGNATURE-----
_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: "guile-config link" reports LDFLAGS it has been compiled with
2007-12-16 14:57 "guile-config link" reports LDFLAGS it has been compiled with Marijn Schouten (hkBst)
@ 2008-02-16 23:01 ` Ludovic Courtès
2008-02-16 23:21 ` Ludovic Courtès
0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2008-02-16 23:01 UTC (permalink / raw)
To: guile-devel
Hi Marijn,
Sorry for the later reply.
"Marijn Schouten (hkBst)" <hkBst@gentoo.org> writes:
> "guile-config link" reports LDFLAGS it has been compiled with. Usage says:
>
> guile-config link - print libraries to link with
>
> so one of these is inaccurate. To resolve this issue in favor of the description:
>
> --- guile-1.8.3.orig/configure.in
> +++ guile-1.8.3/configure.in
> @@ -1333,7 +1333,7 @@ AC_SUBST(LIBGUILE_SRFI_SRFI_60_INTERFACE
> #######################################################################
>
> dnl Tell guile-config what flags guile users should compile and link with.
> -GUILE_LIBS="$LDFLAGS $LIBS"
> +GUILE_LIBS="$LIBS"
> GUILE_CFLAGS="$PTHREAD_CFLAGS"
> AC_SUBST(GUILE_LIBS)
> AC_SUBST(GUILE_CFLAGS)
>
> as reported in http://bugs.gentoo.org/show_bug.cgi?id=202205
Indeed, you're right.
Conversely, I think `GUILE_CFLAGS' must include `$CPPFLAGS' since this
may include `-I' flags that are required (e.g., `-I/path/to/gmp').
Thanks,
Ludovic.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: "guile-config link" reports LDFLAGS it has been compiled with
2008-02-16 23:01 ` Ludovic Courtès
@ 2008-02-16 23:21 ` Ludovic Courtès
2008-02-18 14:19 ` Greg Troxel
0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2008-02-16 23:21 UTC (permalink / raw)
To: guile-devel
ludo@gnu.org (Ludovic Courtès) writes:
> "Marijn Schouten (hkBst)" <hkBst@gentoo.org> writes:
>> dnl Tell guile-config what flags guile users should compile and link with.
>> -GUILE_LIBS="$LDFLAGS $LIBS"
>> +GUILE_LIBS="$LIBS"
>> GUILE_CFLAGS="$PTHREAD_CFLAGS"
>> AC_SUBST(GUILE_LIBS)
>> AC_SUBST(GUILE_CFLAGS)
> Indeed, you're right.
>
> Conversely, I think `GUILE_CFLAGS' must include `$CPPFLAGS' since this
> may include `-I' flags that are required (e.g., `-I/path/to/gmp').
Wait, no: the argument for `$CPPFLAGS' also applies to `$LDFLAGS'. That
is, if `-L' flags are passed through `$LDFLAGS' to `configure', like
`-L/path/to/gmp', then they must be reported by "guile-config link".
Admittedly, it has the undesirable side-effect of reporting completely
unrelated link flags as well. Unfortunately, I'm not sure we could
reliably filter `$LDFLAGS'.
Thanks,
Ludovic.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: "guile-config link" reports LDFLAGS it has been compiled with
2008-02-16 23:21 ` Ludovic Courtès
@ 2008-02-18 14:19 ` Greg Troxel
2008-02-18 14:59 ` Ludovic Courtès
0 siblings, 1 reply; 7+ messages in thread
From: Greg Troxel @ 2008-02-18 14:19 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: guile-devel
Wait, no: the argument for `$CPPFLAGS' also applies to `$LDFLAGS'. That
is, if `-L' flags are passed through `$LDFLAGS' to `configure', like
`-L/path/to/gmp', then they must be reported by "guile-config link".
Admittedly, it has the undesirable side-effect of reporting completely
unrelated link flags as well. Unfortunately, I'm not sure we could
reliably filter `$LDFLAGS'.
And, this really should use -Wl,-R as well on platforms that use the -R
convention.
Strictly speaking, I think we need both LDFLAGS and -L$(libdir); it may
be that guile was built with prereqs from one place and installed in
another. But it may always put it's own libdir in LDFLAGS.
It would be nice to see a pkg-config file for guile; I kind of remember
this floating around on the list but 1.8.3 doesn't seem to install it.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: "guile-config link" reports LDFLAGS it has been compiled with
2008-02-18 14:19 ` Greg Troxel
@ 2008-02-18 14:59 ` Ludovic Courtès
0 siblings, 0 replies; 7+ messages in thread
From: Ludovic Courtès @ 2008-02-18 14:59 UTC (permalink / raw)
To: guile-devel
Hi,
Greg Troxel <gdt@ir.bbn.com> writes:
> And, this really should use -Wl,-R as well on platforms that use the -R
> convention.
>
> Strictly speaking, I think we need both LDFLAGS and -L$(libdir); it may
> be that guile was built with prereqs from one place and installed in
> another. But it may always put it's own libdir in LDFLAGS.
Right.
> It would be nice to see a pkg-config file for guile; I kind of remember
> this floating around on the list but 1.8.3 doesn't seem to install it.
Yes, there are several (!) patches for this, so that should go in 1.8.5.
Thanks,
Ludovic.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: "guile-config link" reports LDFLAGS it has been compiled with
@ 2008-02-18 16:45 dsmich
2008-02-18 16:53 ` Ludovic Courtès
0 siblings, 1 reply; 7+ messages in thread
From: dsmich @ 2008-02-18 16:45 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: guile-devel
---- "Ludovic Courtès" <ludo@gnu.org> wrote:
> Greg Troxel <gdt@ir.bbn.com> writes:
> > It would be nice to see a pkg-config file for guile; I kind of remember
> > this floating around on the list but 1.8.3 doesn't seem to install it.
>
> Yes, there are several (!) patches for this, so that should go in 1.8.5.
Why wait for 1.8.5? Why not put this in 1.8.4?
-Dale
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: "guile-config link" reports LDFLAGS it has been compiled with
2008-02-18 16:45 dsmich
@ 2008-02-18 16:53 ` Ludovic Courtès
0 siblings, 0 replies; 7+ messages in thread
From: Ludovic Courtès @ 2008-02-18 16:53 UTC (permalink / raw)
To: guile-devel
Hi,
<dsmich@adelphia.net> writes:
> Why wait for 1.8.5? Why not put this in 1.8.4?
Because 1.8.4 should already be available, if only the GNU sysadmins
were a bit more responsive. :-)
Cheers,
Ludovic.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2008-02-18 16:53 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-12-16 14:57 "guile-config link" reports LDFLAGS it has been compiled with Marijn Schouten (hkBst)
2008-02-16 23:01 ` Ludovic Courtès
2008-02-16 23:21 ` Ludovic Courtès
2008-02-18 14:19 ` Greg Troxel
2008-02-18 14:59 ` Ludovic Courtès
-- strict thread matches above, loose matches on Subject: below --
2008-02-18 16:45 dsmich
2008-02-18 16:53 ` Ludovic Courtès
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).