Skip to content
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

Remove build dependencies from environment.yml #30

Merged
merged 1 commit into from Feb 23, 2018
Merged

Remove build dependencies from environment.yml #30

merged 1 commit into from Feb 23, 2018

Conversation

bourque
Copy link
Collaborator

@bourque bourque commented Feb 23, 2018

This PR removes several things from the environment.yml file:

  1. The prefix that was specific to the admin account
  2. Specific builds for each dependency that was specific to an operating system
  3. Kept only top-level dependencies as to avoid OS-channel specific dependencies.

I've tested this file on both OS X and Linux with conda 4.3.30 and it seemingly works fine.

…s so that this environment builds on both linux and mac osx
@bourque
Copy link
Collaborator Author

bourque commented Feb 23, 2018

@SaraOgaz Could you look this over and make sure i'm not overlooking anything here?

@SaOgaz
Copy link
Collaborator

SaOgaz commented Feb 23, 2018

Didn't see anything off hand. Guess we'll find out later if we're missing anything :D

@bourque
Copy link
Collaborator Author

bourque commented Feb 23, 2018

@SaraOgaz Great, thanks!

@bourque bourque merged commit 9432048 into spacetelescope:master Feb 23, 2018
@bourque bourque deleted the environment-file-remove-builds branch February 23, 2018 21:34
bourque added a commit to bourque/jwql that referenced this pull request Apr 2, 2019
…move-builds

Remove build dependencies from environment.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants