Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Fix ray rpath errors #246

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix ray rpath errors #246

wants to merge 4 commits into from

Conversation

wdirons
Copy link
Member

@wdirons wdirons commented Apr 29, 2020

Bazel creates the build artifacts without the user having write access to the file,
when conda tries to modify the rpath it gets a permission denied error. To fix this
problem we do a chmod -R u+w $PREFIX as the last steps of the build.sh script.

To solve the issue of python_abi being added as a dependency, had to remove conda-forge
from the config files. Using python from conda-forge causes the python_abi dependency to
get added. Reordered the channel sources to the order we want.

Using the common build process, removed the build_scripts in ray-feedstock.

…s to the file,

when conda tries to modify the rpath it gets a permission denied error. To fix this
problem we do a chmod -R u+w $PREFIX as the last steps of the build.sh script.

To solve the issue of python_abi being added as a dependency, had to remove conda-forge
from  the config files. Using python from conda-forge causes the python_abi dependency to
get added. Reordered the channel sources to the order we want.

Using the commaon build process, removed the build_scripts in ray-feedstock.
Copy link
Contributor

@jayfurmanek jayfurmanek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Glad this is finally working right!

@den-run-ai
Copy link

@wdirons @jayfurmanek any idea how to resolve this problem with Ray installation?

#272

@wdirons
Copy link
Member Author

wdirons commented Oct 9, 2020

@denfromufa , I'm no longer working on Power, but if @jayfurmanek can resolve the Travis build timeout problem so it builds then this PR can be merged to fix in the error.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants