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

An exception is reported when a context file path is longer than 100 chars #26

Closed
pditommaso opened this issue Oct 22, 2023 · 1 comment · Fixed by #38
Closed

An exception is reported when a context file path is longer than 100 chars #26

pditommaso opened this issue Oct 22, 2023 · 1 comment · Fixed by #38
Assignees

Comments

@pditommaso
Copy link
Contributor

The following error is reported when the building context includes long file paths

java.lang.IllegalArgumentException: file name '.docusaurus/docusaurus-plugin-content-docs/default/category-docs-tutorialsidebar-category-tutorial-basics-918.json' is too long ( > 100 bytes)
        at org.apache.commons.compress.archivers.tar.TarArchiveOutputStream.handleLongName(TarArchiveOutputStream.java:702)
        at org.apache.commons.compress.archivers.tar.TarArchiveOutputStream.putArchiveEntry(TarArchiveOutputStream.java:366)
        at io.seqera.wave.util.Packer.makeTar(Packer.java:82)
        at io.seqera.wave.util.Packer.layer(Packer.java:136)
        at io.seqera.wave.util.Packer.layer(Packer.java:132)
        at io.seqera.wave.util.Packer.layer(Packer.java:120)
        at io.seqera.wave.cli.App.prepareContext(App.java:459)
        at io.seqera.wave.cli.App.run(App.java:398)
        at io.seqera.wave.cli.App.main(App.java:217)

A quick search shows that it should be enabled the support for long file names in the tar using the following:

TarArchiveOutputStream stream = new TarArchiveOutputStream(...)
stream.setLongFileMode(TarArchiveOutputStream.LONGFILE_POSIX)

Read more here.

@munishchouhan
Copy link
Member

munishchouhan commented Oct 23, 2023

@pditommaso fix has been made in libseqera
seqeralabs/libseqera#8
@jason-seqera Once this is merged and a new version of wave-util is available
I will create PR here too

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 a pull request may close this issue.

2 participants