[Patches] [ python-Patches-1490190 ] add os.chflags() and os.lchflags() where available

[SourceForge.net]

Patches item #1490190, was opened at 2006-05-17 13:45
Message generated for change (Comment added) made by loewis
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1490190&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Modules
Group: Python 2.5
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: M. Levinson (levinsm)
Assigned to: Neal Norwitz (nnorwitz)
Summary: add os.chflags() and os.lchflags() where available

Initial Comment:
The return value from os.stat() includes st_flags on some systems, but
currently there's not much that can be done with the value; this patch aims
to make st_flags useful by adding some associated constants to stat.py and
the corresponding chflags() and lchflags() functions in posixmodule. For
completeness, shutil.copystat() is also updated to call os.chflags() where
it's available.


----------------------------------------------------------------------

>Comment By: Martin v. Löwis (loewis)
Date: 2007-02-19 11:56

Message:
Logged In: YES 
user_id=21627
Originator: NO

Thanks for the patch. Committed as r53826.

----------------------------------------------------------------------

Comment By: M. Levinson (levinsm)
Date: 2006-05-19 13:47

Message:
Logged In: YES 
user_id=1522893

In addition to MacOS, chflags(2) is available on FreeBSD,
OpenBSD, and
NetBSD. The flags in Lib/stat.py are the full set of
available values
although, as you noted, MacOS hasn't (yet) implemented
several of them.

Thanks for the comments - I've attached an updated version
of the patch
incorporating your suggestions.


----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2006-05-19 09:31

Message:
Logged In: YES 
user_id=33168

What operating systems is this available on?  The only one
I've found is OS X.  The man page says it's from BSD 4.4.  I
tried on Linux of various flavors (4+), Solaris, and Tru64.
 None of them had chflags.  I also could only find some of
the flags in my sys/stat.h that were added to stat.py. 
stat.h didn't have UF_NOUNLINK, SF_NOUNLINK, SF_SNAPSHOT.

As far as the patch itself, it looks good.  There are a
couple of changes if this should be accepted though:  doc
needs \versionadded{2.5}, I would prefer flags as the var
name rather than i in posixmodule.c (btw you shouldn't need
to init path).

Also would need to update Misc/NEWS and ACKS if accepted.

----------------------------------------------------------------------

Comment By: Georg Brandl (gbrandl)
Date: 2006-05-17 16:24

Message:
Logged In: YES 
user_id=849994

Patch looks good. Do we want to include it?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1490190&group_id=5470