Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Fedora Prerequisites are (possibly) inaccurate #9137

Closed
wants to merge 2 commits into from
Closed

Fedora Prerequisites are (possibly) inaccurate #9137

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 16, 2021

python3-devel is required to build the wheel for Twisted, otherwise it'll just fail. psycopg2 is also required for Postgresql. I don't believe these come with the "Development Tools" group, but I might be mistaken.

`python3-devel` is required to build the wheel for Twisted, otherwise it'll just fail. `psycopg2` is also required for Postgresql. I don't believe these come with the "Development Tools" group, but I might be mistaken.
@clokep
Copy link
Member

clokep commented Jan 18, 2021

@compu42 Can you sign-off on your commit? Thanks!

@clokep clokep added the X-Awaiting-Changes A contributed PR which needs changes and re-review before it can be merged label Jan 18, 2021
INSTALL.md Outdated Show resolved Hide resolved
@clokep clokep requested a review from callahad February 3, 2021 17:09
@callahad
Copy link
Contributor

callahad commented Feb 4, 2021

Using Fedora 33 on ARM, I can confirm that libpq-devel and python3-devel are required for pip install -e '.[all,test,lint]' to succeed.

Attempting to pip install --no-binary ':all:' -e '.[all,test,lint]' reveals further dependencies on libxml2-devel and libxslt-devel.

It also trips over matrix-org/python-unpaddedbase64#4: our unpaddedbase64 library doesn't provide a source distribution.

And a few dependencies need wheel installed so that bdist_wheel is available when building.

I did not need to explicitly install tk-devel or redhat-rpm-config, which were introduced years ago in d7739c4 and 5325455 respectively.

I'll tidy those up and get this merged.

@callahad
Copy link
Contributor

callahad commented Feb 4, 2021

Upon looking more closely at install.md, we have lots of very, very outdated and ill-maintained information in there.

We should probably purge it and move most distro-specific information to the GitHub wiki or similar.

@callahad
Copy link
Contributor

callahad commented Feb 4, 2021

I've continued this, cleaned it up, and pushed an additional commit over at #9322. Thank you, @compu42 !

@callahad callahad closed this Feb 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
X-Awaiting-Changes A contributed PR which needs changes and re-review before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants