[PATCH] Update configure script hints wrt precious env vars.

[PATCH] Update configure script hints wrt precious env vars.

[Thien-Thi Nguyen]

Personally, i dislike environment variable dependence.
Maybe that's exposure to lexical binding poisoning my tastes...
In any case, (info "(autoconf) Defining Variables") has more info.

thi

_________________________________________________________________________
From 7a7b846cc2eb1f6aadfb06ccb445642f2bc2dfa7 Mon Sep 17 00:00:00 2001
From: Thien-Thi Nguyen <ttn@gnuvola.org>
Date: Sat, 9 Jan 2010 10:07:48 +0100
Subject: [PATCH] Update configure script hints wrt precious env vars.

* README: Don't suggest setting vars before running configure.
Instead, suggest specifying them on the configure command-line.

Signed-off-by: Thien-Thi Nguyen <ttn@gnuvola.org>
---
 README |   19 +++++++++++--------
 1 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/README b/README
index 5669ce8..e799495 100644
--- a/README
+++ b/README
@@ -50,16 +50,19 @@ names (where that is supported), makes it impossible to later move the
 built executables and libraries to an installation location other than
 the one that was specified at build time.
 
-Another possible approach is to set CPPFLAGS and LDFLAGS before
-running configure, so that they include -I options for all the
+Another possible approach is to set CPPFLAGS and LDFLAGS on the
+configure command-line, so that they include -I options for all the
 non-standard places where you have installed header files and -L
 options for all the non-standard places where you have installed
 libraries.  This will allow configure and make to find those headers
-and libraries during the build.  The locations found will not be
-hardcoded into the build executables and libraries, so with this
-approach you will probably also need to set LD_LIBRARY_PATH
-correspondingly, to allow Guile to find the necessary libraries again
-at runtime.
+and libraries during the build.  E.g.:
+
+  ../configure [...] CPPFLAGS='-I/my/include' LDFLAGS='-L/my/lib'
+
+The locations found will not be hardcoded into the build executables and
+libraries, so with this approach you will probably also need to set
+LD_LIBRARY_PATH correspondingly, to allow Guile to find the necessary
+libraries again at runtime.
 
 
 Required External Packages ================================================
@@ -94,7 +97,7 @@ Guile requires the following external packages:
     compile and link options for libgc.  If you don't have pkg-config
     installed, or you have a version of libgc that doesn't provide a
     .pc file, you can work around this by setting some environment
-    variables before running ./configure:
+    as part of the configure command-line:
 
     - PKG_CONFIG=true
 
-- 
1.6.3.2

Re: [PATCH] Update configure script hints wrt precious env vars.

[Andy Wingo]

On Sat 09 Jan 2010 14:49, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> Personally, i dislike environment variable dependence.
> Maybe that's exposure to lexical binding poisoning my tastes...
> In any case, (info "(autoconf) Defining Variables") has more info.

Ah, interesting info node.

>      .pc file, you can work around this by setting some environment
> -    variables before running ./configure:
> +    as part of the configure command-line:

Can you fix this one and resend?

Thanks :)

A
-- 
http://wingolog.org/

Re: [PATCH] Update configure script hints wrt precious env vars.

[Thien-Thi Nguyen]

() Andy Wingo <wingo@pobox.com>
() Sat, 09 Jan 2010 23:31:55 +0100

   >      .pc file, you can work around this by setting some environment
   > -    variables before running ./configure:
   > +    as part of the configure command-line:

   Can you fix this one and resend?

Yes, if you can tell me what's wrong w/ the above.

thi

Re: [PATCH] Update configure script hints wrt precious env vars.

[Andy Wingo]

On Sun 10 Jan 2010 18:37, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> () Andy Wingo <wingo@pobox.com>
> () Sat, 09 Jan 2010 23:31:55 +0100
>
>    >      .pc file, you can work around this by setting some environment
>    > -    variables before running ./configure:
>    > +    as part of the configure command-line:
>
>    Can you fix this one and resend?
>
> Yes, if you can tell me what's wrong w/ the above.

"setting some environment as part of ..."
         ^^^^^^^^^^^^^^^^

A
-- 
http://wingolog.org/

Re: [PATCH] Update configure script hints wrt precious env vars.

[Thien-Thi Nguyen]

From eb2713038b601abd59fdf377847510f27fd17e83 Mon Sep 17 00:00:00 2001
From: Thien-Thi Nguyen <ttn@gnuvola.org>
Date: Sat, 9 Jan 2010 10:07:48 +0100
Subject: [PATCH] Update configure script hints wrt precious env vars.

* README: Don't suggest setting vars before running configure.
Instead, suggest specifying them on the configure command-line.

Signed-off-by: Thien-Thi Nguyen <ttn@gnuvola.org>
---
BTW, apparently, this is the space for comments;
"git am" should ignore all this stuff.

thi

 README |   21 ++++++++++++---------
 1 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/README b/README
index 5669ce8..ea68081 100644
--- a/README
+++ b/README
@@ -50,16 +50,19 @@ names (where that is supported), makes it impossible to later move the
 built executables and libraries to an installation location other than
 the one that was specified at build time.
 
-Another possible approach is to set CPPFLAGS and LDFLAGS before
-running configure, so that they include -I options for all the
+Another possible approach is to set CPPFLAGS and LDFLAGS on the
+configure command-line, so that they include -I options for all the
 non-standard places where you have installed header files and -L
 options for all the non-standard places where you have installed
 libraries.  This will allow configure and make to find those headers
-and libraries during the build.  The locations found will not be
-hardcoded into the build executables and libraries, so with this
-approach you will probably also need to set LD_LIBRARY_PATH
-correspondingly, to allow Guile to find the necessary libraries again
-at runtime.
+and libraries during the build.  E.g.:
+
+  ../configure [...] CPPFLAGS='-I/my/include' LDFLAGS='-L/my/lib'
+
+The locations found will not be hardcoded into the build executables and
+libraries, so with this approach you will probably also need to set
+LD_LIBRARY_PATH correspondingly, to allow Guile to find the necessary
+libraries again at runtime.
 
 
 Required External Packages ================================================
@@ -93,8 +96,8 @@ Guile requires the following external packages:
     Guile's ./configure script uses pkg-config to discover the correct
     compile and link options for libgc.  If you don't have pkg-config
     installed, or you have a version of libgc that doesn't provide a
-    .pc file, you can work around this by setting some environment
-    variables before running ./configure:
+    .pc file, you can work around this by setting some variables as
+    part of the configure command-line:
 
     - PKG_CONFIG=true
 
-- 
1.6.3.2

Re: [PATCH] Update configure script hints wrt precious env vars.

[Andy Wingo]

On Wed 13 Jan 2010 09:19, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> From eb2713038b601abd59fdf377847510f27fd17e83 Mon Sep 17 00:00:00 2001
> From: Thien-Thi Nguyen <ttn@gnuvola.org>
> Date: Sat, 9 Jan 2010 10:07:48 +0100
> Subject: [PATCH] Update configure script hints wrt precious env vars.
>
> * README: Don't suggest setting vars before running configure.
> Instead, suggest specifying them on the configure command-line.
>
> Signed-off-by: Thien-Thi Nguyen <ttn@gnuvola.org>
> ---
> BTW, apparently, this is the space for comments;
> "git am" should ignore all this stuff.
>
> thi

Applied, and the other one. Also I figured out how to use
shell-command-on-region, M-|, so I don't care any more about whether
there's comments before or after, though Ludovic is probably right about
where they should go. So submit as you like.

Andy
-- 
http://wingolog.org/

Re: [PATCH] Update configure script hints wrt precious env vars.

[Thien-Thi Nguyen]

() Andy Wingo <wingo@pobox.com>
() Wed, 13 Jan 2010 20:52:08 +0100

   I figured out how to use shell-command-on-region, M-|, so I
   don't care any more about whether there's comments before or
   after, though Ludovic is probably right about where they should
   go. So submit as you like.

See also `gnus-summary-pipe-message'.

thi

Re: [PATCH] Update configure script hints wrt precious env vars.

[Andy Wingo]

On Wed 13 Jan 2010 21:35, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> () Andy Wingo <wingo@pobox.com>
> () Wed, 13 Jan 2010 20:52:08 +0100
>
>    I figured out how to use shell-command-on-region, M-|, so I
>    don't care any more about whether there's comments before or
>    after, though Ludovic is probably right about where they should
>    go. So submit as you like.
>
> See also `gnus-summary-pipe-message'.

Yes I tried that one, but it did the wrong thing. A shame, as it's less
typing.

A
-- 
http://wingolog.org/

Re: [PATCH] Update configure script hints wrt precious env vars.

[Thien-Thi Nguyen]

() Andy Wingo <wingo@pobox.com>
() Wed, 13 Jan 2010 22:06:23 +0100

   Yes I tried that one, but it did the wrong thing.

How about `gnus-summary-pipe-output'?

thi

Re: [PATCH] Update configure script hints wrt precious env vars.

[Andy Wingo]

On Thu 14 Jan 2010 12:04, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> () Andy Wingo <wingo@pobox.com>
> () Wed, 13 Jan 2010 22:06:23 +0100
>
>    Yes I tried that one, but it did the wrong thing.
>
> How about `gnus-summary-pipe-output'?

Hm, will try one of those. Thanks for the suggestions :)

Andy
-- 
http://wingolog.org/