Skip to content

CI: Fixes to conda environment names #23866

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Merged
Show file tree
Hide file tree
Changes from 16 commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
88bc2b7
change pandas to pandas-dev
charlesdong1991 Nov 22, 2018
fc88e2b
add source activate
charlesdong1991 Nov 22, 2018
fb34dcc
remove source activate
charlesdong1991 Nov 22, 2018
04e721c
clean the script
charlesdong1991 Nov 22, 2018
cd81131
debug
charlesdong1991 Nov 22, 2018
463be05
debug
charlesdong1991 Nov 22, 2018
6efdedd
add pandas-dev back at two lines
charlesdong1991 Nov 23, 2018
24bc4f6
change pandas to pandas-dev
charlesdong1991 Nov 22, 2018
28a3ebf
add source activate
charlesdong1991 Nov 22, 2018
9f56258
remove source activate
charlesdong1991 Nov 22, 2018
bf6bca2
clean the script
charlesdong1991 Nov 22, 2018
2c43b2f
debug
charlesdong1991 Nov 22, 2018
3a2a193
debug
charlesdong1991 Nov 22, 2018
bacb230
add pandas-dev back at two lines
charlesdong1991 Nov 23, 2018
393f754
test again after catching the latest master
charlesdong1991 Nov 23, 2018
840c566
Merge branch 'conda_env_name' of github.com:charlesdong1991/pandas in…
charlesdong1991 Nov 23, 2018
d969685
debugging
charlesdong1991 Nov 23, 2018
a0a978a
debugging
charlesdong1991 Nov 23, 2018
e024ba0
debugging
charlesdong1991 Nov 23, 2018
26df0d4
debugging
charlesdong1991 Nov 23, 2018
53f8eaa
debugging
charlesdong1991 Nov 23, 2018
de8c1bc
debugging
charlesdong1991 Nov 23, 2018
3b6d646
debugging
charlesdong1991 Nov 23, 2018
1d9e93a
debugging
charlesdong1991 Nov 23, 2018
e3c9fc1
debugging
charlesdong1991 Nov 23, 2018
603bd0b
debugging
charlesdong1991 Nov 23, 2018
8a37dd0
debugging
charlesdong1991 Nov 23, 2018
e59c787
debugging
charlesdong1991 Nov 23, 2018
4b0b03e
debugging
charlesdong1991 Nov 23, 2018
45e5f06
debugging
charlesdong1991 Nov 23, 2018
1c4ddfb
debugging
charlesdong1991 Nov 24, 2018
61b7f6f
debugging
charlesdong1991 Nov 24, 2018
997d680
debugging
charlesdong1991 Nov 24, 2018
49231a7
debugging
charlesdong1991 Nov 24, 2018
a0331bf
Some more fixes
datapythonista Nov 24, 2018
adea341
Merge branch 'conda_env_name_backup_backup' of github.com:charlesdong…
datapythonista Nov 24, 2018
afe3b1f
Removing unneeded activation
datapythonista Nov 24, 2018
fbde055
fixing wrong environment name in circle
datapythonista Nov 24, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ before_script:
- ci/before_script_travis.sh

script:
- source activate pandas-dev
- echo "script start"
- ci/run_build_docs.sh
- ci/script_single.sh
Expand All @@ -115,7 +116,7 @@ after_success:

after_script:
- echo "after_script start"
- source activate pandas && pushd /tmp && python -c "import pandas; pandas.show_versions();" && popd
- source activate pandas-dev && pushd /tmp && python -c "import pandas; pandas.show_versions();" && popd
- if [ -e test-data-single.xml ]; then
ci/print_skipped.py test-data-single.xml;
fi
Expand Down
2 changes: 0 additions & 2 deletions ci/build_docs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ if [ "$DOC" ]; then

echo "Will build docs"

source activate pandas

echo ###############################
echo # Log file for the doc build #
echo ###############################
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/azure-27-compat.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/azure-36-locale_slow.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/azure-37-locale.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/azure-macos-35.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
dependencies:
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/azure-windows-27.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/azure-windows-36.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/circle-36-locale.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/travis-27-locale.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/travis-27.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/travis-36-doc.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/travis-36-slow.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/travis-36.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/travis-37-numpydev.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
dependencies:
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/travis-37.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: pandas
name: pandas-dev
channels:
- defaults
- conda-forge
Expand Down
2 changes: 1 addition & 1 deletion ci/incremental/build.cmd
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
@rem https://github.com/numba/numba/blob/master/buildscripts/incremental/build.cmd
call activate %CONDA_ENV%


@rem Build numba extensions without silencing compile errors
python setup.py build_ext -q --inplace
Expand Down
1 change: 0 additions & 1 deletion ci/incremental/build.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
#!/bin/bash

source activate $CONDA_ENV

# Make sure any error below is reported as such
set -v -e
Expand Down
5 changes: 2 additions & 3 deletions ci/incremental/setup_conda_environment.cmd
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,10 @@ call deactivate
@rem Display root environment (for debugging)
conda list
@rem Clean up any left-over from a previous build
conda remove --all -q -y -n %CONDA_ENV%
conda remove -all -q -y -n pandas-dev
Copy link
Member

Choose a reason for hiding this comment

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

I think the removed - is needed

@rem Scipy, CFFI, jinja2 and IPython are optional dependencies, but exercised in the test suite
conda env create -n %CONDA_ENV% --file=ci\deps\azure-windows-%CONDA_PY%.yaml
conda env create --file=ci\deps\azure-windows-%CONDA_PY%.yaml

call activate %CONDA_ENV%
Copy link
Member

Choose a reason for hiding this comment

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

I think this, and another one deleted, need to be moved to .travis.yml before those scripts are called

Copy link
Member

Choose a reason for hiding this comment

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

same comment as above

conda list

if %errorlevel% neq 0 exit /b %errorlevel%
9 changes: 3 additions & 6 deletions ci/incremental/setup_conda_environment.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,23 +5,20 @@ set -v -e
CONDA_INSTALL="conda install -q -y"
PIP_INSTALL="pip install -q"


# Deactivate any environment
source deactivate
# Display root environment (for debugging)
conda list
# Clean up any left-over from a previous build
# (note workaround for https://github.com/conda/conda/issues/2679:
# `conda env remove` issue)
conda remove --all -q -y -n $CONDA_ENV
conda remove --all -q -y -n pandas-dev

echo
echo "[create env]"
time conda env create -q -n "${CONDA_ENV}" --file="${ENV_FILE}" || exit 1
time conda env create -q --file="${ENV_FILE}" || exit 1

# Activate first
set +v
source activate $CONDA_ENV
set -v

# remove any installed pandas package
# w/o removing anything else
Expand Down
1 change: 0 additions & 1 deletion ci/script_single.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@

echo "[script_single]"

source activate pandas

Copy link
Member

Choose a reason for hiding this comment

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

Can you remove one of the blank lines besides de source activate and leave just one

if [ -n "$LOCALE_OVERRIDE" ]; then
echo "Setting LC_ALL and LANG to $LOCALE_OVERRIDE"
Expand Down
1 change: 0 additions & 1 deletion ci/upload_coverage.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ if [ -z "$COVERAGE" ]; then
exit 0
fi

source activate pandas

echo "uploading coverage"
bash <(curl -s https://codecov.io/bash) -Z -c -F single -f /tmp/cov-single.xml
Expand Down