source: rtems/bsps/m68k/ @ 18f285c

Last change on this file since 18f285c was 2afb22b, checked in by Chris Johns <chrisj@…>, on Dec 23, 2017 at 7:18:56 AM

Remove make preinstall

A speciality of the RTEMS build system was the make preinstall step. It
copied header files from arbitrary locations into the build tree. The
header files were included via the -Bsome/build/tree/path GCC command
line option.

This has at least seven problems:

  • The make preinstall step itself needs time and disk space.
  • Errors in header files show up in the build tree copy. This makes it hard for editors to open the right file to fix the error.
  • There is no clear relationship between source and build tree header files. This makes an audit of the build process difficult.
  • The visibility of all header files in the build tree makes it difficult to enforce API barriers. For example it is discouraged to use BSP-specifics in the cpukit.
  • An introduction of a new build system is difficult.
  • Include paths specified by the -B option are system headers. This may suppress warnings.
  • The parallel build had sporadic failures on some hosts.

This patch removes the make preinstall step. All installed header
files are moved to dedicated include directories in the source tree.
Let @RTEMS_CPU@ be the target architecture, e.g. arm, powerpc, sparc,
etc. Let @RTEMS_BSP_FAMILIY@ be a BSP family base directory, e.g.
erc32, imx, qoriq, etc.

The new cpukit include directories are:

  • cpukit/include
  • cpukit/score/cpu/@RTEMS_CPU@/include
  • cpukit/libnetworking

The new BSP include directories are:

  • bsps/include
  • bsps/@RTEMS_CPU@/include
  • bsps/@RTEMS_CPU@/@RTEMS_BSP_FAMILIY@/include

There are build tree include directories for generated files.

The include directory order favours the most general header file, e.g.
it is not possible to override general header files via the include path

The "bootstrap -p" option was removed. The new "bootstrap -H" option
should be used to regenerate the "" files.

Update #3254.

  • Property mode set to 100644
File size: 2.0 KB
1## This file was generated by "./boostrap -H".
3include_HEADERS =
4include_HEADERS += ../../../../../bsps/m68k/include/mvme16x_hw.h
6include_bspdir = $(includedir)/bsp
7include_bsp_HEADERS =
8include_bsp_HEADERS += ../../../../../bsps/m68k/include/bsp/linker-symbols.h
10include_mcf5206dir = $(includedir)/mcf5206
11include_mcf5206_HEADERS =
12include_mcf5206_HEADERS += ../../../../../bsps/m68k/include/mcf5206/mcf5206e.h
13include_mcf5206_HEADERS += ../../../../../bsps/m68k/include/mcf5206/mcfmbus.h
14include_mcf5206_HEADERS += ../../../../../bsps/m68k/include/mcf5206/mcfuart.h
16include_mcf5223xdir = $(includedir)/mcf5223x
17include_mcf5223x_HEADERS =
18include_mcf5223x_HEADERS += ../../../../../bsps/m68k/include/mcf5223x/mcf5223x.h
20include_mcf5225xdir = $(includedir)/mcf5225x
21include_mcf5225x_HEADERS =
22include_mcf5225x_HEADERS += ../../../../../bsps/m68k/include/mcf5225x/fec.h
23include_mcf5225x_HEADERS += ../../../../../bsps/m68k/include/mcf5225x/mcf5225x.h
25include_mcf5235dir = $(includedir)/mcf5235
26include_mcf5235_HEADERS =
27include_mcf5235_HEADERS += ../../../../../bsps/m68k/include/mcf5235/mcf5235.h
29include_mcf5272dir = $(includedir)/mcf5272
30include_mcf5272_HEADERS =
31include_mcf5272_HEADERS += ../../../../../bsps/m68k/include/mcf5272/mcf5272.h
33include_mcf5282dir = $(includedir)/mcf5282
34include_mcf5282_HEADERS =
35include_mcf5282_HEADERS += ../../../../../bsps/m68k/include/mcf5282/mcf5282.h
37include_mcf532xdir = $(includedir)/mcf532x
38include_mcf532x_HEADERS =
39include_mcf532x_HEADERS += ../../../../../bsps/m68k/include/mcf532x/mcf532x.h
41include_mcf548xdir = $(includedir)/mcf548x
42include_mcf548x_HEADERS =
43include_mcf548x_HEADERS += ../../../../../bsps/m68k/include/mcf548x/MCD_dma.h
44include_mcf548x_HEADERS += ../../../../../bsps/m68k/include/mcf548x/MCD_progCheck.h
45include_mcf548x_HEADERS += ../../../../../bsps/m68k/include/mcf548x/MCD_tasksInit.h
46include_mcf548x_HEADERS += ../../../../../bsps/m68k/include/mcf548x/mcdma_glue.h
47include_mcf548x_HEADERS += ../../../../../bsps/m68k/include/mcf548x/mcf548x.h
Note: See TracBrowser for help on using the repository browser.