Skip to content

Add appveyor support #2

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 2 commits into from
Sep 2, 2015
Merged

Conversation

nicoddemus
Copy link
Member

No description provided.

@nicoddemus
Copy link
Member Author

Any guess as to why the failures in py26 and py34-pytest26? 😟

@RonnyPfannschmidt
Copy link
Member

Yes, its a ordering issue in the internal statemachine

Please install setuptools-scm with tox

@nicoddemus
Copy link
Member Author

Hmmm I see it is already mentioned in the deps section in the tox.ini file:

[testenv]
changedir=testing
deps =
    pycmd
    setuptools_scm # to avoid .eggs
    pytest24: pytest~=2.4.0
    pytest25: pytest~=2.5.0

    pytest26: pytest~=2.6.1
    pytest27: pytest~=2.7.2
    pexpect: pexpect

@RonnyPfannschmidt
Copy link
Member

problem is that tox runs setup.py in the host env -so you need both tox and setuptools_scm

@nicoddemus
Copy link
Member Author

problem is that tox runs setup.py in the host env -so you need both tox and setuptools_scm

Got it, thanks! 😄

Let's see how that fares.

@nicoddemus
Copy link
Member Author

Hmmm same error. I will try to reproduce this at home, but meanwhile do you have any other tips?

@RonnyPfannschmidt
Copy link
Member

thats a state ordering bug - its hard to trigger on normal fast computers, but seems quite noticable on the ci systems, the pr is good

RonnyPfannschmidt added a commit that referenced this pull request Sep 2, 2015
@RonnyPfannschmidt RonnyPfannschmidt merged commit 14f39a7 into pytest-dev:master Sep 2, 2015
@nicoddemus nicoddemus deleted the appveyor branch September 2, 2015 13:43
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.

2 participants