search

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
291 stars 179 forks source link

Update error message on version mismatch #2854

Closed Lestropie closed 7 months ago

Lestropie commented 7 months ago

Closes #2822.

I think what's happening is versions of these files generated by the old build script are getting grabbed by a cmake glob rather than the versions of those files that cmake is itself generating in the build directory. Rather than do anything too clever, I propose to just update the error message so that if any fellow dullards manage to run into the same problem, they'll know what to do.

Pretty sure I tried just deleting those files and re-building, and the compiler failed reporting a missing file; so starting with a clean build directory might be required.

github-actions[bot] commented 7 months ago

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