Skip to content

build: consume immutable FFTW dependency bundles - #3

Closed
galenlynch wants to merge 1 commit into
package-fftw-dependenciesfrom
consume-fftw-dependency-bundles
Closed

galenlynch wants to merge 1 commit into
package-fftw-dependenciesfrom
consume-fftw-dependency-bundles

Conversation

@galenlynch

Copy link
Copy Markdown
Collaborator

This is the second of two packaging PRs. It changes normal OpenEphysFFTW builds to consume the immutable FFTW 3.3.11 release produced by the first PR, then removes the superseded binaries from libs/ without rewriting Git history.

Changes

  • Select the platform archive and checksum sidecar in CMake.
  • Download both files from the immutable dependency release.
  • Verify the archive against its published SHA-256 checksum before extraction.
  • Provide an offline cache/root override for local and restricted builds.
  • Preserve the current install layout and runtime-library handling.
  • Remove the old Linux, Windows, and macOS library blobs from the working tree.

The repository does not copy generated archive hashes into source. The release's immutable checksum sidecars remain the source of truth, while CMake performs the verification locally.

Merge prerequisite

Do not merge this PR until:

  1. The packaging PR is merged.
  2. Release immutability is enabled.
  3. fftw-dependencies-3.3.11-1 has been published successfully.
  4. All release assets and checksum sidecars are available.

Verification

CI must configure, build, and test OpenEphysFFTW on Linux, Windows, and macOS using the published bundles. An offline-cache test should also prove that builds do not require network access when the dependency archive is supplied locally.

@galenlynch
galenlynch changed the base branch from main to package-fftw-dependencies September 8, 2026 22:56
@galenlynch
galenlynch force-pushed the consume-fftw-dependency-bundles branch from 243406a to 76bcf0a Compare September 8, 2026 23:06
Normal builds still use the divergent FFTW binaries committed to libs/, so the verified dependency release does not yet govern the libraries that OpenEphysFFTW links and redistributes.

Resolve the platform bundle from the versioned release during configuration, validate its checksum sidecar, and retain extracted-root and local-archive overrides for offline builds. Package both precision runtimes with their license and provenance, then remove the superseded binary blobs.
@galenlynch
galenlynch force-pushed the consume-fftw-dependency-bundles branch from 76bcf0a to 1efb034 Compare September 8, 2026 23:17
@galenlynch

Copy link
Copy Markdown
Collaborator Author

#2 no longer requires this PR

@galenlynch galenlynch closed this Sep 8, 2026
@anjaldoshi
anjaldoshi deleted the consume-fftw-dependency-bundles branch September 9, 2026 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant