-
Notifications
You must be signed in to change notification settings - Fork 37.7k
Guix documentation improvements #26335
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
53f604e
to
1aa4182
Compare
sipa
reviewed
Oct 18, 2022
1aa4182
to
1d194a9
Compare
1d194a9
to
c7fd4d6
Compare
eba538c
to
b364528
Compare
b364528
to
e4398e9
Compare
Tracks time-machine bump in 298389e.
e4398e9
to
d09c89c
Compare
This was already suggested in the troubleshooting section, but recommending it upfront would prevent the issue in the first place and speed up builds.
They are mentioned in the figure above, but having them in the table makes it easier to (apt) install everything required.
Co-Authored-By: Stephan Oeste <emzy@emzy.de>
d09c89c
to
68fab72
Compare
The current commits are enough for me to build Guix itself from source as root. |
Also drop unused links.
ACK 10660c0 Haven't followed the purge directions, but this all seems like a strict improvement. |
sidhujag
pushed a commit
to syscoin/syscoin
that referenced
this pull request
Nov 25, 2022
10660c0 doc: move Guix uninstall instructions to INSTALL.md (Sjors Provoost) 68fab72 guix: OpenSSL test failure workaround (Sjors Provoost) d612dca guix: reminder to migrate guix-daemon-original customization (Sjors Provoost) 8aa460c guix: add guile-gnutls and guile-json to install list (Sjors Provoost) 9b9991e guix: recommend mounting a tmpfs on /tmp (Sjors Provoost) 6822834 guix: bump recommended hash for manual installation (Sjors Provoost) Pull request description: I'm manually installing Guix on a fresh Ubuntu machine. Will be pushing more documentation fixes to this PR as I run into things. 1. Bump minimum hash to match time-machine bump in bitcoin#25099. It's not necessary for the root Guix version to match the time-machine version in our build, because `guix build` will automatically perform an upgrade for the user, but imo it's better to get any build issues (in Guix itself) over with while the user is going though `INSTALL.md`, rather than during their first Guix build (of Bitcoin Core). 2. Recommend mapping a tmpfs to /tmp upfront, rather than in the troubleshooting section 3. Add `guile-gnutls` and `guile-json` to the table of stuff to install (avoids having to find out in the `./configure` phase) 4. Improve systemd doc 5. Workaround OpenSSL v1.1.1l and v1.1.1n test failure (change machine time) 6. Move uninstallation instructions to INSTALL.md, drop unused footnote / links ACKs for top commit: jamesob: ACK bitcoin@10660c0 Tree-SHA512: ff1278b16f03ea9c63e23e97a852340ab824d5f6c64645cb70237dd828b9a439b4133b60cd2b89672573f6546e99419021d092e236f731908158a7aa6473b0ef
kwvg
added a commit
to kwvg/dash
that referenced
this pull request
Apr 28, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm manually installing Guix on a fresh Ubuntu machine. Will be pushing more documentation fixes to this PR as I run into things.
guix build
will automatically perform an upgrade for the user, but imo it's better to get any build issues (in Guix itself) over with while the user is going thoughINSTALL.md
, rather than during their first Guix build (of Bitcoin Core).guile-gnutls
andguile-json
to the table of stuff to install (avoids having to find out in the./configure
phase)