Re: cluts makefiles

["Luka Marčetić" <paxcoder@gmail.com>]

On 08/09/2011 01:30 PM, Vasiliy Kulikov wrote:
> Hi,
>
> This is a patch to enhance musl building things.
>
> 1) Divided a single Makefile to the cluts, tests/, compile flags.
>
> 2) Used gcc's ability to identify dependencies.

Do others free compilers support nr 2?

> Unrelated things:
>
> include "sequence.c" is weird :)  It's better to use .h with
> declarations and .c files which build into .o.

It's an ad-hoc dir structure, therefore suboptimal. I'll add prototypes 
and other declarations into .h some day.

> With glibc:
>
> a) _SVID_SOURCE is needed for alphasort.

Maybe older glibc needed it (I get no warnings). I'm not sure it 
warrants an inclusion (it's SUSv4).

> b) SA_NODEFER is undefined in all .c.

It should be defined in signal.h. Alexander had problems with this. 
Trying to apply the patch that you wrote told me you're missing an 
_XOPEN_SOURCE flag I've relatively recently added to the Make-file on 
Rich's incentive, so perhaps you could should try that. If it's an old 
kernel version instead, necessary modifications would be more extensive, 
involving unblocking when entering the handler (sigprocmask?), removing 
SA_NODEFERs, using sig*jmp, and perhaps more in some cases.

> diff --git a/Makefile b/Makefile

Having made dirs a and b (both from the same repo dir), I tried `patch 
-p0 < yourpatch`, and got:

Hunk #1 FAILED at 7.
1 out of 1 hunk FAILED -- saving rejects to file a/Makefile.rej
patching file b/defines.mk
patching file b/tests/Makefile

I tried to replace the offending line with:

-CFLAGS = -std=c99 -D_XOPEN_SOURCE=700 -D_POSIX_C_SOURCE=200809L -Wall 
-Wextra -O2

but somehow the error message persists. Is there something I'm missing?

Thanks,
Luka