openzl 0.1.0 (new formula)#260213
Conversation
OpenZL is a format-aware compression framework that delivers high compression ratios while preserving high speed. It's designed for engineers dealing with large quantities of specialized datasets. - Version: 0.1.0 (Latest stable release) - License: BSD-3-Clause - Build system: CMake - Provides: CLI tool (zli), library, and headers for development Closes #nnn (replace with issue number if applicable) 🤖 Generated with Claude Code
|
Caution Failed to change base branch to |
|
AI generated PR totally break our PR guideline. :( |
|
OpenZL is not yet support system installed |
botantony
left a comment
There was a problem hiding this comment.
- Rebase the branch (your branch is almost 100,000 commits behind)
- Formula definition should be in
Formula/o/openzl.rb, not inFormula/openzl.rb. Please runbrew style openzlandbrew audit openzl - Don't delete our PR template:
- Have you followed the guidelines for contributing?
- Have you ensured that your commits follow the commit style guide?
- Have you checked that there aren't other open pull requests for the same formula update/change?
- Have you built your formula locally with
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where<formula>is the name of the formula you're submitting?- Is your test running fine
brew test <formula>, where<formula>is the name of the formula you're submitting?- Does your build pass
brew audit --strict <formula>(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it passbrew audit --new <formula>?
- Ensure you follow the commit style: https://docs.brew.sh/Formula-Cookbook#commit
- Address all CI failures
There was a problem hiding this comment.
| "-DOPENZL_BUILD_TESTS=OFF", | |
| "-DOPENZL_BUILD_BENCHMARKS=OFF", | |
| "-DOPENZL_BUILD_CLI=ON", | |
| "-DOPENZL_BUILD_TOOLS=ON", | |
| "-DOPENZL_BUILD_EXAMPLES=OFF", | |
| "-DOPENZL_INSTALL=ON", | |
| "-DOPENZL_CPP_INSTALL=ON", | |
| "-DZSTD_BUILD_PROGRAMS=OFF", | |
| "-DZSTD_BUILD_CONTRIB=OFF", | |
| "-DZSTD_BUILD_TESTS=OFF", | |
| *std_cmake_args |
There was a problem hiding this comment.
| system "cmake", "--build", "build" | |
| system "cmake", "--install", "build" |
There was a problem hiding this comment.
| assert_match version.to_s, shell_output("#{bin}/zli --version") |
Homebrew requires a simple test that checks if a program can do basic stuff correctly. From Formula Cookbook:
We want tests that don’t require any user input and test the basic functionality of the application. For example
foo build-foo input.foois a good test and (despite their widespread use)foo --versionandfoo --helpare bad tests. However, a bad test is better than no test at all.
There was a problem hiding this comment.
| bin.install "build/cli/zli" |
There was a problem hiding this comment.
It doesn't use brewed zstd. Either remove the dependency or (even better) try to find a way to dynamically link openzl with brewed zstd
|
This AI slop is a waste of maintainer time. |
Summary
This pull request adds a Homebrew formula for OpenZL, a format-aware compression framework that delivers high compression ratios while preserving high speed.
OpenZL is designed for engineers dealing with large quantities of specialized datasets (such as AI workloads) and require high speed for their processing pipelines.
Details
Installed Components
zlicommand for compression/decompressionlibopenzl.astatic library for developmentTesting
The formula has been:
Usage Examples
Additional Resources
🤖 Generated with Claude Code