[GitHub] [qpid-proton] astitcher edited a comment on issue #200: PROTON-2126 Fix Appveyor CI build

[GitBox <gi...@apache.org>]

astitcher edited a comment on issue #200: PROTON-2126 Fix Appveyor CI build
URL: https://github.com/apache/qpid-proton/pull/200#issuecomment-549889796
 
 
   For reference I choose python 3.5 because it is present on every appveyor image and so avoided messing with a matrix of different versions.
   I left the --user flag not because it's needed here, but because it allows the build logic to be common across different CI platforms even if the actual 'code' is different - ie it will work on a non privileged account as well on Windows or Unix.
   Also note that the python binding (rather than the python build code) is tested using tox (if present) and this will test py26, py27, py35, py36, py37 & py38 (I think that's the correct list) if it can find them regardless of the python used for the build.
   I removed the VS9 install part because once you move to python3 for the build it's not needed (and doesn't fail the tox test) and is not important for this kind of CI run IMO.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org