Skip to content

Conversation

@downiec
Copy link
Contributor

@downiec downiec commented Oct 22, 2018

Updated meta.yaml files to make sure python2 is used in conda environment to avoid run from having python 3. Added Sterling's handy conda_build.sh. Other minor changes and cleanup.

@coveralls
Copy link

coveralls commented Oct 22, 2018

Coverage Status

Coverage remained the same at 67.677% when pulling 3fd25f7 on install_cleanup into a073da3 on master.

Copy link
Contributor

@doutriaux1 doutriaux1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@downiec please make sure we do not need space before =2 and also we need to tag vcs and vcs to some stable versions.,

@sterlingbaldwin
Copy link
Collaborator

Can anyone give a quick explanation of the meta.yaml.in and meta.yaml.official? These files cant be used by conda directly, so are they being manipulated by Travis or something? @James-Crean @doutriaux1

@doutriaux1
Copy link
Contributor

@sterlingbaldwin official is the recipe used to produce the official/stable release, just so we easily remember what we used. The ".in" one is the one we used in relation with http://github.com/cdat/conda-recipes to generate the meta.yaml for nightly using correct branch, etc... Also has capabilites for repos with "features" (only vtk-cdat at this point)

Copy link
Collaborator

@sterlingbaldwin sterlingbaldwin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the conda create line needs to install the vcdat package.

scripts/setup.sh Outdated

# Create a new one
conda create -y -n ${CONDA_ENV} -c cdat/label/nightly -c conda-forge -c cdat --file $current_dir/backend/requirements.txt
conda create -y -n ${CONDA_ENV} -c cdat/label/nightly -c conda-forge -c cdat
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should read

conda create -y -n ${CONDA_ENV} -c cdat/label/nightly -c conda-forge -c cdat vcdat

Copy link
Contributor Author

@downiec downiec Nov 1, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I think you're right. I went ahead and updated that. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants