Incorrectly interpreting `skip: True # [not py35]` line

When using conda smithy rerender on a recipe that contains this line (skip: True # [not py35]), it is removing appveyor.yml and .travis.yml, as if it had to skip all platforms.

[tadeu]

In your case, @tadeu, I think conda-smithy, conda-build-all maybe even conda-build should complain that a py selector is being used when python is not a requirement. It results in a strange skip platform case that is not obvious initially. I don’t know if that is enough of a hint, but it would at least get one to poke around in the right places. Not sure if this warning should come out of the linter or conda-smithy directly. Would be happy to discuss all of this with you and others in a new issue.

@jakirkham I agree, thanks for looking into this. The not py35 is there just for ensuring it will not build more than one package with the same name, but I agree it seems weird if there’d no requirement for python.

[tadeu]

Interesting. Also what happened to the build/features in that commit @tadeu ?

@jakirkham I’ve removed it, since doxygen doesn’t seem to need a runtime (it is copied from the binary distribution, which doesn’t include any runtime. I’ve confirmed it with dependency walker).