Add libs folder to bundled static libs#16655
Add libs folder to bundled static libs#16655Mytherin merged 1 commit intoduckdb:v1.2-histrionicusfrom
Conversation
|
I am not sure this is correct, the |
|
@carlopi, we must keep the https://github.com/duckdb/duckdb/actions/runs/13857853364?pr=16655, the artifacts look good, no? |
carlopi
left a comment
There was a problem hiding this comment.
Looks right, thanks!!
Note that this changes the artifacts from static-lib-* to static-libs-*, but I do agree this is better given the artifact are differently packaged and it allow to more easily spot places to update (we are aware of this being used only in go client).
I think this is OK (and better) in v1.2-histrionicus, green from me.
|
Thanks! |
Avoid UMA (duckdb/duckdb#16641) Add libs folder to bundled static libs (duckdb/duckdb#16655)
For some OS + architecture combinations, the bundled static libraries exceed 100MB, which can be inconvenient.
This PR adds a gather-libs target that collects the individual libraries (pre-bundling) and puts them in a libs folder. When publishing the existing artifact, it also zips that additional libs folder.
Follow-up to #16654.