MRtrix3 / mrtrix3

MRtrix3 provides a set of tools to perform various advanced diffusion MRI analyses, including constrained spherical deconvolution (CSD), probabilistic tractography, track-density imaging, and apparent fibre density
http://www.mrtrix.org
Mozilla Public License 2.0
295 stars 182 forks source link

Add workflow to create tarball for Linux #2807

Closed daljit46 closed 9 months ago

daljit46 commented 9 months ago

This PR adds a new GitHub Actions workflow to generate self-contained tarballs for Linux releases. To achieve this, mrtrix3 is built on Ubuntu 20.04 and dependencies are packaged with the help of linuxdeploy. In theory, this will allow users to run MRtrix3 on any distro with a version of glibc >= 2.31 (users may still need to install essential packages such as OpenGL).

I've tested this on fresh installs of (using Gnome Boxes):

If there are distros that you think should be tested, please let me know.

There are still some things to work out:

We already have these scripts in our codebase, but they may require modifications to work as expected.

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

daljit46 commented 9 months ago

The build workflow has now been simplified by adding a new bash script package-linux-tarball.sh. This allows an easier testing of the workflow on a local Ubuntu system.

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 9 months ago

clang-tidy review says "All clean, LGTM! :+1:"

daljit46 commented 9 months ago

This can now be merged. May need to do some changes to the packagiing script after dealing with #2827, but can handle that later.