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

[tomcat8] fix world access permissions #149

Merged
merged 2 commits into from
Aug 24, 2016

Conversation

chetan
Copy link
Contributor

@chetan chetan commented Aug 18, 2016

Default permissions included in the tarball don't give any world access. This is now a problem with the fixed svc_user/group in hab 0.9.0 (habitat-sh/habitat#1012)

@thesentinels
Copy link
Contributor

@chetan, thanks for your PR! By analyzing the annotation information on this pull request, we identified @jtimberman to be a potential reviewer

@chetan chetan force-pushed the fix_tomcat8_perms branch 2 times, most recently from 8808658 to 7f2543e Compare August 19, 2016 19:58
@bookshelfdave
Copy link
Contributor

@jtimberman any thoughts on this one?

@chetan
Copy link
Contributor Author

chetan commented Aug 23, 2016

I've actually got some more commits coming, based on discussions in slack. Should fix a few more issues..

@chetan chetan force-pushed the fix_tomcat8_perms branch from ff5a65d to d3da063 Compare August 24, 2016 13:50
@reset reset added the Bug label Aug 24, 2016
@reset
Copy link
Collaborator

reset commented Aug 24, 2016

@chetan drop a note here when you've finished with your additional commits 😄

@chetan
Copy link
Contributor Author

chetan commented Aug 24, 2016

@reset the last push was it. good to go

@reset reset merged commit cb2d3e3 into habitat-sh:master Aug 24, 2016
@reset reset removed the In Progress label Aug 24, 2016
@reset
Copy link
Collaborator

reset commented Aug 24, 2016

core/tomcat8/8.5.4/20160824210409 has been built and uploaded to the depot. 💖

@chetan chetan deleted the fix_tomcat8_perms branch August 26, 2016 14:39
@eeyun eeyun added Type:Bug and removed Bug labels Jun 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants