diff mbox

configure.ac: align libs configure output

Message ID 1417523453-18059-1-git-send-email-maxim.uvarov@linaro.org
State New
Headers show

Commit Message

Maxim Uvarov Dec. 2, 2014, 12:30 p.m. UTC
if $AM_LDFLAGS is empty then libs output shifted with white-spaces
according to other prints.

Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
---
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Maxim Uvarov Dec. 9, 2014, 12:16 p.m. UTC | #1
Anders, please review.

Maxim.

On 12/02/2014 03:30 PM, Maxim Uvarov wrote:
> if $AM_LDFLAGS is empty then libs output shifted with white-spaces
> according to other prints.
>
> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
> ---
>   configure.ac | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index 5110cd9..91ae817 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -209,7 +209,7 @@ AC_SEARCH_LIBS([timer_create],[rt posix4])
>   ##########################################################################
>   # distribute the changed variables among the Makefiles
>   
> -AM_LDFLAGS="$AM_LDFLAGS -pthread -lrt"
> +AM_LDFLAGS="-pthread -lrt $AM_LDFLAGS"
>   
>   AC_SUBST([LIBS])
>   AC_SUBST([AM_CFLAGS])
Anders Roxell Dec. 9, 2014, 1:12 p.m. UTC | #2
On 2014-12-02 15:30, Maxim Uvarov wrote:
> if $AM_LDFLAGS is empty then libs output shifted with white-spaces
> according to other prints.
> 
> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

Reviewed-by: Anders Roxell <anders.roxell@linaro.org>

> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 5110cd9..91ae817 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -209,7 +209,7 @@ AC_SEARCH_LIBS([timer_create],[rt posix4])
>  ##########################################################################
>  # distribute the changed variables among the Makefiles
>  
> -AM_LDFLAGS="$AM_LDFLAGS -pthread -lrt"
> +AM_LDFLAGS="-pthread -lrt $AM_LDFLAGS"
>  
>  AC_SUBST([LIBS])
>  AC_SUBST([AM_CFLAGS])
> -- 
> 1.8.5.1.163.gd7aced9
> 
> 
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/lng-odp
Anders Roxell Dec. 9, 2014, 1:13 p.m. UTC | #3
On 2014-12-02 15:30, Maxim Uvarov wrote:
> if $AM_LDFLAGS is empty then libs output shifted with white-spaces
> according to other prints.
> 
> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 5110cd9..91ae817 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -209,7 +209,7 @@ AC_SEARCH_LIBS([timer_create],[rt posix4])
>  ##########################################################################
>  # distribute the changed variables among the Makefiles
>  
> -AM_LDFLAGS="$AM_LDFLAGS -pthread -lrt"
> +AM_LDFLAGS="-pthread -lrt $AM_LDFLAGS"
>  
>  AC_SUBST([LIBS])
>  AC_SUBST([AM_CFLAGS])
> -- 
> 1.8.5.1.163.gd7aced9
> 
> 
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/lng-odp

we need another patch to clean up am_cflags as well if you don't do that
in the same patch.


Cheers,
Anders
diff mbox

Patch

diff --git a/configure.ac b/configure.ac
index 5110cd9..91ae817 100644
--- a/configure.ac
+++ b/configure.ac
@@ -209,7 +209,7 @@  AC_SEARCH_LIBS([timer_create],[rt posix4])
 ##########################################################################
 # distribute the changed variables among the Makefiles
 
-AM_LDFLAGS="$AM_LDFLAGS -pthread -lrt"
+AM_LDFLAGS="-pthread -lrt $AM_LDFLAGS"
 
 AC_SUBST([LIBS])
 AC_SUBST([AM_CFLAGS])