Changeset 5c3511e in rtems for c/build-tools/src


Ignore:
Timestamp:
Jan 30, 1998, 9:49:51 PM (23 years ago)
Author:
Joel Sherrill <joel.sherrill@…>
Branches:
4.10, 4.11, 4.8, 4.9, 5, master
Children:
594a413
Parents:
421dfef6
Message:

Big patch form Ralf Corsepius described in this email:

Here is the result of my nightly work to get RTEMS_ROOT=$srcdir working
with different shells and relative/absolute paths.

What I did is relatively simple in principle:
Instead of setting RTEMS_ROOT in configure.in and then let configure
substitute @RTEMS_ROOT@ inside the Makefiles, I now let each Makefile
set RTEMS_ROOT from each Makefile's @top_srcdir@ value.

The difference is subtile, but with enormous side effects:

  • If RTEMS_ROOT is set in configure, then the same single value will be propagated to all Makefiles. This breaks using relative paths, as the relative path to the root of the source tree is used inside of all subdirectory Makefiles.
  • Now each Makefile.in sets RTEMS_ROOT = @top_srcdir@. top_srcdir is computed individually by configure for each single Makefile.in, hereby receiving the correct value, no matter if relative or absolute paths are used.

To get this working, I needed to remove setting RTEMS_ROOT from
target.cfg.in, because this overrides the value of RTEMS_ROOT from each
individual Makefile.

Furthermore, I removed RTEMS_CUSTOM from the Makefiles and replaced all
"include $(RTEMS_CUSTOM)" directives with"include
$(RTEMS_ROOT)/make/custom/$(RTEMS_BSP)". Perhaps you don't like this,
but I think, to have one variable less is clearer and easier to
understand than having several variables refering to the next one.

I enclose a small patch to this mail, which

  • fixes the config.h problem (to finally clearify misunderstands)
  • removes assignment/subsitution of RTEMS_ROOT from configure.in
  • contains a workaround for the application Makefile's RTEMS_ROOT problem (reported by Eric)
  • removes some unused lines from the toplevel Makefile.in
  • removes assignment of RTEMS_ROOT from make/target.cfg.in
File:
1 edited

Legend:

Unmodified
Added
Removed
  • c/build-tools/src/Makefile.in

    r421dfef6 r5c3511e  
    55@SET_MAKE@
    66srcdir = @srcdir@
    7 top_srcdir = @top_srcdir@
    87exec_prefix = @exec_prefix@
    98bindir = @bindir@
     
    1413
    1514VPATH = @srcdir@
    16 RTEMS_ROOT = @RTEMS_ROOT@
     15RTEMS_ROOT = @top_srcdir@
    1716PROJECT_ROOT = @PROJECT_ROOT@
    18 RTEMS_CUSTOM = $(RTEMS_ROOT)/make/custom/$(RTEMS_BSP).cfg
    1917
    2018# we use host compiler in this directory
     
    3331PGMS=$(ARCH)/cklength $(ARCH)/eolstrip $(ARCH)/packhex $(ARCH)/unhex
    3432
    35 include $(RTEMS_CUSTOM)
     33include $(RTEMS_ROOT)/make/custom/$(RTEMS_BSP).cfg
    3634include $(RTEMS_ROOT)/make/leaf.cfg
    3735
Note: See TracChangeset for help on using the changeset viewer.