-
Notifications
You must be signed in to change notification settings - Fork 38.7k
doc: Fedora build requirements #7968
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
utACK 6acd133 |
|
utACK 6acd133 |
doc/README.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting. Good catch - this hasn't been true for a while. We now link qt statically on Linux, so there is no actual need to install any run-time library.
Building is explained in build-unix.md so this part can go entirely (as this is targeted at people installing the binaries).
|
Changes to build-unix.md ACK |
|
OK, so you want removal of stuff from doc/README.md, will do. |
|
Yes, either you should remove it or leave it alone (to be cleaned up by someone else). Extending it at least makes no sense. |
6acd133 to
0281678
Compare
|
Force pushed to update only the build requirements. doc/README.md should be cleaned up separately. |
0281678 doc: Fedora build requirements (Warren Togami)
|
|
||
| To build with Qt 5 (recommended) you need the following: | ||
|
|
||
| sudo dnf install qt5-qttools-devel qtr5-qtbase-devel protobuf-devel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/qtr5/qt5/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch
0281678 doc: Fedora build requirements (Warren Togami)
No description provided.