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

Set current user as an owner for parent directories #22677

Open
wants to merge 1 commit into
base: blead
Choose a base branch
from

Conversation

oxidase
Copy link

@oxidase oxidase commented Oct 18, 2024

The problem happens in a Docker container with mounted case-sensitive APFS when running `miniperl_top "-I../../lib" -MExtUtils::Command -e 'mkpath' -- ../../lib/auto/lib as

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
ERROR: Can't create '../../lib/auto'
Do not have write permissions on '../../lib/auto'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

where ../../lib/auto has root as an owner instead of the current user.

The PR should fix the issue by setting the owner for all created directories.

TODO: fill description here


  • This set of changes requires a perldelta entry, and it is included.
  • This set of changes requires a perldelta entry, and I need help writing it.
  • This set of changes does not require a perldelta entry.

The problem happens in a Docker container with mounted case-sensitive APFS when running `miniperl_top "-I../../lib" -MExtUtils::Command -e 'mkpath' -- ../../lib/auto/lib
as
```
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
ERROR: Can't create '../../lib/auto'
Do not have write permissions on '../../lib/auto'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
```

where ../../lib/auto has root as an owner instead of the current user. 

The PR should fix the issue by setting the owner for all created directories.
@Grinnz
Copy link
Contributor

Grinnz commented Oct 18, 2024

File::Path is maintained upstream on CPAN, so changes must be made there first. Issue tracker: https://rt.cpan.org/Public/Dist/Display.html?Name=File-Path, repository: https://github.com/jkeenan/File-Path

@oxidase
Copy link
Author

oxidase commented Oct 18, 2024

Gotcha, opened a PR on github and sent an email to bug-tracker

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.

2 participants