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

Comply with current umask when generating new app #12048

Merged
merged 1 commit into from Aug 27, 2013
Merged

Comply with current umask when generating new app #12048

merged 1 commit into from Aug 27, 2013

Conversation

ghost
Copy link

@ghost ghost commented Aug 27, 2013

When creating a new application using the default rails generator
(rails new my_app), it will apply some permissions bypassing the
current process umask. The `bin' directory and any files inside it will
always be set a mode of 0755.

This change removes the current umask bits from the mode applied by
the generator on the `bin' directory and its content.

  When creating a new application using the default rails generator
(`rails new my_app`), it will apply some permissions bypassing the
current process umask. The `bin' directory and any files inside it will
always be set a mode of 0755.

  This change removes the current umask bits from the mode applied by
the generator on the `bin' directory and its content.
tenderlove added a commit that referenced this pull request Aug 27, 2013
Comply with current umask when generating new app
@tenderlove tenderlove merged commit 169c214 into rails:master Aug 27, 2013
tenderlove added a commit that referenced this pull request Aug 27, 2013
Comply with current umask when generating new app
tenderlove added a commit that referenced this pull request Aug 27, 2013
Comply with current umask when generating new app
Conflicts:
	railties/lib/rails/generators/rails/app/app_generator.rb
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.

None yet

1 participant