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.

Sign up for GitHub

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

Jump to bottom

Incorrect documentation on cacheDirPerm #1340

Closed
hangshao0 opened this issue May 23, 2024 · 0 comments · Fixed by #1345
Closed

Incorrect documentation on cacheDirPerm #1340

hangshao0 opened this issue May 23, 2024 · 0 comments · Fixed by #1345
Assignees
@Sreekala-Gopakumar
Milestone
Release 0.46 (Jav...

Comments

@hangshao0
Copy link
Contributor

Page containing the error:

https://eclipse.dev/openj9/docs/xshareclasses/#cachedirperm

Describe the error and the update needed:

  1. If the cache directory is the platform default directory, <HomeDir>/.cache/javasharedresources, being the user's home directory, this suboption is ignored and the cache directory permissions are set to 0777.

    The default directory permission under homeDir is not set to 0777, so it should be changed to

    If the cache directory is the platform default directory, this suboption is ignored. The permission is set according to the following table.

  2. In the table:
    The cache directory is <HomeDir>/.cache/javasharedresources. If this directory already exists with different permissions, the permissions are changed when the cache is opened.† 0777

    We only change permission of /tmp/javasharedresources to 0777, so it should be changed to:

    The cache directory is /tmp/javasharedresources. If this directory already exists with different permissions, the permissions are changed when the cache is opened.† 0777

  3. In the table:
    The cache directory already exists and is not <HomeDir>/.cache/javasharedresources Unchanged
    It should be changed to:
    The cache directory already exists and is not /tmp/javasharedresources Unchanged
@hangshao0 hangshao0 mentioned this issue May 23, 2024
Merged
@Sreekala-Gopakumar Sreekala-Gopakumar self-assigned this May 31, 2024
Sreekala-Gopakumar added a commit to Sreekala-Gopakumar/openj9-docs that referenced this issue May 31, 2024
@Sreekala-Gopakumar
Update documentation on cacheDirPerm
50c0286 
eclipse-openj9#1340

Updated the [-Xshareclasses] (https://eclipse.dev/openj9/docs/xshareclasses/#cachedirperm) topic with the correct cache directory.

Closes eclipse-openj9#1340
Signed-off-by: Sreekala Gopakumar <[email protected]>
@Sreekala-Gopakumar Sreekala-Gopakumar mentioned this issue May 31, 2024
Merged
Sreekala-Gopakumar added a commit to Sreekala-Gopakumar/openj9-docs that referenced this issue Jun 4, 2024
@Sreekala-Gopakumar
Update documentation on cacheDirPerm
2b6633f 
eclipse-openj9#1340

Updated the [-Xshareclasses] (https://eclipse.dev/openj9/docs/xshareclasses/#cachedirperm) topic with the correct cache directory. Incorporated the feedback.

Closes eclipse-openj9#1340
Signed-off-by: Sreekala Gopakumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sreekala-Gopakumar Sreekala-Gopakumar

Labels
None yet
Projects
None yet
Milestone
Release 0.46 (Java 8, 11, 17, 21, 22)...
Development

Successfully merging a pull request may close this issue.

Update documentation on cacheDirPerm Sreekala-Gopakumar/openj9-docs
3 participants
@hangshao0 @pshipton @Sreekala-Gopakumar