Skip to content

[CMake][Release] Use the TXZ cpack generator for binaries #90138

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

Merged
merged 4 commits into from
Jun 7, 2024

Conversation

tstellar
Copy link
Collaborator

No description provided.

@tstellar tstellar requested a review from tru April 25, 2024 22:33
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Apr 25, 2024
@llvmbot
Copy link
Member

llvmbot commented Apr 25, 2024

@llvm/pr-subscribers-github-workflow

@llvm/pr-subscribers-clang

Author: Tom Stellard (tstellar)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/90138.diff

1 Files Affected:

  • (modified) clang/cmake/caches/Release.cmake (+1)
diff --git a/clang/cmake/caches/Release.cmake b/clang/cmake/caches/Release.cmake
index c164d5497275f3..aa7e6f99e94d52 100644
--- a/clang/cmake/caches/Release.cmake
+++ b/clang/cmake/caches/Release.cmake
@@ -90,4 +90,5 @@ endif()
 # Final Stage Config (stage2)
 set_final_stage_var(LLVM_ENABLE_RUNTIMES "${LLVM_RELEASE_ENABLE_RUNTIMES}" STRING)
 set_final_stage_var(LLVM_ENABLE_PROJECTS "${LLVM_RELEASE_ENABLE_PROJECTS}" STRING)
+set_final_stage_var(CPACK_GENERATOR "TGZ" STRING)
 

@tru
Copy link
Collaborator

tru commented Apr 26, 2024

If this is for permanent storage I suggest we use TXZ instead, since it has better compression. You should also set CPACK_ARCHIVE_THREADS to something higher than the default (1). Since xz really benefits from this.

@tstellar tstellar changed the title [CMake][Release] Use the TGZ cpack generator for binaries [CMake][Release] Use the TXZ cpack generator for binaries May 17, 2024
@tru
Copy link
Collaborator

tru commented May 20, 2024

Did you consider the parallel setting as well?

@tstellar
Copy link
Collaborator Author

Did you consider the parallel setting as well?

Forgot about that part, I will make that change too.

@tstellar tstellar added this to the LLVM 19.X Release milestone May 22, 2024
@tstellar tstellar merged commit 0d1b367 into llvm:main Jun 7, 2024
8 checks passed
@HerrCai0907 HerrCai0907 mentioned this pull request Jun 13, 2024
tstellar added a commit to tstellar/llvm-project that referenced this pull request Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category github:workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants