Linux Permissions #132

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
3 participants
@eliotfowler

Type octal notation "0644" and get back:
User: Read and Write
Group: Read
Others: Read

rw-r--r--

Future:
Right now I don't do anything with the special bit (the first one) so I would like to add support for that in later commits

I would also eventually like to add support for reverse octal notation, for example:
rwx-rw-rw- to 0766

@majuscule

This comment has been minimized.

Show comment Hide comment
@majuscule

majuscule Nov 2, 2012

Member

Hi fowlerje!

Thanks for writing this plugin :-). Right off the bat, it looks good! Here are a few things I'd like your opinions on:

Is there anything here specific to linux? If not, maybe we should rename this UnixPermissions? Also, I don't see any reason we shouldn't also accept modes without the special bit. We should also consider additional trigger words, i.e. trigger on queries like "permissions 0644", and perhaps add a link to documentation like this wiki page? And a last, trivial change, lets only capitalize the first word, like "Read, write and execute".

Let me know all your thoughts!

Thanks again,
Dylan

Member

majuscule commented Nov 2, 2012

Hi fowlerje!

Thanks for writing this plugin :-). Right off the bat, it looks good! Here are a few things I'd like your opinions on:

Is there anything here specific to linux? If not, maybe we should rename this UnixPermissions? Also, I don't see any reason we shouldn't also accept modes without the special bit. We should also consider additional trigger words, i.e. trigger on queries like "permissions 0644", and perhaps add a link to documentation like this wiki page? And a last, trivial change, lets only capitalize the first word, like "Read, write and execute".

Let me know all your thoughts!

Thanks again,
Dylan

@eliotfowler

This comment has been minimized.

Show comment Hide comment
@eliotfowler

eliotfowler Nov 7, 2012

I agree with all of this and will get working on it! I am unsure as how to add html to the end of the response but I will try to figure it out.

I agree with all of this and will get working on it! I am unsure as how to add html to the end of the response but I will try to figure it out.

@zekiel

This comment has been minimized.

Show comment Hide comment
@zekiel

zekiel Feb 9, 2013

Owner

Any update on this?

Owner

zekiel commented Feb 9, 2013

Any update on this?

@majuscule

This comment has been minimized.

Show comment Hide comment
@majuscule

majuscule Mar 22, 2013

Member

Hi everyone,

@koosha-- reimplemented this plugin and finished the functionality we discussed. We've merged it in, so I'm closing this pull request. That said, @FowlerJE, thank you for writing this spice! I hope we see another pull request from you in the future.

Dylan

Member

majuscule commented Mar 22, 2013

Hi everyone,

@koosha-- reimplemented this plugin and finished the functionality we discussed. We've merged it in, so I'm closing this pull request. That said, @FowlerJE, thank you for writing this spice! I hope we see another pull request from you in the future.

Dylan

@majuscule majuscule closed this Mar 22, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment