All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jim Meyering <meyering@redhat.com>
To: Daniel J Walsh <dwalsh@redhat.com>
Cc: Russell Coker <russell@coker.com.au>
Cc: Stephen Smalley <sds@tycho.nsa.gov>,
	James Morris <jmorris@namei.org>, Eric Paris <eparis@redhat.com>,
	selinux@tycho.nsa.gov
Subject: Re: does mv need a --context=CTX (-Z) option, too?
Date: Thu, 10 Aug 2006 19:35:33 +0200	[thread overview]
Message-ID: <87y7twijdm.fsf@rho.meyering.net> (raw)
In-Reply-To: <44DB4585.3070500@redhat.com> (Daniel J. Walsh's message of "Thu, 10 Aug 2006 10:41:09 -0400")

Daniel J Walsh <dwalsh@redhat.com> wrote:
> Jim Meyering wrote:
>> Stephen Smalley <sds@tycho.nsa.gov> wrote:
>>
>>> On Thu, 2006-08-10 at 12:13 +0200, Jim Meyering wrote:
>>>
>>>> It might make sense to add a --context=CTX (-Z) option to mv.  Currently,
>>>> cp, install, mkdir, mknod, mkfifo all have that option, but not mv.
>>>> Most of the time, mv would have no need, since it simply calls rename.
>>>> But when that fails, it reverts to using the very same copying code
>>>> (copy.c) that cp uses.  It is trivial to add this option to mv, with the
>>>> understanding that it'd take effect solely for e.g., cross-device moves.
>>>> I.e., if you want to simulate a cross device move, you'd have to use
>>>> cp -pr and rm -rf, so if it makes sense for cp to have the --context=CTX
>>>> (-Z) option, then it follows that mv must accept it as well.
>>>>
>>> mv should just preserve the context of the original in all cases,
>>> whether it is just doing a rename(2) or copying the file.
>>>
>>
>> What if it's not possible to preserve the original, yet it
>> would be possible to set some other desirable context?
>>
>> Stepping back a little, if mv doesn't need the option, then why
>> does cp need it?  After all, cp has no option to set classic
>> permissions.
>>
>>
> I agree cp should not need this qualifier.  It should either be preserve
> or get the new default context.
> mv should just try to preserve
> install should use the matchpathcon similar to rpm.

Thanks, for your replies Dan and Karl.
I'm glad I asked.  Barring objections (Russell?), cp's -Z/--context=CTX
option will not make it to "upstream".

--
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:[~2006-08-10 17:35 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-10 10:13 does mv need a --context=CTX (-Z) option, too? Jim Meyering
2006-08-10 13:51 ` kmacmillan
2006-08-10 15:15   ` Jim Meyering
2006-08-10 16:00     ` James Antill
2006-08-10 16:01     ` Karl MacMillan
2006-08-10 17:39       ` Jim Meyering
2006-08-10 13:54 ` Stephen Smalley
2006-08-10 14:27   ` Jim Meyering
2006-08-10 14:41     ` Daniel J Walsh
2006-08-10 15:47       ` Casey Schaufler
2006-08-10 15:53         ` Daniel J Walsh
2006-08-10 16:01           ` Casey Schaufler
2006-08-10 16:03       ` Karl MacMillan
2006-08-10 17:35       ` Jim Meyering [this message]
2006-08-10 22:56         ` Russell Coker
2006-08-10 16:18   ` James Antill

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=87y7twijdm.fsf@rho.meyering.net \
    --to=meyering@redhat.com \
    --cc=dwalsh@redhat.com \
    --cc=russell@coker.com.au \
    /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.