public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ulrich Drepper <drepper@redhat.com>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Al Viro <viro@ftp.linux.org.uk>,
	linux-kernel@vger.kernel.org, akpm@osdl.org
Subject: Re: [PATCH] Implement AT_SYMLINK_FOLLOW flag for linkat
Date: Sun, 18 Jun 2006 12:40:43 -0700	[thread overview]
Message-ID: <4495AC3B.4020508@redhat.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0606181220590.5498@g5.osdl.org>

[-- Attachment #1: Type: text/plain, Size: 1022 bytes --]

Linus Torvalds wrote:
> Well, the patch as sent in does seem sane, as long as glibc doesn't start 
> defaulting to the insane behaviour. Giving users the _ability_ to link to 
> the symlink target is certainly not wrong, regardless of any standard. 
> Doing it by default is another matter.

I do not intend to change the link implementation in glibc.  That would
be majorly stupid, it'd break the ABI.

The AT_SYMLINK_FOLLOW flag to linkat was the result of the discussion
how to resolve the issue of the conflict between POSIX and the Linux
implementation of link (BTW: the Solaris link syscall behaves the same
as Linux's).  This is an easy an non-intrusive way to help people who
depend on the questionable POSIx-mandated behavior to work around the
incompatiblity.  Nothing more.  Don't change the link syscall, don't
assume the glibc will be changed.  This is only one little extra bit of
new functionality.

-- 
➧ Ulrich Drepper ➧ Red Hat, Inc. ➧ 444 Castro St ➧ Mountain View, CA ❖


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 251 bytes --]

  parent reply	other threads:[~2006-06-18 19:39 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-17 19:13 [PATCH] Implement AT_SYMLINK_FOLLOW flag for linkat Ulrich Drepper
2006-06-18 19:16 ` Al Viro
2006-06-18 19:22   ` Linus Torvalds
2006-06-18 19:32     ` Al Viro
2006-06-18 19:40     ` Ulrich Drepper [this message]
2006-06-18 22:06       ` Al Viro

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=4495AC3B.4020508@redhat.com \
    --to=drepper@redhat.com \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@osdl.org \
    --cc=viro@ftp.linux.org.uk \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox