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

Update Setup.pm for clarity about permissions #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dhutty
Copy link
Contributor

@dhutty dhutty commented Apr 4, 2013

No description provided.

print_tty "into data dir for different users. For non-root services to work\n";
print_tty "1777 grants for some data dir subdirectories is required.\n";
print_tty "(1777 grants means that everyone is able to write to the dir,\n";
print_tty "into various data directories for different users.\n";
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Note: English is not my native language. So I just want you to confirm that there's no ambiguity in your version.)

I'm reading "various data directories" as "one directory per user", which is not what happens. There's /var//lib/ubic/{status,lock,tmp,...}, each of these are 1777.
Am I wrong in my parsing of this sentence? Does saying "various" instead of, for example, "separate", clear things up enough in your opinion?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 4/6/13 5:52 AM, Vyacheslav Matyukhin wrote:

@@ -291,9 +291,10 @@ sub setup { my $enable_1777; if ($is_root)
{ print_tty "\nSystem-wide installations usually need to store
service-related data\n"; - print_tty "into data dir for
different users. For non-root services to work\n"; -
print_tty "1777 grants for some data dir subdirectories is
required.\n"; - print_tty "(1777 grants means that
everyone is able to write to the dir,\n"; + print_tty
"into various data directories for different users.\n";

(Note: English is not my native language. So I just want you to
confirm that there's no ambiguity in your version.)

Oh, I know. As a native English speaker who has just a smattering of a
few other languages, I'm always immensely impressed with how well
non-native speakers often learn English.

I'm reading "various data directories" as "one directory per user",
which is not what happens. There's
/var//lib/ubic/{status,lock,tmp,...}, each of these are 1777. Am
I wrong in my parsing of this sentence? Does saying "various"
instead of, for example, "separate", clear things up enough in your
opinion?

That was exactly why I wanted to eliminate "for different users".

Hm, it seems that github's 'edit in place' chewed up some of my
changes here. In order to be both correct and clear, I think it should
read something like:

System-wide installations usually need to store service-related data
in various data directories. In order for services that are not
running as root to work, those directories need to be accessible such
as with permissions mode 1777. (1777 means that everyone is able to
read and write to the directory but the 'sticky bit' is set so only
the file owner is able to modify or delete files)


Duncan Hutty
http://www.allgoodbits.org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.17 (Darwin)
Comment: GPGTools - http://gpgtools.org
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlFgFdoACgkQCFuTFybf1wpPbACfev3hFtXm11GG5DxfOX8uCm8F
KCYAnjZv1w+unm8Ax8rk4SpLLAt23seC
=Wsho
-----END PGP SIGNATURE-----

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants