Skip to content

Conversation

@tmds
Copy link
Member

@tmds tmds commented Jul 7, 2022

Without the 'x'-bit, the directories that get created are not accessible.

@eerhardt @carlossanlop @dotnet/area-system-io ptal.

Without the 'x'-bit, the directories that get created are not
accessible.
@ghost ghost added area-System.IO community-contribution Indicates that the PR has been added by a community member labels Jul 7, 2022
@ghost
Copy link

ghost commented Jul 7, 2022

Tagging subscribers to this area: @dotnet/area-system-io
See info in area-owners.md if you want to be subscribed.

Issue Details

Without the 'x'-bit, the directories that get created are not accessible.

@eerhardt @carlossanlop @dotnet/area-system-io ptal.

Author: tmds
Assignees: -
Labels:

area-System.IO

Milestone: -

@stephentoub
Copy link
Member

Without the 'x'-bit, the directories that get created are not accessible.

Do we need a test that untars something containing a directory and then tries to access it, e.g. copying something inside it, reading something from it, etc.? It sounds like that would fail without this fix?

Co-authored-by: Stephen Toub <stoub@microsoft.com>
@tmds
Copy link
Member Author

tmds commented Jul 7, 2022

Do we need a test that untars something containing a directory and then tries to access it, e.g. copying something inside it, reading something from it, etc.? It sounds like that would fail without this fix?

Yes, it would fail without the fix.
The existing tests have the expected permission hard coded, so they'll also fail if this regresses.

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Copy link
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@carlossanlop
Copy link
Contributor

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I ran the runtime-extra-platforms pipeline, to make sure all the exotic platforms pass too.
If it all looks good, we can merge it.

@carlossanlop
Copy link
Contributor

No tar failures. :shipit:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Formats.Tar community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants