Re: [patch 15/15] coding style fixes

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

 



On Tuesday 24 July 2007 02:25:32 Robert Schwebel wrote:
> The coding style was really messed up all over the tree. This patch
> collects all changes to refactor Makefiles in a sane way.

Took a little munging to get into svn, and had to make one more space to tabs 
conversion... Thanks for all the cleanup - very much appreciated.

>
> Signed-off-by: Robert Schwebel <r.schwebel@xxxxxxxxxxxxxx>

Acked-by: Darren Hart <dvhltc@xxxxxxxxxx>

>
> ---
>  GNUmakefile.am                           |    8 ++++----
>  config/GNUmakefile.am                    |    2 +-
>  func/GNUmakefile.am                      |   27
> +++++++++++++-------------- func/async_handler/GNUmakefile.am        |   24
> +++++++++++------------- func/gtod_latency/GNUmakefile.am         |   10
> +++-------
>  func/gtod_latency/GNUmakefile.am.rej     |   21 ---------------------
>  func/matrix_mult/GNUmakefile.am          |    6 +++---
>  func/measurement/GNUmakefile.am          |    3 +--
>  func/periodic_cpu_load/GNUmakefile.am    |    6 +++---
>  func/pi-tests/GNUmakefile.am             |   14 +++++++-------
>  func/prio-preempt/GNUmakefile.am         |   13 +++++++------
>  func/prio-wake/GNUmakefile.am            |   13 ++++++-------
>  func/pthread_kill_latency/GNUmakefile.am |    8 ++++----
>  func/sched_jitter/GNUmakefile.am         |   11 +++++------
>  func/sched_latency/GNUmakefile.am        |   15 +++++++--------
>  perf/GNUmakefile.am                      |    1 +
>  stress/GNUmakefile.am                    |    5 ++---
>  17 files changed, 78 insertions(+), 109 deletions(-)
>
> Index: func/gtod_latency/GNUmakefile.am
> ===================================================================
> --- func/gtod_latency/GNUmakefile.am.orig
> +++ func/gtod_latency/GNUmakefile.am
> @@ -12,9 +12,9 @@ gtod_infinite_CFLAGS = \
>  	-O2
>
>  LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +         $(top_builddir)/include/libjvmsim.a \
> +         $(top_builddir)/include/librt.a \
> +         $(top_builddir)/include/libstats.a

Not sure if something got lost in translation... what did you change here?  I 
think you intended to make them tabs?  will merge in as tabs.

>
>  LDFLAGS = \
>  	-L$(DESTDIR)/usr/lib/nptl \
> @@ -23,10 +23,6 @@ LDFLAGS = \
>  gtod_latency_CFLAGS = \
>  	-O2
>
> -#rdtsc_latency_LDADD = \
> -#	-L$(DESTDIR)/usr/lib/nptl \
> -#	-lpthread -lrt -lm
> -
>  MAINTAINERCLEANFILES = \
>  	GNUmakefile.in
>
> Index: func/matrix_mult/GNUmakefile.am
> ===================================================================
> --- func/matrix_mult/GNUmakefile.am.orig
> +++ func/matrix_mult/GNUmakefile.am
> @@ -17,9 +17,9 @@ matrix_mult_CFLAGS = \
>          -O2
>
>  LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  matrix_mult_LDFLAGS = \
>          -lpthread -lrt -lm
> Index: func/measurement/GNUmakefile.am
> ===================================================================
> --- func/measurement/GNUmakefile.am.orig
> +++ func/measurement/GNUmakefile.am
> @@ -1,7 +1,6 @@
> -
>  bin_PROGRAMS = \
>  	preempt_timing \
> -	rdtsc-latency
> +	rdtsc-latency
>
>  AM_CPPFLAGS = \
>  	-I$(top_srcdir)/include \
> Index: func/periodic_cpu_load/GNUmakefile.am
> ===================================================================
> --- func/periodic_cpu_load/GNUmakefile.am.orig
> +++ func/periodic_cpu_load/GNUmakefile.am
> @@ -17,9 +17,9 @@ AM_LDFLAGS = \
>          -lpthread -lrt -lm
>
>  periodic_cpu_load_LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  periodic_cpu_load_single_LDADD = \
>  	$(top_builddir)/include/libjvmsim.a \
> Index: func/pi-tests/GNUmakefile.am
> ===================================================================
> --- func/pi-tests/GNUmakefile.am.orig
> +++ func/pi-tests/GNUmakefile.am
> @@ -24,17 +24,17 @@ AM_CPPFLAGS = \
>
>  AM_CFLAGS = \
>  	-O2
> -LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
>
> +LDADD = \
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  AM_LDFLAGS = \
>  	-L$(DESTDIR)/usr/lib/nptl \
> -        -lpthread \
> -        -lrt \
> -        -lm
> +	-lpthread \
> +	-lrt \
> +	-lm
>
>  testpi_7_CPPFLAGS = \
>  	$(AM_CPPFLAGS) \
> Index: func/prio-preempt/GNUmakefile.am
> ===================================================================
> --- func/prio-preempt/GNUmakefile.am.orig
> +++ func/prio-preempt/GNUmakefile.am
> @@ -9,15 +9,16 @@ AM_CPPFLAGS = \
>
>  prio_preempt_CFLAGS = \
>  	-O2
> +
>  LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  prio_preempt_LDFLAGS = \
> -        -lpthread \
> -        -lrt \
> -        -lm
> +	-lpthread \
> +	-lrt \
> +	-lm
>
>  MAINTAINERCLEANFILES = \
>  	GNUmakefile.in
> Index: func/prio-wake/GNUmakefile.am
> ===================================================================
> --- func/prio-wake/GNUmakefile.am.orig
> +++ func/prio-wake/GNUmakefile.am
> @@ -8,18 +8,17 @@ AM_CPPFLAGS = \
>  	-I$(DESTDIR)/usr/include/nptl
>
>  LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  prio_wake_CFLAGS = \
>  	-O2
>
>  prio_wake_LDFLAGS = \
> -        -lpthread  \
> -        -lrt  \
> -        -lm
> -
> +	-lpthread  \
> +	-lrt  \
> +	-lm
>
>  MAINTAINERCLEANFILES = \
>  	GNUmakefile.in
> Index: func/pthread_kill_latency/GNUmakefile.am
> ===================================================================
> --- func/pthread_kill_latency/GNUmakefile.am.orig
> +++ func/pthread_kill_latency/GNUmakefile.am
> @@ -12,12 +12,12 @@ AM_CFLAGS = \
>  	-O2
>
>  LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  pthread_kill_latency_LDFLAGS = \
> -        -lpthread -lrt -lm
> +	-lpthread -lrt -lm
>
>  MAINTAINERCLEANFILES = \
>  	GNUmakefile.in
> Index: func/sched_jitter/GNUmakefile.am
> ===================================================================
> --- func/sched_jitter/GNUmakefile.am.orig
> +++ func/sched_jitter/GNUmakefile.am
> @@ -10,14 +10,13 @@ sched_jitter_CFLAGS = \
>  	-O2
>
>  LDADD = \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libstats.a
>
>  sched_jitter_LDFLAGS = \
> -        -L/usr/lib/nptl \
> -        -lpthread \
> -        -lrt \
> -        -lm
> -
> +	-L/usr/lib/nptl \
> +	-lpthread \
> +	-lrt \
> +	-lm
>
>  MAINTAINERCLEANFILES = \
>  	GNUmakefile.in
> Index: func/sched_latency/GNUmakefile.am
> ===================================================================
> --- func/sched_latency/GNUmakefile.am.orig
> +++ func/sched_latency/GNUmakefile.am
> @@ -11,16 +11,15 @@ sched_latency_CFLAGS = \
>  	-O2
>
>  LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  sched_latency_LDFLAGS = \
> -        -L/usr/lib/nptl \
> -        -lpthread \
> -        -lrt \
> -        -lm
> -
> +	-L/usr/lib/nptl \
> +	-lpthread \
> +	-lrt \
> +	-lm
>
>  MAINTAINERCLEANFILES = \
>  	GNUmakefile.in
> Index: stress/GNUmakefile.am
> ===================================================================
> --- stress/GNUmakefile.am.orig
> +++ stress/GNUmakefile.am
> @@ -1,7 +1,6 @@
>  SUBDIRS = \
> -        pi-tests
> +	pi-tests
>
>  MAINTAINERCLEANFILES = \
> -        GNUmakefile.in
> -
> +	GNUmakefile.in
>
> Index: func/async_handler/GNUmakefile.am
> ===================================================================
> --- func/async_handler/GNUmakefile.am.orig
> +++ func/async_handler/GNUmakefile.am
> @@ -1,29 +1,27 @@
>  if HAVE_PTHREAD_MUTEXATTR_SETPROTOCOL
>  bin_PROGRAMS = \
> -        async_handler \
> +	async_handler \
>  	async_handler_jk \
>  	async_handler_tsc
>  endif
>
> -
>  AM_CPPFLAGS = \
> -        -I$(top_srcdir)/include \
> -        -I$(top_builddir)/include \
> -        -I$(DESTDIR)/usr/include/nptl
> +	-I$(top_srcdir)/include \
> +	-I$(top_builddir)/include \
> +	-I$(DESTDIR)/usr/include/nptl
>
>  AM_CFLAGS = \
> -        -O2 \
> -        -D_GNU_SOURCE
> +	-O2 \
> +	-D_GNU_SOURCE
>
>  LDADD = \
> -         ${top_builddir}/include/libjvmsim.a \
> -         ${top_builddir}/include/librt.a \
> -         ${top_builddir}/include/libstats.a
> +	$(top_builddir)/include/libjvmsim.a \
> +	$(top_builddir)/include/librt.a \
> +	$(top_builddir)/include/libstats.a
>
>  AM_LDFLAGS = \
> -        -lpthread -lrt -lm
> +	-lpthread -lrt -lm
>
>  MAINTAINERCLEANFILES = \
> -        GNUmakefile.in
> -
> +	GNUmakefile.in
>
> Index: func/GNUmakefile.am
> ===================================================================
> --- func/GNUmakefile.am.orig
> +++ func/GNUmakefile.am
> @@ -1,19 +1,18 @@
> -SUBDIRS=\
> -	async_handler\
> -	gtod_latency\
> -	matrix_mult\
> -	measurement\
> -	periodic_cpu_load\
> -	pi-tests\
> -	prio-preempt\
> -	prio-wake\
> -	pthread_kill_latency\
> -	sched_football\
> -	sched_jitter\
> -	sched_latency\
> +SUBDIRS = \
> +	async_handler \
> +	gtod_latency \
> +	matrix_mult \
> +	measurement \
> +	periodic_cpu_load \
> +	pi-tests \
> +	prio-preempt \
> +	prio-wake \
> +	pthread_kill_latency \
> +	sched_football \
> +	sched_jitter \
> +	sched_latency \
>  	thread_clock
>
> -
>  MAINTAINERCLEANFILES = \
>          GNUmakefile.in
>
> Index: config/GNUmakefile.am
> ===================================================================
> --- config/GNUmakefile.am.orig
> +++ config/GNUmakefile.am
> @@ -1,3 +1,3 @@
>  MAINTAINERCLEANFILES = \
> -        GNUmakefile.in
> +	GNUmakefile.in
>
> Index: GNUmakefile.am
> ===================================================================
> --- GNUmakefile.am.orig
> +++ GNUmakefile.am
> @@ -1,9 +1,9 @@
>  SUBDIRS = \
>  	config \
> -        include \
> -        func \
> -        perf \
> -        stress
> +	include \
> +	func \
> +	perf \
> +	stress
>
>  EXTRA_DIST = \
>  	autogen.sh
> Index: perf/GNUmakefile.am
> ===================================================================
> --- perf/GNUmakefile.am.orig
> +++ perf/GNUmakefile.am
> @@ -3,3 +3,4 @@ SUBDIRS = \
>
>  MAINTAINERCLEANFILES = \
>  	GNUmakefile.in
> +
> Index: func/gtod_latency/GNUmakefile.am.rej
> ===================================================================
> --- func/gtod_latency/GNUmakefile.am.rej
> +++ /dev/null
> @@ -1,21 +0,0 @@
> -*************** gtod_infinite_CFLAGS = \
> -*** 12,20 ****
> -  	-O2
> -
> -  LDADD = \
> --          ${top_srcdir}/include/libjvmsim.a \
> --          ${top_srcdir}/include/librt.a \
> --          ${top_srcdir}/include/libstats.a
> -
> -  LDFLAGS = \
> -  	-L$(DESTDIR)/usr/lib/nptl \
> ---- 12,20 ----
> -  	-O2
> -
> -  LDADD = \
> -+ 	$(top_srcdir)/include/libjvmsim.a \
> -+ 	$(top_srcdir)/include/librt.a \
> -+ 	$(top_srcdir)/include/libstats.a
> -
> -  LDFLAGS = \
> -  	-L$(DESTDIR)/usr/lib/nptl \
>
> --
>  Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
>  Pengutronix - Linux Solutions for Science and Industry
>    Handelsregister:  Amtsgericht Hildesheim, HRA 2686
>      Hannoversche Str. 2, 31134 Hildesheim, Germany
>    Phone: +49-5121-206917-0 |  Fax: +49-5121-206917-9



-- 
Darren Hart
IBM Linux Technology Center
Realtime Linux Team
-
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux