{tools}[GCCcore/12.3.0] Python-bundle-PyPI v2023.06, hatchling v1.18.0, poetry v1.5.1, setuptools-rust v1.6.0, virtualenv v20.23.1, ... w/ Python 3.11.3#18208
Conversation
…chling-1.18.0-GCCcore-12.3.0.eb, virtualenv-20.23.1-GCCcore-12.3.0.eb, poetry-1.5.1-GCCcore-12.3.0.eb and patches: Python-3_9-blist-1.3.6-fix-undefined_symbol_PyObject_GC_IS_TRACKED.patch, Python-3.10-bist-1.3.6-compatibility.patch, Python-3.11-bist-1.3.6-compatibility.patch
…and hatchling-1.18.0-GCCcore-12.3.0.eb
This comment was marked as outdated.
This comment was marked as outdated.
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
@SebastianAchilles Except for It's very difficult to asses whether they're only used as build dependencies or not, I think. There may be some clear exceptions, like |
…or building python packages
|
@boegel Sounds reasonable. I have added most extensions back, except:
|
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
There was a problem hiding this comment.
changes that should be made here:
- separate easyconfig for
cryptographyusingCargoPythonPackage(mainly to avoid download of Rust crates during installation)- maybe using the wheel, if needed?
- use
python3 -m easybuild.easyblocks.cargo .insrc/rustdirectory ofcryptographyto generatecrateslist
- separate easyconfig for
setuptools-rust(build dep forcryptography) - no need for
virtualenvas build dep inPython-bundle-PyPI+virtualenvshould depend onPython-bundle-PyPI
…te easyconfig for setuptools-rust, virtualenv, pycparser, cffi and scikit-build
Good ideas 👍 The requested changes are now implemented. |
|
Test report by @SebastianAchilles |
|
Test report by @SebastianAchilles |
|
Was there any reason not to bundle in pycparser with cffi (in an effort to minimize number of PYTHONPATHs)? |
lexming
left a comment
There was a problem hiding this comment.
Two more changes and this is totally ready on my side.
Co-authored-by: Alex Domingo <[email protected]>
Co-authored-by: Alex Domingo <[email protected]>
Co-authored-by: Alex Domingo <[email protected]>
Co-authored-by: Alex Domingo <[email protected]>
Co-authored-by: Alex Domingo <[email protected]>
There was no reason. When I worked on that, I still though both |
|
Test report by @SebastianAchilles |
There was a problem hiding this comment.
@SebastianAchilles thanks for all the updates, I just made a last minor change removing 2 extensions from poetry already provided by the dep in virtualenv
LGTM
I've been also keeping track of the dependency graph with a diagram 🙂
|
@boegelbot: please test @ generoso |
|
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1643914410 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen2 |
|
@lexming: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1645311125 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
This is ready to be merged. I think the best test now is to start building on top of these easyconfigs in case potential issues would show up down the line. So going in, thanks @SebastianAchilles ! |
Review addressed by author
(created using
eb --new-pr)Alternative for #18193
This PR splits of
poetry,hatchling, andvirtualenvinto their own easyconfigs, since they are mostly used for building Python packages.Poetryis usinghatchlingandvirtualenvas dependencies.Python-bundle-PyPIusespoetry,hatchling, andvirtualenvasbuilddependencies. It contains the same packages (but newer versions) as https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/p/Python/Python-3.10.8-GCCcore-12.2.0.eb except the following packages which I think are mostly needed for building python packages:tomlisetuptools_scmsetuptools-rustdistlibpathspeceditablestrove_classifiershatchlinghatch_vcsfilelockvirtualenvhatch_fancy_pypi_readmeflit_scmtomli_wflitclikittomlkitshellinghamrequests-toolbeltpkginfoptyprocesspexpectdistroscikit_buildlockfilepoetry_coredulwichpoetry_plugin_exportbuildinstallerpyproject_hookspoetryFor now I am keeping the removed packages as comments in case we want to add any of them back. However, before merging this MR, I want to clean up the comments.