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

[SourceForge.net]

Patches item #1490190, was opened at 2006-05-17 04:45
Message generated for change (Comment added) made by nnorwitz
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: Open
Resolution: None
Priority: 5
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: Neal Norwitz (nnorwitz)
Date: 2006-05-19 00: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 07: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