Skip to content

Search rpmbuild in more paths#10172

Closed
xrmx wants to merge 1 commit into
python:masterfrom
xrmx:lookforrpmbuildinusrlocal
Closed

Search rpmbuild in more paths#10172
xrmx wants to merge 1 commit into
python:masterfrom
xrmx:lookforrpmbuildinusrlocal

Conversation

@xrmx

@xrmx xrmx commented Oct 28, 2018

Copy link
Copy Markdown
Contributor

So there's more chances that is found. At least on MacOS when
installed with brew. This is the minimal fix i can find to help
with: pypa/setuptools#1456.

Comment thread Lib/distutils/command/bdist_rpm.py Outdated
@xrmx xrmx force-pushed the lookforrpmbuildinusrlocal branch from 1671982 to 1596f90 Compare October 29, 2018 10:13
@xrmx xrmx changed the title Search rpmbuild in /usr/local/bin Search rpmbuild in more paths Oct 29, 2018
So there's more chances that is found. At least on MacOS when
installed with brew. This is the minimal fix i can find to help
with: pypa/setuptools#1456.
@xrmx xrmx force-pushed the lookforrpmbuildinusrlocal branch from 1596f90 to b370d17 Compare December 29, 2018 22:35
@xrmx

xrmx commented Dec 30, 2018

Copy link
Copy Markdown
Contributor Author

I've rebased the PR to latest master. I consider this patch trivial so I've omitted the issue and the news file, are something else from my side missing to move this PR forward? Thanks

@asottile asottile left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@csabella

Copy link
Copy Markdown
Contributor

Closing as duplicate of #10594.

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.

5 participants