search

aces / Loris-MRI

The set of scripts that preprocess and insert MRI data into the database.
10 stars 51 forks source link

Fix Perl dependencies (& CI) #1205

Closed maximemulder closed 1 month ago

maximemulder commented 1 month ago

I noticed the LORIS-MRI Docker image was no longer building in CI for #1191. After some investigation, it appears to be because of an update in Module::Pluggable (the first in 9 years !), which some of our required packages depend on (notably DateTime, but also others I think). This PR fixes the issue by pinning the version of Module::Pluggable to version 5.2.

During the investigation of this PR, I also changed our pipeline to use cpan instead of cpanm, which seems to be the recommended package installer by the Perl community. It provides both a much clearer output (no longer literal tens of thousands of lines) and is faster by approximately 5 minutes.

maximemulder commented 1 month ago

Sometimes, you gotta do what you gotta do.