Jonathan Hanks (0f07691d) at 28 Nov 01:13
Version bump for a rebuild
Jonathan Hanks (0981eb4e) at 28 Nov 01:12
Version bump for a rebuild
@jonathan-hanks Is there a reason the path to curl
needs to be hardcoded in the remote medm patch?
Running medm_llo
in a continuumio/miniconda3
docker container with the CDS conda env installed leads to the following error:
sh: 1: /usr/bin/curl: not found
Here the CDS conda env does provide /opt/conda/envs/cds/bin/curl
. But since the /usr/bin
path is specified, there's a dependency on the underlying OS, which this particular container does not satisfy.
Camilla found that some arguments aren't being passed between medm screens.
Apparently argument lists, which are comma separated between files, are turned into space separated lists, but if a variable is empty or undefined, the next variable name gets put in as a value, and it's skipped. That next variable is therefore not passed to the new medm screen.
Erik von Reis (f8cc29bb) at 30 Nov 23:03
empty value strings in Display argument lists were missed by strtok(). This could cause loss of the next argument. Replacing strtok() with strsep() fixes the problem.
Jonathan Hanks (5b87cb6e) at 30 Nov 23:02
Merge branch 'fix_empty_values' into 'master'
... and 1 more commit
empty value strings in Display argument lists were missed by strtok(). This could cause loss of the next argument. Replacing strtok() with strsep() fixes the problem.
Erik von Reis (f8cc29bb) at 25 Nov 01:17
Add patch to fix empty values in arg strings
Camilla found that some arguments aren't being passed between medm screens.
Apparently argument lists, which are comma separated between files, are turned into space separated lists, but if a variable is empty or undefined, the next variable name gets put in as a value, and it's skipped. That next variable is therefore not passed to the new medm screen.