Hello

After reading again the documentations and playing
again with kbuild I've found:


BUG:

$ make mrproper
$ make mrproper
Error: you must create a .config first

mrproper should not depend on .config (contents and
existance).


COMMENT:

Your documentation is extensive, but I didn't found
the meaning of targets.
I think that now (with kbuild) we can nearly always
replace mrproper with clean. (and mrproper will
do clean + rm .config).
What do you things?
(in that case we should replace the occurence of
mrproper, (also on kbuild docs)).


BUG ?

making arch/i386/boot/bbotsect.o:
old:
 bbootsect.s: Assembler message:
 bbootsect.s:257: Warning ....
new:
 /tmp/ccM????: Assempler message
 /tmp/ccM5???:257" Warning ...

What is the more user-friendly message?

The cause: you forget '-pipe'.
(see next comment)


COMMENT AS #1

-pipe should be passed to AS (i386),
(as the -I directives)
How? duplicating the flags, or reusing the
compiler flags?
(but see also next comment)


COMMENT AS #2, maybe BUG

hmm.
>From (g)as docs: 'as' need already preproccesed
code (no macro, ...).

You define AS as GCC, thus as CPP + AS (thus
.[Ss] -> .o). But this is IMHO wrong. Users may
expect AS as the 'simple' assembler (.s -> .o).
I don't cross-compile, so I don't know who explict
use AS and how it is used.
Possible change:
- replace AS with GCC
This will force the use of same assembler of gcc.
(and gcc can use also external/non-GNU assemblers)
Maybe we can also modify the other tools to use
GCC, e.g. LD=$GCC, CC=$GCC, ...
In this case we have an alternate form to call
cross-compiler: make GCC=gcc -V..., and
also in this case -pipe will be an option to GCC
(-pipe is one of the few general option of gcc)
- allow user to change also the 'true' assembler.
(without performance loss for the other  99.999
people) (maybe this should be only a few line
of documentation)


COMMENT

You should change the number of the phases.
I really don't like phase 5 before phase 4.
(Let a missing phase number of first compile,
but let the pahse always be in order)


   giacomo

_______________________________________________
kbuild-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/kbuild-devel

Reply via email to