-
Notifications
You must be signed in to change notification settings - Fork 92
Better conda #512
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
Better conda #512
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
c1cd48f
Use conda_build_config to build for multiple python/numpy.
abergeron 32d498d
Fix the numpy dependency.
abergeron d19813f
Make appveyor build conda packages.
abergeron 1d82d00
Add compiler requirement.
abergeron 2d608df
Attempt at fixing the compiler issue
abergeron 388896d
Fix bld.bat
abergeron 6ed1218
Maybe working windows build?
abergeron 1b2aa4c
Add versioneer for better versions.
abergeron ca7d1ba
Use versioneer to reduce the version updating madness when doing a re…
abergeron 97fac84
Go back to build matrix for appveyor.
abergeron c815a9a
Set the version for conda packages automatically.
abergeron 3e7f386
Beginings of upload stuff + fix for build errors.
abergeron 438aead
Fix parse problem.
abergeron cf3361b
Set the MSVC paths manually since conda doesn't look for "VC++ for py…
abergeron b909a81
Fix pygpu version.
abergeron 0f363e1
Fix syntax?
abergeron 61785cb
Fix cmake not picking up the right compiler.
abergeron 9446117
Fix brokeness with vs2008 through a conda-forge trick.
abergeron 728ea1d
Remove unecessary stuff.
abergeron 25d623f
Try to save the built packages.
abergeron File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,45 +1,47 @@ | ||
| version: '0.7.0.{build}' | ||
| version: '1.0.{build}' # This number doesn't matter | ||
|
|
||
| pull_requests: | ||
| do_not_increment_build_number: true | ||
|
|
||
| image: Visual Studio 2015 | ||
|
|
||
| init: | ||
| - git config --global core.autocrlf input | ||
| - cmd: cmake --version | ||
| - cmd: msbuild /version | ||
|
|
||
| platform: | ||
| - x64 | ||
|
|
||
| image: Visual Studio 2015 | ||
|
|
||
| clone_folder: C:\projects\libgpuarray | ||
|
|
||
| configuration: | ||
| - Release | ||
| - Debug | ||
|
|
||
| environment: | ||
| matrix: | ||
| - PYTHON: "C:\\Python27" | ||
| VS_PATH: "C:\\Users\\appveyor\\AppData\\Local\\Programs\\Common\\Microsoft\\Visual C++ for Python\\9.0" | ||
| - PYTHON: "C:\\Python35" | ||
| VS_PATH: "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC" | ||
| - CONDA_LOC: "C:\\Miniconda-x64" | ||
| - CONDA_LOC: "C:\\Miniconda35-x64" | ||
|
|
||
| install: | ||
| - python -m pip install mako cython | ||
|
|
||
| build_script: | ||
| - echo "Python:" "%PYTHON%" | ||
| - echo "Config:" "%CONFIGURATION%" | ||
| - echo "VS path:" "%VS_PATH%" | ||
| - cd "%VS_PATH%" | ||
| - vcvarsall x64 | ||
| - set | ||
| - cd C:\projects\libgpuarray | ||
| - md %CONFIGURATION% | ||
| - cd %CONFIGURATION% | ||
| - cmake .. -DCMAKE_BUILD_TYPE=%CONFIGURATION% -G "NMake Makefiles" | ||
| - cmake --build . --config %CONFIGURATION% | ||
|
|
||
| build: script | ||
| # This breaks conda-build because of git | ||
| - cmd: rmdir C:\cygwin /s /q | ||
| - cmd: call %CONDA_LOC%\Scripts\activate.bat | ||
| - cmd: set PYTHONUNBUFFERED=1 | ||
| - cmd: conda install -n root --yes conda conda-env conda-build anaconda-client | ||
| # We borrow a trick from conda-forge to fix the VS2008 compiler | ||
| - cmd: conda config --append channels conda-forge | ||
| - cmd: conda install --yes vs2008_express_vc_python_patch | ||
| - cmd: call setup_x64 | ||
|
|
||
| build: off | ||
|
|
||
| test_script: | ||
| - cmd: for /f "tokens=*" %%i in ('python -c "import versioneer; print(versioneer.get_version())"') do set GPUARRAY_VERSION=%%i | ||
| - cmd: conda build conda | ||
| - cmd: mkdir pkgs | ||
| - cmd: xcopy "%CONDA_LOC%"\conda-bld\win-64\pygpu* pkgs\ /Y | ||
| - cmd: xcopy "%CONDA_LOC%"\conda-bld\win-64\libgpuarray* pkgs\ /Y | ||
|
|
||
| artifacts: | ||
| - path: pkgs/* | ||
| name: "Conda Packages" | ||
|
|
||
| #deploy: | ||
| # on: | ||
| # appveyor_repo_tag: true |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| pygpu/_version.py export-subst |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| include versioneer.py | ||
| include pygpu/_version.py |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,10 @@ | ||
| cmake -G"NMake Makefiles" ^ | ||
| -DCMAKE_BUILD_TYPE=Release ^ | ||
| cmake -G "%CMAKE_GENERATOR%" ^ | ||
| -DCMAKE_PREFIX_PATH="%LIBRARY_PREFIX%" ^ | ||
| -DCMAKE_INSTALL_PREFIX="%LIBRARY_PREFIX%" ^ | ||
| -DCMAKE_C_FLAGS="-I%LIBRARY_PREFIX%\include" ^ | ||
| "%SRC_DIR%" | ||
| if errorlevel 1 exit 1 | ||
| cmake --build . --config Release --target ALL_BUILD | ||
| cmake --build . --config Release --target install | ||
| if errorlevel 1 exit 1 | ||
| cmake --build . --config Release --target install | ||
| if errorlevel 1 exit 1 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it mean it will only support numpy 1.11? Why not bump that to 1.13 to support the more recent version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it was discovered that numpy is forward-compatible. So it should work with numpy 1.12 and 1.13 too (and 1.14 whenever it comes out).