Skip to content
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.

Sign up for GitHub

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

Jump to bottom

tips: add instructions for using RPM Fusion #137

Merged
merged 1 commit into from
Nov 22, 2022
Merged

tips: add instructions for using RPM Fusion #137

merged 1 commit into from
Nov 22, 2022

Conversation

miabbott
Copy link
Member

I've (anecdotally) seen a number of folks complaining about using RPM Fusion with Silverblue and the challenges that come with that.

Let's just admit the obvious and give users the straight-forward way of enabling the repos.

@travier
Copy link
Member

travier commented Nov 18, 2022 •
edited
Loading

We can simplify the second command with the one in https://discussion.fedoraproject.org/t/simplifying-updates-for-rpm-fusion-packages-and-other-packages-shipping-their-own-rpm-repos/30364/34?u=siosm:

$ sudo rpm-ostree update \
	--uninstall rpmfusion-free-release \
	--uninstall rpmfusion-nonfree-release \
	--install rpmfusion-free-release \
	--install rpmfusion-nonfree-release

@travier
Copy link
Member

travier commented Nov 18, 2022

Maybe we should provide our own full set of commands until this is fixed in the RPM Fusion Wiki.

@travier
Copy link
Member

travier commented Nov 18, 2022

The "correct" and complete way of setting this up right directly is:

# Install RPM Fusion repos
$ sudo rpm-ostree install \
	https://mirrors.rpmfusion.org/free/fedora/rpmfusion-free-release-$(rpm -E %fedora).noarch.rpm \
	https://mirrors.rpmfusion.org/nonfree/fedora/rpmfusion-nonfree-release-$(rpm -E %fedora).noarch.rpm
$ reboot

# Replace the versioned local packages by overlayed packages from the repo
$ sudo rpm-ostree update \
	--uninstall rpmfusion-free-release \
 	--uninstall rpmfusion-nonfree-release \
	--install rpmfusion-free-release \
	--install rpmfusion-nonfree-release
$ reboot

travier
travier reviewed Nov 18, 2022
View reviewed changes
modules/ROOT/pages/tips-and-tricks.adoc Outdated
== Enabling RPM Fusion repos

Users may want to take advantage of the non-free software that is made available via the https://rpmfusion.org/[RPM Fusion]
repos in order to use the proprietary NVidia drivers, multimedia codecs, or other software not
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
repos in order to use the proprietary NVidia drivers, multimedia codecs, or other software not
repos in order to use the proprietary NVIDIA drivers, multimedia codecs, or other software not

@miabbott
Copy link
Member Author

We can simplify the second command with the one in https://discussion.fedoraproject.org/t/simplifying-updates-for-rpm-fusion-packages-and-other-packages-shipping-their-own-rpm-repos/30364/34?u=siosm:

$ sudo rpm-ostree update \
	--uninstall rpmfusion-free-release \
	--uninstall rpmfusion-nonfree-release \
	--install rpmfusion-free-release \
	--install rpmfusion-nonfree-release

Good catch; I was just copy/pasting from the top comment on that thread.

@miabbott
Copy link
Member Author

Updated with Timothee's suggestions ⬆️

@tpopela
Copy link

tpopela commented Nov 21, 2022 •
edited
Loading

This is definitely an improvement, but I don't know whether we rather shouldn't fix the RPMFusion wiki instead - specifically the https://rpmfusion.org/Howto/OSTree linked from https://rpmfusion.org/Configuration. Also https://docs.fedoraproject.org/en-US/quick-docs/setup_rpmfusion/ might need a "OSTree based" specific section as well. I would also like to see a header section as in the Fedora Docs:

""
This page discusses third-party software sources not officially affiliated with or endorsed by the Fedora Project. Use them at your own discretion. Fedora recommends the use of free and open source software and avoidance of software encumbered by patents.
""

We could take as it is only replacing the "This page" with "This section".

@travier
Copy link
Member

travier commented Nov 21, 2022 •
edited
Loading

@tpopela See https://discussion.fedoraproject.org/t/simplifying-updates-for-rpm-fusion-packages-and-other-packages-shipping-their-own-rpm-repos/30364/32?u=siosm. I don't feel good pinging folks so many times and I can not edit the Wiki.

@travier
Copy link
Member

travier commented Nov 21, 2022

Linking coreos/rpm-ostree#1978 here too

@travier
Copy link
Member

travier commented Nov 21, 2022 •
edited
Loading

We should try to find another workaround that does not involve two reboots.

@miabbott
Copy link
Member Author

This is definitely an improvement, but I don't know whether we rather shouldn't fix the RPMFusion wiki instead - specifically the https://rpmfusion.org/Howto/OSTree linked from https://rpmfusion.org/Configuration. Also https://docs.fedoraproject.org/en-US/quick-docs/setup_rpmfusion/ might need a "OSTree based" specific section as well. I would also like to see a header section as in the Fedora Docs:

"" This page discusses third-party software sources not officially affiliated with or endorsed by the Fedora Project. Use them at your own discretion. Fedora recommends the use of free and open source software and avoidance of software encumbered by patents. ""

We could take as it is only replacing the "This page" with "This section".

https://pagure.io/fedora-docs/quick-docs/pull-request/505

@miabbott
tips: add instructions for using RPM Fusion
852addd 
I've (anecdotally) seen a number of folks complaining about using RPM
Fusion with Silverblue and the challenges that come with that.

Let's just admit the obvious and give users the straight-forward way
of enabling the repos.
@miabbott
Copy link
Member Author

Updated with the warning message ⬆️

@tpopela tpopela merged commit d104c45 into fedora-silverblue:master Nov 22, 2022
@tpopela
Copy link

tpopela commented Nov 22, 2022

Thank you @miabbott !!

@westurner
Copy link

To get this working with SecureBoot:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@travier travier travier approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@miabbott @travier @tpopela @westurner