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

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

 



Am 2013-08-19 22:08, schrieb Allen Martin:
> On Thu, Aug 01, 2013 at 01:17:54PM -0700, Stephen Warren wrote:
>> On 07/23/2013 09:49 AM, Thierry Reding wrote:
>> > On Sun, Jul 21, 2013 at 11:25:28PM +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> --- src/Makefile.am
>> >> | 2 ++ 1 file changed, 2 insertions(+)
>> >>
>> >> 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 + AM_CFLAGS = -Wall -std=c99
>> >>
>> >> bin_PROGRAMS = cbootimage bct_dump
>> >
>> > Hi Stefan,
>> >
>> > Thanks for the patch. Generally this looks good to me, though
>> > automake now complains:
>> >
>> > src/Makefile.am:36: compiling `t114/parse_t114.c' in subdir
>> > requires `AM_PROG_CC_C_O' in `configure.ac'
>> >
>> > Which can quite easily be solved by adding that line in
>> > configure.ac. Immediately following the AC_PROG_CC looks like a
>> > good place for it.
>>
>> Stefan, are you planning on revising the patch based on Thierry's
>> suggestion?
> 
> Hi, what's the status of this patch?  Is there a version 2 to review?
Funny, just sent the updated patch seconds ago! Altough, my mailer says 
I lost the race by 2 minutes :-(

--
Stefan

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




[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