Pin conda-build to version 1.x

[jjhelmus]

should address conda-forge/conda-smithy#282

[conda-forge-linter]

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]

Please bump version to 3.1.2 (i.e. the patch number).

[jakirkham]

LOL. This is fun. 😒

Using Anaconda API: https://api.anaconda.org
Traceback (most recent call last):
  File "/feedstock_root/ci_support/upload_or_check_non_existence.py", line 124, in <module>
    main()
  File "/feedstock_root/ci_support/upload_or_check_non_existence.py", line 104, in main
    exists = built_distribution_already_exists(cli, meta, owner)
  File "/feedstock_root/ci_support/upload_or_check_non_existence.py", line 24, in built_distribution_already_exists
    fname = bldpkg_path(meta)
TypeError: bldpkg_path() missing 1 required positional argument: 'config'

I think we are going to have to do some pinnings in the CI scripts too. It's ugly, but tolerable for now.

[jakirkham]

# -> ::

[jjhelmus]

I think we are going to have to do some pinnings in the CI scripts too. It's ugly, but tolerable for now.

Agreed, Circle seems do-able, but how about Travis and AppVeyor?

[jakirkham]

Would just add a line after here in .travis.yml and after here in appveyor.yml.

Comments referencing this PR would be nice.

[jjhelmus]

Oh wait, this recipe doesn't use itself, so patching is not as hard as I though.

[jakirkham]

Oh wait, this recipe doesn't use itself, so patching is not as hard as I though.

Yeah, I have been waiting for that to bite us, but we are lucky thus far. Will give it some more thought. Admittedly the same kludgy things we are doing here should still work in that case. Though ideally we would bootstrap this somehow.

[jakirkham]

LGTM. Will merge when it passes, thanks for putting this together.

[jjhelmus]

No problem. I'll add the issue from conda-smithy to this repo so as a reminder to remove this kludge once the upload script is fixed.

[jakirkham]

Also opened issue ( #29 ) to discuss how we handle building bootstrapping conda-forge-build-setup.

[asmeurer]

Checks pass. Should this get merged?

[asmeurer]

And once this is merged, will I need to wait for my feedstock to be re-rendered, or will it just work?

[jakirkham]

Awesome. Thanks for the message.

Nope. Should just work as that feedstock already uses conda-forge-build-setup.