mailing list of musl libc
 help / color / mirror / code / Atom feed
From: Rich Felker <dalias@aerifal.cx>
To: musl@lists.openwall.com
Subject: Re: Namespace issues, missing functions, _BSD_SOURCE for unistd.h
Date: Thu, 5 Apr 2012 21:52:59 -0400	[thread overview]
Message-ID: <20120406015259.GB8803@brightrain.aerifal.cx> (raw)
In-Reply-To: <20120405180307.38e2b00f@newbook>

On Thu, Apr 05, 2012 at 06:03:07PM -0700, Isaac Dunham wrote:
> > There is no "ANSI" namespace in unistd.h because it's not a standard C
> > header but a POSIX header. It's definitely correct for POSIX 2008 base
> > as-is without any patching.
> Should have checked that. 
> I guess it will be a rewrite then.
> 
> Which profiles are you aiming to support? Latest versions only for
> POSIX, ISO, and X/Open? Or only versions in the past decade?
> I don't see any signs of  version checks, so I assume you want POSIX2008
> and XOPEN2008 (_XOPEN_SOURCE >= 700) only, and no support for SUSv3.
> This will make the patch a lot smaller and cleaner.

My aim has always been the minimum set of profiles that will meet
real-world needs. Latest-version seems to be fine for that. The only
possible issue is interfaces that have been removed, but code that's
old enough to still be using interfaces that were marked obsolescent
11 years ago and removed 4 years ago probably needs _GNU_SOURCE or
similar anyway..

> <snip>
> > AFAIK most of these do not exist in musl at the moment.
> Guessed that.  It was meant as a comment on the status.

OK.

> > > +#if defined(_BSD_SOURCE) && !defined(L_SET)
> > > +#define L_SET
> > > +#define L_INCR
> > > +#define L_XTND
> > > +#endif
> Forgot to add the SEEK_SET, etc. here.
> Should be 
> #define L_SET SEEK_SET
> ...
> hence the placement.

OK.

Rich


  reply	other threads:[~2012-04-06  1:52 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-05 22:55 Isaac Dunham
2012-04-06  0:10 ` Rich Felker
2012-04-06  1:03   ` Isaac Dunham
2012-04-06  1:52     ` Rich Felker [this message]
2012-04-06  1:45   ` [PATCH] _BSD_SOURCE for unistd.h, take 2 Isaac Dunham
2012-04-06  2:04     ` Rich Felker
2012-04-06  2:40       ` Isaac Dunham
2012-04-06  2:51         ` Rich Felker
2012-04-06 14:48     ` Rich Felker
2012-04-06 15:48       ` Isaac Dunham
2012-04-06 23:32         ` Rich Felker
2012-04-07  5:47           ` Isaac Dunham

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120406015259.GB8803@brightrain.aerifal.cx \
    --to=dalias@aerifal.cx \
    --cc=musl@lists.openwall.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/musl/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).