FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Use NIO to set file permissions by chingor13 · Pull Request #557 · googleapis/google-http-java-client · GitHub

Use NIO to set file permissions - #557

Merged
chingor13 merged 5 commits into
googleapis:masterfrom
chingor13:file-permissions-nio
Jun 17, 2019
Merged

Use NIO to set file permissions#557
chingor13 merged 5 commits into
googleapis:masterfrom
chingor13:file-permissions-nio

Conversation

Copy link
Copy Markdown
Contributor

Fixes #315

We're dropping Java 6 support in this next release, so we can now use NIO to set file permissions in Windows environments

chingor13 requested a review from a team January 3, 2019 00:45
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 3, 2019

Copy link
Copy Markdown
Contributor Author

Adding windows test runner in #562

chingor13 added the status: blocked Resolving the issue is dependent on other work. label Jan 4, 2019
yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Feb 7, 2019
sduskis added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 10, 2019
kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 10, 2019

acurvers commented Jun 1, 2019

Copy link
Copy Markdown

why did this not get merged?

Copy link
Copy Markdown
Contributor Author

This was not merged as we were waiting until we set up a Windows test machine. This is blocked by #716

chingor13 added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed status: blocked Resolving the issue is dependent on other work. labels Jun 16, 2019
kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 16, 2019
yoshi-automation added the 🚨 This issue needs some love. label Jun 16, 2019
chingor13 merged commit 4e77793 into googleapis:master Jun 17, 2019
chingor13 deleted the file-permissions-nio branch June 17, 2019 17:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. 🚨 This issue needs some love.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FileDataStoreFactory method setPermissionsToOwnerOnly only supports Unix

6 participants


Back | FazBrowse Home | New Git URL