Re: [PATCH v2 tegra-cbootimage] Add automake option subdir-objects to prevent warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 19, 2013 at 10:10:20PM +0200, Stefan Agner wrote:
> The SoC specific code is in subdirectories which leads to warnings in
> newer
> automake versions. The option subdir-objects prevents those warnings.
> 
> Signed-off-by: Stefan Agner <stefan@xxxxxxxx>
> ---
> 
> Sorry, was in vacation. I could not reproduce the message Thierry sees
> on
> my automake 1.14 installation, however, the option make sense as far as
> I 
> understand its meaning.

Yes, automake 1.14 doesn't warn about it, presumably because it makes
sure to select AM_PROG_CC_C_O if subdir-objects is enabled. I can
reproduce the warning by downgrading to 1.13.4, though.

>  configure.ac    | 1 +
>  src/Makefile.am | 2 ++
>  2 files changed, 3 insertions(+)
> 
> diff --git a/configure.ac b/configure.ac
> index ccd0a55..9b83a21 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -10,6 +10,7 @@ AC_CONFIG_HEADERS([config.h])
>  # Checks for programs.
>  AC_PROG_CXX
>  AC_PROG_CC
> +AM_PROG_CC_C_O
>  PKG_PROG_PKG_CONFIG
>  
>  # Checks for libraries.
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 6fcc0d8..6fecfc7 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -1,3 +1,5 @@
> +AUTOMAKE_OPTIONS = subdir-objects
> +

I don't remember if I mentioned it before, but I've seen a lot of
projects that keep these options in the configure.ac file (in the
AM_INIT_AUTOMAKE command). The GNU documentation isn't specific
about which variant is preferred, so either way:

Reviewed-by: Thierry Reding <thierry.reding@xxxxxxxxx>

Attachment: pgpG9op9kA0Lb.pgp
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux