[go: nahoru, domu]

Skip to content
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

Update README.md - clarification #6427

Closed
wants to merge 3 commits into from

Conversation

johndpope
Copy link
Contributor

I initially followed instructions - and got more errors.
The build all script should be first call to action for newbies.

I initially followed instructions - and got more errors. 
The build all script should be first call to action for newbies.
@tensorflow-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@johndpope johndpope closed this Dec 20, 2016
copybara-service bot pushed a commit that referenced this pull request Oct 19, 2023
…fig.py instead of using `glob()`

Imported from GitHub PR openxla/xla#6427

We know the exact location and don't need to glob() here. The issue with glob is that it can be slow, in particular when re-running the configure script after building with Bazel: the glob will process tens of GB of build directories!
Copybara import of the project:

--
d0ccab91c465871c6c87f9bcf1746cbd5da47967 by Mehdi Amini <mamini@nvidia.com>:

Fix configure.py to more directly specify the find_cuda_config.py instead of using `glob()`

We know the exact location and don't need to glob() here.
The issue with glob is that it can be slow, in particular when re-running the
configure script after building with Bazel: the glob will process tens of GB
of build directories!

Merging this change closes #6427

PiperOrigin-RevId: 575020900
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants