Repository navigation
ci: publish release archives with native Tika for Linux, macOS and Windows; rename to documents-plugin - #2
Merged
Merged
Conversation
Test on Linux, macOS and Windows, x86-64 and ARM64 (ubuntu-24.04-arm and windows-11-arm native runners), and publish matching release archives. Release builds now run the test suite before packaging.
Package and smoke-test each platform's archive (libtika_native beside the executable via @loader_path, $ORIGIN, or the exe directory), build Linux on Ubuntu 22.04, build Windows with MSVC, and port the store and home lookup to Windows. No Windows on ARM: GraalVM has no native-image for it. Pin mesh-llm-plugin to #3012's current head.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
 |
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Title check |  |
This assessment has not completed. The check will be retried on the next pre-merge check run. |
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled.
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
…are now "documents" Co-authored-by: Gandalf <474797421bc8061b82a684ffc1438abc87e1355d67aa3cc59917808655971c9f@meshllm.communities.buzz.xyz>
michaelneale
changed the title
Use temp_dir paths instead of Unix-only absolute paths, and run the /bin/sh-based Tika worker tests on Unix only.
…in tests On Windows PDFBox scans every system font on its first text extraction (~70 s for 346 fonts on a GitHub runner), then reads ~/.pdfbox.cache (~1 s). The worker test's 60 s deadline killed that first run.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Builds on #1 (i386's example-plugin workflow standard) and makes its archives actually run: the template copies only the executable, but this plugin needs
libtika_native(Tika compiled with GraalVM) beside it.What a release now ships (on a
v*tag):pdf-<version>-<target>and the stable aliaspdf-<target>forx86_64-unknown-linux-gnu,aarch64-unknown-linux-gnu(built on Ubuntu 22.04, glibc 2.35+)aarch64-apple-darwin,x86_64-apple-darwinx86_64-pc-windows-msvcNo Windows on ARM: GraalVM has no native-image for it.
Changes
scripts/package.shpackages on every OS: the library is loaded from beside the executable (@loader_path+ re-sign on macOS,$ORIGINvia patchelf on Linux, exe directory on Windows), plusplugin-manifest.json.scripts/smoke.shunpacks an archive outside the build tree and runs its--packaged-manifestand a native Tika extraction oftests/fixtures/smoke.pdf. CI and release run it on each platform.USERPROFILE, Tesseract's default install folder is searched.mesh-llm-pluginpinned to #3012's current head (355d29c8c); the old pin was from before its rebase.Validation (local, M5): release clippy clean, 19 tests pass,
package.sh+smoke.shok, andmesh-llm plugins install --archive(#3012 build, temp HOME) installs it. The other platforms are validated by this PR's CI.