All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jim Meyering <jim@meyering.net>
To: James Antill <james.antill@redhat.com>
Cc: Stephen Smalley <sds@tycho.nsa.gov>,
	Daniel J Walsh <dwalsh@redhat.com>,
	SE Linux <selinux@tycho.nsa.gov>,
	Karl MacMillan <kmacmillan@mentalrootkit.com>
Subject: Re: We currently have a problem with cp -a /media/cdrom  /etc
Date: Fri, 12 Jan 2007 22:19:58 +0100	[thread overview]
Message-ID: <87ps9keydd.fsf@rho.meyering.net> (raw)
In-Reply-To: <1168632626.13080.212.camel@code.and.org> (James Antill's message of "Fri, 12 Jan 2007 15:10:26 -0500")

James Antill <james.antill@redhat.com> wrote:
> On Fri, 2007-01-12 at 13:55 -0500, Stephen Smalley wrote:
>> On Fri, 2007-01-12 at 13:46 -0500, Daniel J Walsh wrote:

Thanks for the Cc.

...
>> First, I seem to recall debates about whether -p (and thus -a) should
>> automatically enable preservation of SELinux context or whether that
>> should remain a separate option only.  Pros and cons to both sides.
>
>  I never saw the original discussion, but I'm 99% sure I complained to
> Jim and/or Dan that cp -a/-p should do "the right thing" and preserve
> everything, when I was adding the xattr support.

If only "the right thing" worked in all cases.
FWIW, my cut at this[*] has cp -a still equivalent to cp -dpPR,
just like rawhide, below.

Do you know what motivated the "mode = acls[1] no" change below?
That introduces a new incompatibility with upstream.
Otherwise, upstream matches rawhide.

[*] I resumed work on my coreutils selinux merge recently, so you should
hear more about it soon -- on the bug-coreutils and bug-gnulib lists.

>  Note that the changes in question are not really consistent:
>
>                FC-5           FC-6            rawhide
>
> mode = acls[1] yes            yes             no
> -p             trad.          trad.           trad.+acls+xattrs
> -a             -dpPR          -dpPRc          -dpPR
> --preserve=all trad.+context  trad.+context   trad.+context+acls+xattrs
...
> [1] This means that preserving the "unix permission mode bits" implies
> preserving any ACLs.

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

  parent reply	other threads:[~2007-01-12 21:19 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-12 18:46 We currently have a problem with cp -a /media/cdrom /etc Daniel J Walsh
2007-01-12 18:55 ` Stephen Smalley
2007-01-12 20:10   ` James Antill
2007-01-12 20:24     ` Stephen Smalley
2007-01-12 20:29       ` Stephen Smalley
2007-01-12 22:29         ` Casey Schaufler
2007-01-13  9:13           ` Russell Coker
2007-01-13 20:01             ` Casey Schaufler
2007-01-12 21:15       ` James Antill
2007-01-12 21:19         ` Stephen Smalley
2007-01-13 10:05         ` Jim Meyering
2007-01-15  5:16           ` James Antill
2007-01-15  7:54             ` Jim Meyering
2007-01-16 13:33               ` Stephen Smalley
2007-01-13 10:55         ` Russell Coker
2007-01-12 21:01     ` Stephen Smalley
2007-01-12 21:29       ` Jim Meyering
2007-01-12 21:19     ` Jim Meyering [this message]
2007-01-12 21:21       ` Stephen Smalley
2007-01-12 21:47         ` Jim Meyering
2007-01-12 21:56           ` Stephen Smalley

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=87ps9keydd.fsf@rho.meyering.net \
    --to=jim@meyering.net \
    --cc=dwalsh@redhat.com \
    --cc=james.antill@redhat.com \
    --cc=kmacmillan@mentalrootkit.com \
    --cc=sds@tycho.nsa.gov \
    --cc=selinux@tycho.nsa.gov \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.