Skip to content

py-setuptools: from wheel to source package, and backport rpath patch#40205

Merged
adamjstewart merged 1 commit intospack:developfrom
haampie:py-setuptools/backport-distutils-rpath-flag-patch
Oct 8, 2023
Merged

py-setuptools: from wheel to source package, and backport rpath patch#40205
adamjstewart merged 1 commit intospack:developfrom
haampie:py-setuptools/backport-distutils-rpath-flag-patch

Conversation

@haampie
Copy link
Copy Markdown
Member

@haampie haampie commented Sep 26, 2023

Now that #40224 is merged, we can turn setuptools into a source
package again, so that the rpath patch that we had on Python itself
for version <= 3.11 can be applied to distutils vendored by setuptools.

@spackbot-app spackbot-app bot requested a review from adamjstewart September 26, 2023 10:18
@haampie haampie changed the title py-setuptools: backport pass rpath args to linker through compiler patch py-setuptools: backport patch that fixes rpath flag to linker through compiler Sep 26, 2023
@haampie

This comment was marked as outdated.

@haampie haampie force-pushed the py-setuptools/backport-distutils-rpath-flag-patch branch 5 times, most recently from ae7f10d to 4006b19 Compare September 26, 2023 11:24
@haampie haampie changed the title py-setuptools: backport patch that fixes rpath flag to linker through compiler py-setuptools: from wheel to source package, and backport rpath patch Sep 26, 2023
@haampie

This comment was marked as outdated.

@spackbot-app

This comment was marked as outdated.

@adamjstewart adamjstewart self-assigned this Sep 26, 2023
@adamjstewart
Copy link
Copy Markdown
Member

Not opposed to going back to sdist but want to get @alalazo's thoughts first.

@haampie

This comment was marked as outdated.

@haampie

This comment was marked as outdated.

@haampie

This comment was marked as outdated.

@spackbot-app spackbot-app bot added build-systems core PR affects Spack core functionality labels Sep 27, 2023
@haampie haampie force-pushed the py-setuptools/backport-distutils-rpath-flag-patch branch from 03f11fb to b137908 Compare September 27, 2023 15:08
@haampie haampie force-pushed the py-setuptools/backport-distutils-rpath-flag-patch branch from b137908 to 0ff077d Compare October 6, 2023 14:43
@haampie haampie requested a review from adamjstewart October 6, 2023 14:46
@haampie
Copy link
Copy Markdown
Member Author

haampie commented Oct 6, 2023

Rebased this after #40224

@adamjstewart
Copy link
Copy Markdown
Member

@spackbot run pipeline

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Oct 8, 2023

I've started that pipeline for you!

@adamjstewart adamjstewart enabled auto-merge (squash) October 8, 2023 15:10
@adamjstewart adamjstewart merged commit 512e41a into spack:develop Oct 8, 2023
haampie added a commit to haampie/spack that referenced this pull request Oct 31, 2023
haampie added a commit that referenced this pull request Nov 1, 2023
* Revert "Improve build isolation in PythonPipBuilder (#40224)"

This reverts commit 0f43074.

* Revert "py-setuptools: sdist + rpath patch backport (#40205)"

This reverts commit 512e41a.
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Nov 2, 2023
gabrielctn pushed a commit to gabrielctn/spack that referenced this pull request Nov 24, 2023
* Revert "Improve build isolation in PythonPipBuilder (spack#40224)"

This reverts commit 0f43074.

* Revert "py-setuptools: sdist + rpath patch backport (spack#40205)"

This reverts commit 512e41a.
@haampie haampie deleted the py-setuptools/backport-distutils-rpath-flag-patch branch December 13, 2023 10:11
mtaillefumier pushed a commit to mtaillefumier/spack that referenced this pull request Dec 14, 2023
* Revert "Improve build isolation in PythonPipBuilder (spack#40224)"

This reverts commit 0f43074.

* Revert "py-setuptools: sdist + rpath patch backport (spack#40205)"

This reverts commit 512e41a.
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Jan 11, 2024
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants