Skip to content

Fix invocation of trial on Windows with Twisted 16.4.0#2373

Merged
tardyp merged 1 commit intobuildbot:masterfrom
rodrigc:appveyor
Aug 28, 2016
Merged

Fix invocation of trial on Windows with Twisted 16.4.0#2373
tardyp merged 1 commit intobuildbot:masterfrom
rodrigc:appveyor

Conversation

@rodrigc
Copy link
Copy Markdown
Contributor

@rodrigc rodrigc commented Aug 28, 2016

In Twisted 16.4.0, trial was converted to a Python console
script. It must be invoked as "trial" or "python -m twisted.trial".
%PYTHON%\Scripts\trial.py is gone in Twisted 16.4.0.

@mention-bot
Copy link
Copy Markdown

@rodrigc, thanks for your PR! By analyzing the annotation information on this pull request, we identified @tomprince, @rutsky and @anish to be potential reviewers

%PYTHON%\\Scripts\trial.py is gone in Twisted 16.4.0.
In Twisted 16.4.0, trial was converted to a Python console
script.  It must be invoked as "trial" or "python -m twisted.trial".
@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 28, 2016

Current coverage is 86.65% (diff: 100%)

Merging #2373 into master will not change coverage

@@             master      #2373   diff @@
==========================================
  Files           294        294          
  Lines         30549      30549          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          26471      26471          
  Misses         4078       4078          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 11c980d...c6d7ecf

@tardyp tardyp merged commit a01308a into buildbot:master Aug 28, 2016
@gsemet
Copy link
Copy Markdown

gsemet commented Aug 28, 2016

someone knows why "python -m trial" is better than just trial, just for my information?

@rodrigc
Copy link
Copy Markdown
Contributor Author

rodrigc commented Aug 29, 2016

"python -m twisted.trial" is equivalent, it is not better or worse.
In Twisted 16.4.0, pip install twisted on Windows will install a binary trial.exe
in the PATH. I needed to quickly submit something that worked, because with the new Twisted,
the appveyor build suddenly started breaking, and I am submitting a lot of patches.

@gsemet
Copy link
Copy Markdown

gsemet commented Aug 29, 2016

ok thx :)

@rodrigc rodrigc deleted the appveyor branch August 29, 2016 08:08
@tardyp tardyp mentioned this pull request Sep 6, 2016
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.

5 participants