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.
Description
This PR aims to address the following:
Fixes #(issue)
#34
Type of change
Feature/Issue validation/testing
Dockerfile updates
To support compiling for python version after 3.6, an additional
-fPIC
position independent code flag was required in compiling python for themanylinux2014
image. A pull request is currently open (pypa/manylinux#1258) to merge this flag change into the manylinux repo. For testing, themanylinux2014
was compiled on my local machine, followed by adocker build .
, which would then use the localmanylinux2014
image. The defaultDockerfile
now compiles amanylinux
compatible wheel for python 3.6-3.10.cibuildwheel progress
A cibuildwheel action has been created to build Labgraph wheels for Mac, Windows, and Linux
DYLD_LIBRARY_PATH
(https://cibuildwheel.readthedocs.io/en/stable/faq/#macos-passing-dyld_library_path-to-delocate).Dockerfile.Windows
which is able to successfully compile labgraph in a container, and it sets the environment variables with the following:RUN C:\Program^ Files^ ^(x86^)\Microsoft^ Visual^ Studio\2019\BuildTools\VC\Auxiliary\Build\vcvars64.bat && python setup_py36.py build
Though I call
vcvars64.bat
inscripts/prepare_build_for_windows.bat
, the build job fails to set the environment correctly. You may be able to make use ofCIBW_ENVIRONMENT
(https://cibuildwheel.readthedocs.io/en/stable/options/#environment) to inject the correct variables into the build environment, though I have failed to find a way to execute the command and set the environment variables yet.