diff mbox

[cbootimage,1/2] autogen.sh: allow spaced in configure arguments

Message ID 1377284162-7700-1-git-send-email-swarren@wwwdotorg.org
State Accepted, archived
Headers show

Commit Message

Stephen Warren Aug. 23, 2013, 6:56 p.m. UTC 
From: Stephen Warren <swarren@nvidia.com>

Using "$@" (rather than #@) quotes any arguments if required. This is
useful if you want to run ./autogen.sh CFLAGS="-ggdb -O0".

Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
 autogen.sh | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Thierry Reding Aug. 26, 2013, 7:52 a.m. UTC | #1 
On Fri, Aug 23, 2013 at 12:56:01PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>

s/spaced/spaces/ in the patch subject?

Otherwise:

Reviewed-by: Thierry Reding <treding@nvidia.com>
Stephen Warren Aug. 26, 2013, 8:20 p.m. UTC | #2 
On 08/26/2013 01:52 AM, Thierry Reding wrote:
> On Fri, Aug 23, 2013 at 12:56:01PM -0600, Stephen Warren wrote:
>> From: Stephen Warren <swarren@nvidia.com>
> 
> s/spaced/spaces/ in the patch subject?

Sigh. Guess I need to pay more attention too:-)

> Otherwise:
> 
> Reviewed-by: Thierry Reding <treding@nvidia.com>

Thanks. I've pushed this one patch out with the subject fixed.

--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/autogen.sh b/autogen.sh
index e925023..530783b 100755
--- a/autogen.sh
+++ b/autogen.sh
@@ -1,5 +1,4 @@ 
 #!/bin/sh -e
 
 autoreconf --install --symlink
-./configure $@
-
+./configure "$@"