doc: minor updates to guix README #25833

pull satsie wants to merge 1 commits into bitcoin:master from satsie:update-guix-readme changing 2 files +3 −3

satsie commented at 5:22 PM on August 12, 2022: contributor
Two minor updates to the guix docs:
- contrib/guix/README.md: fix broken link
- contrib/guix/INSTALL.md: Change Ubuntu version in the section on distribution maintained packages from 21.04 (Hirsute Hippo) to 22.04 (Jammy Jellyfish). The previous link to the Ubuntu Guix package (https://packages.ubuntu.com/hirsute/guix) was for Hirsute. That link is now broken, likely because Hirsute reached EOL in January. I was unable to locate a general page for Ubuntu Guix packages so I replaced the broken link with the search results for all Ubuntu Guix packages. That page currently displays Guix packages for three different versions of Ubuntu. Happy to replace this link if there is a better option.
DrahtBot added the label Docs on Aug 12, 2022
adam2k commented at 7:28 PM on August 12, 2022: none

ACK 57a1a2dbbcb404a294477b0055c6607570f5c6f3

Checked the updated links. Both work as expected.
adam2k approved

in contrib/guix/INSTALL.md:75 in 57a1a2dbbc outdated

  71 | @@ -72,11 +72,11 @@ writing (July 2021). Guix is expected to be more widely packaged over time. For
  72 |  an up-to-date view on Guix's package status/version across distros, please see:
  73 |  https://repology.org/project/guix/versions
  74 |  
  75 | -### Debian 11 (Bullseye)/Ubuntu 21.04 (Hirsute Hippo)
  76 | +### Debian 11 (Bullseye)/Ubuntu 22.04 (Jammy Jellyfish)

fanquake commented at 2:03 PM on August 13, 2022:

### Debian / Ubuntu

Probably easiest to just drop the versioning / names from here.

jarolrod commented at 7:18 PM on August 14, 2022:

would second this

satsie commented at 7:01 PM on August 15, 2022:

Thank you for the code suggestion! I just committed it and GitHub auto resolved this comment.

jarolrod commented at 7:18 PM on August 14, 2022: member

concept ack
jarolrod commented at 7:05 PM on August 15, 2022: member

@satsie you will have to squash the commits as a clean commit history is required 👮
satsie force-pushed on Aug 15, 2022
satsie commented at 7:57 PM on August 15, 2022: contributor

thanks @jarolrod! Rebased (and reread that section in the CONTRIBUTING.md) ^-^

update #1: I left in the "co-authored" part of the git commit message. It felt like the right thing to do, but can remove if that is against best practices

update #2: linter got me on that 2 line commit message. will fix now!
doc: minor updates to guix README 98383d6d0d
satsie force-pushed on Aug 15, 2022
jarolrod approved
jarolrod commented at 8:09 PM on August 15, 2022: member

ACK 98383d6d0dade5c2af2adcf10e274141fc7981aa

verified links
maflcko merged this on Aug 16, 2022
maflcko closed this on Aug 16, 2022
sidhujag referenced this in commit 27eb49e2b6 on Aug 16, 2022
satsie deleted the branch on Oct 31, 2022
bitcoin locked this on Oct 31, 2023

Contributors

Labels