[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 09 Aug 2011 17:48:42 +0200
From: Luka Marčetić <paxcoder@...il.com>
To: musl@...ts.openwall.com
Subject: Re: cluts makefiles
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
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
