search

pcdshub / solid-attenuator

LCLS Solid Attenuator System IOC
https://pcdshub.github.io/solid-attenuator
Other
1 stars 4 forks source link

CI/BLD: update #83

Closed klauer closed 1 year ago

klauer commented 1 year ago

Description

Update CI settings and conda recipe

Motivation and Context

~Tests are failing on CI and I'm not sure why (see #82 )~ Let's update the conda recipe and tweak the old Travis config ~to see if we can't get to the bottom of it~

The issue

OK, I had forgotten about this mamba bug: In mamba create ... --file requirements.txt requirements.txt is not allowed to have blank lines. It does support comments, though. This C++ code is at the root of it (either the first or the second throw): https://github.com/mamba-org/mamba/blob/f058d3cfc91d8cd4623c26eb09767814210bb3cd/libmamba/src/core/match_spec.cpp#L203 I'm not sure why this is an error condition and not just an "ignore blank dependencies" scenario.

klauer commented 1 year ago

Does conda do this with requrements?

Looks like you already figured it out - but we had this working on conda prior to the switch to mamba. I haven't a clue why their parsing differs...