# configure clobbers LDFLAGS

3 messages from 2009-01-05 to 2009-01-06. Participants: Paul Jarc, Jeff King, Junio C Hamano.
Thread: https://gitlist.dev/t/16990

## Paul Jarc, 2009-01-05 02:27

Subject: configure clobbers LDFLAGS
Message-ID: <m3tz8e32ru.fsf@multivac.cwru.edu>
URL: https://gitlist.dev/e/m3tz8e32ru.fsf%40multivac.cwru.edu

```
(I'm not subscribed; Mail-Followup-To set.)

In a couple of tests, configure clobbers the LDFLAGS value set by the
caller.  This patch fixes it.


paul


diff --git a/configure.ac b/configure.ac
index 8821b50..0a5fc8c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -127,7 +127,7 @@ else
       SAVE_LDFLAGS="${LDFLAGS}"
       LDFLAGS="${SAVE_LDFLAGS} -Wl,-rpath,/"
       AC_LINK_IFELSE(AC_LANG_PROGRAM([], []), [ld_wl_rpath=yes], [ld_wl_rpath=no])
-      LDFLAGS="${SAVE_LD_FLAGS}"
+      LDFLAGS="${SAVE_LDFLAGS}"
    ])
    if test "$ld_wl_rpath" = "yes"; then
       AC_SUBST(CC_LD_DYNPATH, [-Wl,-rpath,])
@@ -136,7 +136,7 @@ else
          SAVE_LDFLAGS="${LDFLAGS}"
          LDFLAGS="${SAVE_LDFLAGS} -rpath /"
          AC_LINK_IFELSE(AC_LANG_PROGRAM([], []), [ld_rpath=yes], [ld_rpath=no])
-         LDFLAGS="${SAVE_LD_FLAGS}"
+         LDFLAGS="${SAVE_LDFLAGS}"
       ])
       if test "$ld_rpath" = "yes"; then
          AC_SUBST(CC_LD_DYNPATH, [-rpath])

```

## Jeff King, 2009-01-05 06:00

Subject: Re: configure clobbers LDFLAGS
Message-ID: <20090105060058.GB13189@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20090105060058.GB13189%40coredump.intra.peff.net
In-Reply-To: <m3tz8e32ru.fsf@multivac.cwru.edu>

```
On Sun, Jan 04, 2009 at 09:27:41PM -0500, Paul Jarc wrote:

> In a couple of tests, configure clobbers the LDFLAGS value set by the
> caller.  This patch fixes it.

Thanks for the patch. It looks obviously correct to me, and the problem
was just some typos introduced by 798a9450.

Junio, I think this should go onto maint.

> (I'm not subscribed; Mail-Followup-To set.)

The usual behavior on this list is to reply to the sender, cc'ing the
list and any previous to or cc's. So MFT is unnecessary and generally
frowned upon here.

Also, if you are planning on sending more patches, the usual practice is
for them to be inline rather than attached, with the commit message as
the body and any cover letter (like MFT discussion) after the
triple-dash; git format-patch will generate this format for you.  More
information is in Documentation/SubmittingPatches.

-Peff

```

## Junio C Hamano, 2009-01-06 08:19

Subject: Re: configure clobbers LDFLAGS
Message-ID: <7v1vvgkfst.fsf@gitster.siamese.dyndns.org>
URL: https://gitlist.dev/e/7v1vvgkfst.fsf%40gitster.siamese.dyndns.org
In-Reply-To: <20090105060058.GB13189@coredump.intra.peff.net>

```
Jeff King <peff@peff.net> writes:

> On Sun, Jan 04, 2009 at 09:27:41PM -0500, Paul Jarc wrote:
>
>> In a couple of tests, configure clobbers the LDFLAGS value set by the
>> caller.  This patch fixes it.
>
> Thanks for the patch. It looks obviously correct to me, and the problem
> was just some typos introduced by 798a9450.
>
> Junio, I think this should go onto maint.

Thanks; will queue.

```