janelia-flyem / NeuTu

Software package for neuron reconstruction and visualization
Other
38 stars 13 forks source link

conda-recipe: Pin gcc to 7.5.0 #364

Closed stuarteberg closed 4 years ago

stuarteberg commented 4 years ago

This PR pins the compiler version via line in the recipe's conda_build_config.yaml.

Background:

Yesterday @tingzhao had trouble getting NeuTu to compile in our docker container due to a mysterious error:

/tmp/ccpjQ535.s: Assembler messages:
/tmp/ccpjQ535.s:3575: Error: expecting string instruction after `rep'
/tmp/ccpjQ535.s:3610: Error: expecting string instruction after `rep'
/tmp/ccpjQ535.s:3614: Error: expecting string instruction after `rep'
/tmp/ccpjQ535.s:5369: Error: expecting string instruction after `rep'
/tmp/ccpjQ535.s:5396: Error: expecting string instruction after `rep'

We determined the problem to be with a recent upgrade to binutils_impl_linux-64 in conda-forge. We fixed the issue by explicitly downgrading gcc and binutils.

But in the mean time, conda-forge has fixed the broken binutils package. We can upgrade to gcc 7.5.0 again (I tested it). While we're at it, we should also pin the version of gcc so we don't get surprised if they upgrade the default compiler.

stuarteberg commented 4 years ago

BTW, I tested these changes. I can build NeuTu in the docker container.