Skip to content

Feature/so versioning#482

Merged
diptorupd merged 2 commits intomasterfrom
feature/so-versioning
May 28, 2021
Merged

Feature/so versioning#482
diptorupd merged 2 commits intomasterfrom
feature/so-versioning

Conversation

@oleksandr-pavlyk
Copy link
Copy Markdown
Contributor

@oleksandr-pavlyk oleksandr-pavlyk commented May 28, 2021

Reenabled so-versioning and tweaked setup commands to correctly copy symbolic links on Linux

@oleksandr-pavlyk
Removed DPCLSyclInterface library from MANIFEST.in
529947e 
Manifest is meant for source distributions, and binary libraries
does not belong there.

Added custom build_py step to copy shared object library from install/
to build_lib folder (build/lib.cpython-..../dpctl)

Added custom post-build step to install command to remove hard links
of versioned shared object library on linux, and recopy them from
build_lib to installation folder.
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented May 28, 2021 •
edited
Loading

Coverage Status

Coverage remained the same at 60.917% when pulling bc4b924 on feature/so-versioning into 76680ff on master.

@oleksandr-pavlyk oleksandr-pavlyk force-pushed the feature/so-versioning branch from 237fd10 to bc4b924 Compare May 28, 2021 03:02
@oleksandr-pavlyk
Copy link
Copy Markdown
Contributor Author

oleksandr-pavlyk commented May 28, 2021 •
edited
Loading

Using 3.7 artifact built by CI:

(w) [23:04:12 nuc04 tmp]$ ls
dpctl-0.8.0-py37hf4177b6_4.tar.bz2

(w) [23:04:13 nuc04 tmp]$ tar xf dpctl-0.8.0-py37hf4177b6_4.tar.bz2
(w) [23:04:18 nuc04 tmp]$ ls
dpctl-0.8.0-py37hf4177b6_4.tar.bz2  info  lib
(w) [23:04:24 nuc04 tmp]$ ls -l lib/python3.7/site-packages/dpctl/libDPCTL*
lrwxrwxrwx 1 usr group     28 May 27 22:38 lib/python3.7/site-packages/dpctl/libDPCTLSyclInterface.so -> libDPCTLSyclInterface.so.0.8
lrwxrwxrwx 1 usr group     28 May 27 22:38 lib/python3.7/site-packages/dpctl/libDPCTLSyclInterface.so.0 -> libDPCTLSyclInterface.so.0.8
-rw-r--r-- 1 usr group 197840 May 27 22:38 lib/python3.7/site-packages/dpctl/libDPCTLSyclInterface.so.0.8

Mission accomplished.

@diptorupd
Copy link
Copy Markdown
Contributor

diptorupd commented May 28, 2021

Using 3.7 artifact built by CI:

(w) [23:04:12 nuc04 tmp]$ ls
dpctl-0.8.0-py37hf4177b6_4.tar.bz2

(w) [23:04:13 nuc04 tmp]$ tar xf dpctl-0.8.0-py37hf4177b6_4.tar.bz2
(w) [23:04:18 nuc04 tmp]$ ls
dpctl-0.8.0-py37hf4177b6_4.tar.bz2  info  lib
(w) [23:04:24 nuc04 tmp]$ ls -l lib/python3.7/site-packages/dpctl/libDPCTL*
lrwxrwxrwx 1 usr group     28 May 27 22:38 lib/python3.7/site-packages/dpctl/libDPCTLSyclInterface.so -> libDPCTLSyclInterface.so.0.8
lrwxrwxrwx 1 usr group     28 May 27 22:38 lib/python3.7/site-packages/dpctl/libDPCTLSyclInterface.so.0 -> libDPCTLSyclInterface.so.0.8
-rw-r--r-- 1 usr group 197840 May 27 22:38 lib/python3.7/site-packages/dpctl/libDPCTLSyclInterface.so.0.8

Mission accomplished.

Indeed 👍 great job peeling through the onion layers

@diptorupd diptorupd merged commit 0951961 into master May 28, 2021
@diptorupd diptorupd deleted the feature/so-versioning branch May 28, 2021 15:47
@diptorupd diptorupd mentioned this pull request May 28, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PokhodenkoSA PokhodenkoSA Awaiting requested review from PokhodenkoSA

1 more reviewer

@diptorupd diptorupd diptorupd approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@oleksandr-pavlyk @coveralls @diptorupd