This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Per-machine additions to CFLAGS-rtld.c being overridden.

On 09 Jan 2013 18:03, Roland McGrath wrote:
> There are several layers here.

since this is 3 years old, full thread can be found starting here:

> * Using per-file (or per-directory) *FLAGS settings is always the worst
>   option.  Not only is it obscure and easily overlooked, it doesn't get
>   tracked as a dependency to recompile things as needed.  We should avoid
>   it whenever possible.
>   If it's used for things like -D switches there is just no excuse.	
>   That can be done with wrapper source files in sysdeps instead.
>   When it's used for code-generation options, it can't really be avoided at
>   the moment.  One day the GCC pragma/attribute features for embedding such
>   options directly in the sources will be complete and we'll require
>   building libc with a GCC new enough to support all that.  But that will
>   be a long time off.

this seems to be in disagreement with GNU standards where you can do
`make bindir=...` and have the path be used (rather than the one set
at configure time).  ignoring that, it's the only real sane way to
fully expand the path because the path could be nested in other vars.
e.g. something like:

specifically, we're talking about:
SYSCONF-FLAGS := -D'SYSCONFDIR="$(sysconfdir)"'
CFLAGS-ldconfig.c = $(SYSCONF-FLAGS) -D'LIBDIR="$(libdir)"' -D'SLIBDIR="$(slibdir)"'
libof-ldconfig = ldconfig
CFLAGS-dl-cache.c = $(SYSCONF-FLAGS)

do we want to move these -D flags to Makeconfig's CPPFLAGS ?
and do it for all the path related variables ?  it would make
the command line blow up a bit ... but then we wouldn't have
per-file or per-dir settings in more places.

Attachment: signature.asc
Description: Digital signature

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]