Skip to content

Use relative rather than absolute symlink in t/analyse/manifest.t#51

Open
pghmcfc wants to merge 1 commit into
cpants:masterfrom
pghmcfc:symlinks
Open

Use relative rather than absolute symlink in t/analyse/manifest.t#51
pghmcfc wants to merge 1 commit into
cpants:masterfrom
pghmcfc:symlinks

Conversation

@pghmcfc
Copy link
Copy Markdown

@pghmcfc pghmcfc commented May 29, 2026

Archive::Tar since version 3.08 will refuse to extract absolute symlinks unless $Archive::Tar::INSECURE_EXTRACT_MODE is set, leading to this test suite failure:

Symlink 'Module-CPANTS-Analyse-Test-0.01/MANIFEST.lnk' has absolute target. Not extracting under SECURE EXTRACT MODE at /usr/share/perl5/vendor_perl/Archive/Any/Lite.pm line 130.
t/analyse/manifest.t .....
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/14 subtests

This problem can be avoided by using relative symlinks:
MANIFEST.lnk -> MANIFEST
rather than absolute symlinks:
MANIFEST.lnk -> /path/to/tmpdir/MANIFEST
in the test.

Archive::Tar since version 3.08 will refuse to extract absolute symlinks
unless $Archive::Tar::INSECURE_EXTRACT_MODE is set, leading to this test
suite failure:

Symlink 'Module-CPANTS-Analyse-Test-0.01/MANIFEST.lnk' has absolute target. Not extracting under SECURE EXTRACT MODE at /usr/share/perl5/vendor_perl/Archive/Any/Lite.pm line 130.
t/analyse/manifest.t .....
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/14 subtests

This problem can be avoided by using relative symlinks:
  MANIFEST.lnk -> MANIFEST
rather than absolute symlinks:
  MANIFEST.lnk -> /path/to/tmpdir/MANIFEST
in the test.
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.

1 participant