Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

os.Chmod should accept os.FileMode #2733

Closed
edsrzf opened this issue Jan 19, 2012 · 5 comments
Closed

os.Chmod should accept os.FileMode #2733

edsrzf opened this issue Jan 19, 2012 · 5 comments
Milestone

Comments

@edsrzf
Copy link

edsrzf commented Jan 19, 2012

os.Chmod expects its mode argument to be a uint32, but it should be a FileMode instead.
Other os functions with a similar issue:

Mkdir
MkdirAll
OpenFile
File.Chmod
@dsymonds
Copy link
Contributor

Comment 1:

Labels changed: added priority-go1, removed priority-triage.

@bradfitz
Copy link
Contributor

Comment 2:

Owner changed to @bradfitz.

Status changed to Started.

@edsrzf
Copy link
Author

edsrzf commented Jan 19, 2012

Comment 3:

Before you get too far into this, I should point out that this CL is a prerequisite:
http://golang.org/cl/5539063/
(I was also planning on following up, but if you want to do it that's fine too.)

@bradfitz
Copy link
Contributor

Comment 4:

http://golang.org/cl/5553064

@bradfitz
Copy link
Contributor

Comment 5:

This issue was closed by revision 6454a3e.

Status changed to Fixed.

@rsc rsc added this to the Go1 milestone Apr 10, 2015
@rsc rsc removed the priority-go1 label Apr 10, 2015
@golang golang locked and limited conversation to collaborators Jun 24, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants