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

Refine yt version selection #3

Merged
merged 5 commits into from
Jan 11, 2017

Conversation

astrofrog
Copy link
Contributor

The bug was specifically in 3.3.0 and 3.3.1 but works with >= 3.3.2

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jakirkham
Copy link
Member

This needs a re-render with conda-smithy version 1.7.0.

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

  • Selectors are suggested to take a <two spaces>#<one space>[<expression>] form.

…hinx sub-package is only used to build the docs, so is not critical.
@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jakirkham
Copy link
Member

Is it just that we need sphinx? We can re-render that for Python 3.6.

@astrofrog
Copy link
Contributor Author

@jakirkham - no, this was because we bundle numpydoc and this version had small Python 3.6 incompatibilities. The only solution for this release is to remove the sub-packages (which aren't really public API, just used when building the docs) and the next release of Hyperion will support 3.6 properly (I'm removing the bundled numpydoc).

@astrofrog
Copy link
Contributor Author

@jakirkham - however, this is going to fail because yt for Python 3.6 didn't get uploaded for Linux: conda-forge/yt-feedstock#16 - any ideas on that issue?

@astrofrog
Copy link
Contributor Author

Strangely the build for CircleCi here ends with:

TEST START: hyperion-0.9.8-np110py36_2
Deleting work directory, /feedstock_root/build_artefacts/recipe_root_1483868078529/work/Hyperion-0.9.8
...WARNING conda_build.build:create_env(711): failed to create env, retrying.  exception was: The following specifications were found to be in conflict:
  - hyperion 0.9.8 np110py36_2
  - python 3.6*
Use "conda info <package>" to see the dependencies for each package.
...WARNING conda_build.build:create_env(711): failed to create env, retrying.  exception was: The following specifications were found to be in conflict:
  - hyperion 0.9.8 np110py36_2
  - python 3.6*
Use "conda info <package>" to see the dependencies for each package.
...WARNING conda_build.build:create_env(711): failed to create env, retrying.  exception was: The following specifications were found to be in conflict:
  - hyperion 0.9.8 np110py36_2
  - python 3.6*
Use "conda info <package>" to see the dependencies for each package.
...ERROR conda_build.build:create_env(715): Failed to create env, max retries exceeded.
Can't build /recipe_root due to unsatisfiable dependencies:
The following specifications were found to be in conflict:
  - hyperion 0.9.8 np110py36_2
  - python 3.6*
Use "conda info <package>" to see the dependencies for each package.

without further explanation. But I'm pretty sure it's because yt (runtime dependency) isn't available for Python 3.6 on Linux.

@jakirkham
Copy link
Member

jakirkham commented Jan 8, 2017

You're probably right in assuming yt. ATM conda is not great about explaining unsatisfied dependencies, but looking for dependencies missing Python 3.6 packages is a good first step.

Commented on the aforementioned issue. We can keep that discussion there.

Other thing you can try is skipping Python 3.6 in the interim while things get sorted. Just add skip: true [py==36] to do this. May need to combine using or with other selectors (e.g. Windows).

@astrofrog astrofrog merged commit adcf5b8 into conda-forge:master Jan 11, 2017
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.

None yet

3 participants