Skip to content

Added install-capi.sh#553

Open
AkshatRai07 wants to merge 6 commits into
masterfrom
capi-installer
Open

Added install-capi.sh#553
AkshatRai07 wants to merge 6 commits into
masterfrom
capi-installer

Conversation

@AkshatRai07

Copy link
Copy Markdown
Collaborator

A step for #519
In this PR we,

  • Add install-capi.sh script to install the ekore_capi releases.

Copilot AI review requested due to automatic review settings July 21, 2026 17:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a POSIX sh installer script for the ekore_capi release artifacts, intended to simplify installing the C/C++ interface deliverables discussed in #519.

Changes:

  • Introduces crates/ekore_capi/install-capi.sh with --version, --prefix, and --target argument parsing plus platform-based target autodetection.
  • Downloads and extracts the appropriate ekore_capi-<version>-<target>.tar.gz release artifact and patches the installed ekore_capi.pc.
  • Checks for pkg-config/pkgconf and prints guidance/warnings about PKG_CONFIG_PATH.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread crates/ekore_capi/install-capi.sh Outdated
Comment thread crates/ekore_capi/install-capi.sh Outdated
Comment thread crates/ekore_capi/install-capi.sh
Comment thread crates/ekore_capi/install-capi.sh
Comment thread crates/ekore_capi/install-capi.sh
Comment thread crates/ekore_capi/install-capi.sh
Comment thread crates/ekore_capi/install-capi.sh Outdated
@AkshatRai07

AkshatRai07 commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator Author

So we would need to release to actually see if it is working or not, otherwise it is working till this step:

$ ./crates/ekore_capi/install-capi.sh --prefix ~/.local
prefix:  '/home/akshatrai/.local'
target:  'linux-x86_64'
version: 'v0.15.4'
URL:     'https://github.com/NNPDF/eko/releases/download/v0.15.4/ekore_capi-v0.15.4-linux-x86_64.tar.gz'
curl: (22) The requested URL returned error: 404

gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now

Second, this install-capi.sh will be pushed to crates.io, you can check by running cargo package --manifest-path crates/ekore_capi/Cargo.toml --list --allow-dirty, do we need that or not?

@felixhekhorn

Copy link
Copy Markdown
Collaborator

Second, this install-capi.sh will be pushed to crates.io, you can check by running cargo package --manifest-path crates/ekore_capi/Cargo.toml --list --allow-dirty, do we need that or not?

we don't need it, I'd say, so we can drop it - but it would be only a single (random) text file, so no big harm I would say ...

@felixhekhorn

Copy link
Copy Markdown
Collaborator

@cschwan @Radonirinaunimi you may want to port some of the fixes from here to PineAPPL 😇

@felixhekhorn felixhekhorn added enhancement New feature or request rust Rust extension related gsoc26 labels Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request gsoc26 rust Rust extension related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants