SmartRedis icon indicating copy to clipboard operation
SmartRedis copied to clipboard

Spike: Install python module to install folder

Open billschereriii opened this issue 1 year ago • 2 comments

Description

At present, the Python module and library are run directly from the source directory. This means that the smartredis.cpython.*.so library is copied from the build directory directly into the source directory. This is perfectly functional, but it pollutes the source directory with built code. We should install the module into the install folder instead.

How to reproduce

Build SmartRedis

Expected behavior

Once built, there should be no generated files in the SmartRedis src subtree.

Notes

setup.cfg should be updated to point to the installed location

billschereriii avatar May 19 '23 16:05 billschereriii

Result of backlog discussion. This might be better as a spike to figure out how we can not pollute source directory and still be able to do pip install -e . and edit python files without rebuild.

mellis13 avatar May 19 '23 17:05 mellis13

Symbolic linking the python files to an install location might allow installation without impacting editing (so long as no new python files are added)

billschereriii avatar May 23 '23 14:05 billschereriii