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

Fix for tar.extract new method option #28

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

inka
Copy link

@inka inka commented Apr 18, 2019

After e71c22d upgrade of tar from 2.2.1 to 4.4.8 the resolver is still completely broken if you use tar dependencies.

The options of tar had changed from

path: '/path/to/extract/tar/into',

to

cwd: Extract files relative to the specified directory. Defaults to process.cwd(). If provided, this must exist and must be a directory. [Alias: C]

Wasn't able to adjust the fetch test case. Maybe someone else can do.

Please consider 1.0.3 and 1.0.4 as not working.

Would be nice if you can release this PR as 1.0.5 soon.

This fixes #27

@jawwadfarooq
Copy link

When are you going to release the fix?

@jschaeke
Copy link

Any update regarding this prq?

@joedragons
Copy link
Contributor

Please watch and vote for https://issues.sonatype.org/browse/NEXUS-19735. You'll see movement there.

@irbian
Copy link

irbian commented Dec 1, 2021

well, the movement was to close the issue without merging...

@armujahid
Copy link

Yes. https://issues.sonatype.org/browse/NEXUS-19735 resolution status has been changed to " Won't Fix" :(

@mfilotto
Copy link

Hi,
Any update on this one ?

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.

Tar dependencies are unpacked to wrong directory
7 participants