[systemd-devel] [PATCH 1/2] autogen: add shortcut for building with clang

[Thomas H.P. Andersen]

On Sat, Dec 28, 2013 at 1:13 PM, Lennart Poettering
<lennart at poettering.net> wrote:
> On Fri, 27.12.13 22:25, Thomas H.P. Andersen (phomes at gmail.com) wrote:
>
>> From: Thomas Hindoe Paaboel Andersen <phomes at gmail.com>
>>
>> For a while I have been cleaning up warnings when building with clang.
>> There are currently only two sources of warnings left: Wcast-align and Wgnu.
>> I am not convinced that fixing up those up is feasible so I run with them
>> disabled to spot regressions. E.g. clang is a bit more strict wrt to unused
>> variables with the cleanup attribute and I have fixed a number of those since.
>>
>> Like the other options in autogen.sh I have a shortcut for clang as well. I use
>> 'l' for llvm.
>> ---
>>  autogen.sh | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/autogen.sh b/autogen.sh
>> index e6289b3..91df10a 100755
>> --- a/autogen.sh
>> +++ b/autogen.sh
>> @@ -62,6 +62,9 @@ elif [ "x$1" = "xg" ]; then
>>  elif [ "x$1" = "xa" ]; then
>>          ./configure CFLAGS='-g -O0 -Wsuggest-attribute=pure -Wsuggest-attribute=const -ftrapv' --enable-kdbus $args
>>          make clean
>> +elif [ "x$1" = "xl" ]; then
>> +        ./configure CC=clang CFLAGS='-g -O0 -ftrapv -Wno-cast-align -Wno-gnu' --enable-kdbus $args
>> +        make clean
>>  else
>>          echo
>>          echo "----------------------------------------------------------------"
>
> Looks good! Please commit!

Pushed. Thanks.