PR#81: CL8 -> CS8 conversion cleanup - centos/centos.org

centos / centos.org

#81 CL8 -> CS8 conversion cleanup

Merged 3 years ago by rbowen. Opened 3 years ago by carlwgeorge.

centos/ carlwgeorge/centos.org converting into master

Show full package names that are being swapped

Carl George • 3 years ago

6579d5

Use default prompt in conversion steps

Carl George • 3 years ago

24d5ae

Remove unnecessary conversion steps

Carl George • 3 years ago

43457d

centos-stream.md

file modified

+2 -7

		`@@ -17,14 +17,9 @@`
		`## Converting from CentOS Linux to CentOS Stream`

		```console
		`- root@centos-linux# dnf install centos-release-stream`
		`+ [root@centos ~]# dnf swap centos-linux-repos centos-stream-repos`

		`- root@centos-linux# dnf swap centos-{linux,stream}-repos`
		`-`
		`- root@centos-linux# dnf distro-sync`
		`-`
		`- root@centos-stream# cat /etc/centos-release`
		`- CentOS Stream release 8`
		`+ [root@centos ~]# dnf distro-sync`
		```

distro-faq.md

file modified

+2 -7

		`@@ -62,14 +62,9 @@`
		`and are also below for your convenience.`

		`<pre>`
		`- root@centos-linux# dnf install centos-release-stream`
		`+ [root@centos ~]# dnf swap centos-linux-repos centos-stream-repos`

		`- root@centos-linux# dnf swap centos-{linux,stream}-repos`
		`-`
		`- root@centos-linux# dnf distro-sync`
		`-`
		`- root@centos-stream# cat /etc/centos-release`
		`- CentOS Stream release 8`
		`+ [root@centos ~]# dnf distro-sync`
		`</pre>`

		`This will result in some package updates and new packages being installed, which is expected.`

carlwgeorge commented 3 years ago

Remove unnecessary conversion steps
Use default prompt in conversion steps

jamesantill commented 3 years ago

Looks fine although it might be worth keeping the "check /etc/centos-release" step, but obviously if that is wrong and the other steps worked they are in a bad place.

rebased onto 43457d1

3 years ago

carlwgeorge commented 3 years ago

It's not wrong per say, but the goal is to trim down the instructions to as few commands as possible. It's also weird to me to show the contents of that file after the switch but not before also. So I'd say we have that command first and last, or not at all.

carlwgeorge commented 3 years ago

Another thing I was thinking: is the --allowerasing flag a better solution than naming both packages? Both of these work.

dnf swap centos-{linux,stream}-repos
dnf --allowerasing install centos-stream-repos

1 new commit added

Show full package names that are being swapped

3 years ago

carlwgeorge commented 3 years ago

Did a quick straw poll in IRC, and people seem to prefer the swap command, just with no bash expansion for clarity. I pushed another commit to tweak that.

Edited 3 years ago by carlwgeorge