added the windows build for travis CI having py2 interpreter - #152
added the windows build for travis CI having py2 interpreter#152pratik0316 wants to merge 1 commit into
Conversation
Signed-off-by: Pratikrocks <pratikrocks.dey11@gmail.com>
0a47e26 to
27472ca
Compare
| - os: windows | ||
| language: shell | ||
| before_install: | ||
| - choco install python2 -x86 |
There was a problem hiding this comment.
choco install python2 -x86 installs 32 bit libs in a 32 bit system.
without -x86 it installs 64 bit libs also which causes error during configuration.
ref
| language: shell | ||
| before_install: | ||
| - choco install python2 -x86 | ||
| - ln -s /c/Python27/python.exe /c/Python27/python2.7.exe |
There was a problem hiding this comment.
We need to create a soft link pointing to /c/Python27/python.exe as during the configuration it will point to this executable file.
| from deltacode import cli | ||
|
|
||
| options = add_windows_extra_timeout(options) | ||
|
|
There was a problem hiding this comment.
Currently we do not have the support for adding the cli option --timeout, we may require it for the Appevyor windows CI as it timesout after 120 sec.
|
@MaJuRG the windows build is working. |
steven-esser
left a comment
steven-esser
left a comment
There was a problem hiding this comment.
Have you reviewed how this is done for scancode-toolkit?
@MaJuRG |
@MaJuRG sorry I couldn't get you. |
|
@MaJuRG , should I need to make any additional changes in this windows build? |
steven-esser
left a comment
steven-esser
left a comment
There was a problem hiding this comment.
I am not sure we actually want windows builds in Travis CI. We probably want them in Appveryor like scancode-toolkit does.
|
@MaJuRG I will be adding the windows build for appveyor , I am closing this PR for now as it is not required , I will re - open it whenever needed |
2 participants
added the windows build to travis CI having py2 as interpreter