source: rtems/c/src/tests/itrontests/itronmbf01/Makefile.am @ 67077a1

Last change on this file since 67077a1 was 67077a1, checked in by Joel Sherrill <joel.sherrill@…>, on 03/21/00 at 17:11:40

Patches rtems-rc-4.5.0-1.diff from Ralf Corsepius <corsepiu@…>
that addresses the following:

  • Duplicate variables in Makefiles (many Makefile.ams below c/src/test are affected systematically)
  • Erroniously using local.am instead of host.am in host-Makefile.am (Only host Makefile.ams should be affected; Erroniously using local.am in host-Makefiles doesn't desturb much)
  • use '.' instead of '$pwd' in ./bootstrap (using $pwd does not work if $pwd is a symlink on linux).
  • Broken CVS Ids somewhere
  • Removing redundant/obsolete make variables from *.cfg files.

Except of the last item from the list above, most parts of this patch
are fairly harmless, sometimes even cosmetical.

As mentioned before, this patch also contains a new ampolish script.

This script features:

  • Pretty printing of Makefile.ams (eg. removal of trailing spaces, removal of duplicate empty lines, pretty printing make variables, etc.).
  • Some syntactical checks on the contents of Makefiles.am
  • Proper handling of Automake conditionals

FYI:

  • Applying tools/update/rtems-polish.sh -am completely reformats all Makefile.am resulting into a very large (~500k) diff.
  • Applying tools/update/rtems-polish.sh -am twice, finally does not reformat the Makefile.ams anymore.
  • Many parts of the patch above result from merging back issues which have shown when applying this new ampolish (i.e. partially result from extracting the essentials of reformating being proposed by applying it on Makefile.ams).

Though this ampolish is a very nice tool, IMHO, I am hestitant if you
should apply (i.e. run tools/update/rtems-polish.sh -am) it to the
sources before the release, because

  • the resulting diff is fairly large
  • I am not 100% sure it doesn't break anything.

However, applying it after the release would result into compatibility
problems in applying patches ;)

I would suggest that you might consider trying it locally, then to
examine the diff and then to decide whether to apply it in general or
not.

Joel's Comments:

As Ralf points out, this patch is problematic in that applying it before
a release could break things but applying it afterwards will result in
patches being unusable for Makefiles. My inclination is to forge ahead
and apply it.
  • Property mode set to 100644
File size: 678 bytes
Line 
1##
2## $Id$
3##
4
5AUTOMAKE_OPTIONS = foreign 1.4
6TEST = itronmbf01
7PGM = ${ARCH}/$(TEST).exe
8
9MANAGERS = all
10
11C_FILES = init.c
12C_O_FILES = $(C_FILES:%.c=${ARCH}/%.o)
13
14H_FILES = system.h
15noinst_HEADERS =$(H_FILES)
16
17DOCTYPES = doc scn
18DOCS = $(DOCTYPES:%=$(TEST).%)
19
20SRCS = $(DOCS) $(C_FILES) $(H_FILES)
21OBJS = $(C_O_FILES)
22
23PRINT_SRCS = $(DOCS)
24
25include $(RTEMS_ROOT)/make/custom/@RTEMS_BSP@.cfg
26include $(RTEMS_ROOT)/make/leaf.cfg
27include $(top_srcdir)/itrontests.am
28
29#
30# (OPTIONAL) Add local stuff here using +=
31#
32
33${PGM}: $(OBJS) $(LINK_FILES)
34        $(make-exe)
35
36all: $(ARCH) $(TMPINSTALL_FILES)
37
38EXTRA_DIST = $(C_FILES) $(DOCS)
39
40include $(top_srcdir)/../../../../automake/local.am
Note: See TracBrowser for help on using the repository browser.