-
Notifications
You must be signed in to change notification settings - Fork 13
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
FileWriter: Handle default permissions #243
Conversation
12e79a6
to
2fc79e6
Compare
… the default file permissions and create parent directory with CAF::Path
2fc79e6
to
7395d81
Compare
@jrha @jouvin @ned21 summary: we need this PR to fix the issue, and #244 and new build tools with quattor/maven-tools#160 for the unittests. i'm testing the new build-tools, but expect PRs with minor changes in most repos. if i see common pattern, i'll make a PR with even newer build-tools to smoothen the transition and get 17.3 out. |
src/main/perl/FileWriter.pm
Outdated
# Pass NoAction here, as it keeps track of the NoAction value during initialisation and/or keeps_state | ||
my $cafpath = CAF::Path::mkcafpath(log => *$self->{LOG}, NoAction => $options->{noaction}); | ||
# only create the directory if it didn't exist yet | ||
# if not, this would for the directory mode on existing directories |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I don't understand this comment. Did you mean "if not, this would change the directory mode on existing directories"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. i'll fix it
8163420
to
3fa7e4a
Compare
Fixes #241
This PR does not require #244, but a lot of unittests will fail if it is not merged first