Skip to content

[pull] main from quarkusio:main#780

Merged
pull[bot] merged 5 commits intobschuhmann:mainfrom
quarkusio:main
Mar 25, 2026
Merged

[pull] main from quarkusio:main#780
pull[bot] merged 5 commits intobschuhmann:mainfrom
quarkusio:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Mar 25, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

geoand and others added 5 commits March 23, 2026 10:58
We went back and forth on this and it has caused all sorts of issues,
the latest having been fixed by:
#53187
that I just reverted.

I think we need to refactor this and have a very clear and consistent
semantic, which should now be the case.

The reason why an unclear semantic was detrimental is that ZipFileSystem
and Commons Compress have different behavior:
- ZipFileSystem only allows one entry per name and will fail if another
  is added
- Commons Compress will happily add several entries with the same name
  to the Zip file (which is a behavior that is allowed by the format)

When I switched from ZipFileSystem to Commons Compress, I didn't see
this difference in behavior.
Having a clear semantic should avoid all issues.

In passing, make ZipFileSystemArchiveCreator consistent with
ParallelCommonsCompressArchiveCreator, as it derived a bit.
It's not used anymore but who knows...
…n-test

Use new QuarkusExtensionTest everywhere
Refactor jar building to a first-write-wins approach
@pull pull Bot locked and limited conversation to collaborators Mar 25, 2026
@pull pull Bot added the ⤵️ pull label Mar 25, 2026
@pull pull Bot merged commit 85e6784 into bschuhmann:main Mar 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants