8000 Install scripts/perf to share/clBLAS on non WIN32 systems by marbre · Pull Request #74 · clMathLibraries/clBLAS · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Install scripts/perf to share/clBLAS on non WIN32 systems #74

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 1 commit into from
May 12, 2015
Merged

Install scripts/perf to share/clBLAS on non WIN32 systems #74

merged 1 commit into from
May 12, 2015

Conversation

marbre
Copy link
Contributor
@marbre marbre commented Mar 23, 2015

Python scripts should not get installed to /usr/bin. Same as for clFFT clMathLibraries/clFFT#52.

@TimmyLiu
Copy link
Contributor

In my linux system (OpenSuse) that I tested, the python files were installed to ${BUILD_DIR}/package/bin directory. "client", which the python scripts will call, resides in the same directory. The architecture of the "package" also includes "lib64", which contains library binary files, "include" with header files and "samples". I am wondering why the pythons scripts were installed into /usr/bin instead.

@marbre
Copy link
Contributor Author
marbre commented Mar 26, 2015

Actually, this change is similar to clMathLibraries/clFFT@37b1f44. See the discussion in clMathLibraries/clFFT#50.

Following your description, it seems that you just compile the package, but didn't installed it. For Gentoo Linux, the change is definitely, necessary. I recently created an clBLAS ebuild for the gentoo science overlay, where the patch already get's applied. Anyway, we always like to see patches getting upstream. Therefore this pull request :)

@TimmyLiu
Copy link
Contributor

I did install the package. But it seems there was a recent pull request #71 that by default will make the install destination a system directory on linux system, which I have mixed feelings about. Setting the top level install destination (CMAKE_INSTALL_PREFIX) from cmake, to a destination of your choice will install all the binaries and scripts there. I wonder if that would be okay with you.

@marbre
Copy link
Contributor Author
marbre commented Mar 27, 2015

I just build clBLAS from the develop branch.

-- Installing: /var/tmp/portage/sci-libs/clblas-9999/image/usr/bin/clBLAS-client
-- Installing: /var/tmp/portage/sci-libs/clblas-9999/image/usr/bin/testPerfWrapper
-- Installing: /var/tmp/portage/sci-libs/clblas-9999/image/usr/bin/measurePerformance.py
-- Installing: /var/tmp/portage/sci-libs/clblas-9999/image/usr/bin/plotPerformance.py
-- Installing: /var/tmp/portage/sci-libs/clblas-9999/image/usr/bin/blasPerformanceTesting.py
-- Installing: /var/tmp/portage/sci-libs/clblas-9999/image/usr/bin/errorHandler.py
 [..]

The install path /var/tmp/portage/sci-libs/clblas-9999/image/ get's stripped on Gentoo systems in the next step, thus the Python scripts geht installed to /usr/bin.

I also checked pull request #71, this doesn't affect the install location of the python scripts.

@TimmyLiu
Copy link
Contributor

Hi. It sounds like all the binaries were installed correctly to the install path but the python scripts are still installed to /usr/bin, as well as clBLAS-client?

@marbre
Copy link
Contributor Author
marbre commented Apr 2, 2015

This is only a snipped, not all binaries (examples) get installed to the correct location ( as mentioned in #75 ).
Anyway, clBLAS-client get's installed to /usr/bin which is correct, but the python scripts still shouldn't be installed to /usr/bin.

@marbre
Copy link
Contributor Author
marbre commented May 12, 2015

How should we proceed here? I would really like to drop the patches from the clblas ebuild.

TimmyLiu pushed a commit that referenced this pull request May 12, 2015
Install scripts/perf to share/clBLAS on non WIN32 systems
@TimmyLiu TimmyLiu merged commit f8af95c into clMathLibraries:develop May 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0