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.

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

Added instructions for installing EPEL and RPM Fusion. #288

Merged
merged 7 commits into from
Jun 10, 2024

Conversation

rl2939
Copy link
Contributor

@rl2939 rl2939 commented Jul 24, 2023

What does this pull request do?

This pull request adds instructions for adding the EPEL and RPM Fusion software repositories, with GPG key verification.

Why?

This pull request aims to make the process of installing several software repositories more clear for the end user.

The current documentation does not explain how to install EPEL in a clear manner (see #232). This pull request changes the instructions to the official quickstart guide on the Fedora project website.

In addition, there are no instructions on how to install RPMfusion (which provides additional packages) anywhere on the website. The RPMfusion website does have directions on how to install their repositories, but does not do it in a way where the user can verify the GPG signing key in an easy manner. This pull request adds instructions on how to install RPM Fusion as well as verifying the signing keys.

Also included is an guide on how to install both EPEL and RPM Fusion. This is at the bottom of the documentation section so that new users can find it.

fixes #232

xsub added a commit to xsub/wiki that referenced this pull request Jan 10, 2024
xsub added a commit to xsub/wiki that referenced this pull request Jan 11, 2024
… from AlmaLinux#265"

This reverts commit ca3bba0, reversing
changes made to de079ac.
xsub added a commit to xsub/wiki that referenced this pull request Jan 11, 2024
@bennyvasquez
Copy link
Sponsor Member

@xsub @rl2939 it's not clear to me, so I wanted to check. Was this content in this PR included as part of #265?

@rl2939
Copy link
Contributor Author

rl2939 commented Jan 16, 2024

I believe the PR was reverted (not included). @xsub Is there anything that needs to be corrected in this PR?

@xsub
Copy link
Contributor

xsub commented Jan 17, 2024

Hi @rl2939!

I initially cherry-picked your commit and then attempted to enhance it, which unfortunately led to some confusion with versions during the merge. That's why my commit is missing it – my apologies 😅

Regarding the enhancements:

  • I completely get your idea to rename "system series" to "post installation." However, I've decided to skip that for now. This is partly because the After-Installation Guide by @sboldyreva was recently added, which includes "enable-necessary-repositories." Your content could easily link to this guide...

  • Considering the above, I was thinking maybe we could move your instructions to a dedicated "System Series" how-to?

I’d really value your thoughts on this, @rl2939, and @bennyvasquez, I’d also appreciate your input.
Your insights would be very helpful in defining our approach 😊

@bennyvasquez bennyvasquez added the question Further information is requested label Mar 29, 2024
Copy link
Sponsor Member

@bennyvasquez bennyvasquez left a comment

Choose a reason for hiding this comment

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

A few suggestions here, and I think we'll be ready to merge this in, @rl2939. Thanks for your patience!

docs/repos/Extras.md Outdated Show resolved Hide resolved
docs/repos/Extras.md Outdated Show resolved Hide resolved
docs/documentation/epel-and-rpmfusion.md Outdated Show resolved Hide resolved
@bennyvasquez
Copy link
Sponsor Member

@rl2939 If you don't mind, can you also rebase to pull in the updated config.js and add your guides into the updated one?

rl2939 and others added 5 commits June 1, 2024 22:19
Co-authored-by: benny Vasquez <bennyvasquez@users.noreply.github.com>
Co-authored-by: benny Vasquez <bennyvasquez@users.noreply.github.com>
Co-authored-by: benny Vasquez <bennyvasquez@users.noreply.github.com>
Removed extra word.
@rl2939
Copy link
Contributor Author

rl2939 commented Jun 2, 2024

@bennyvasquez I updated the config.js to match the current one. Will this work?

@bennyvasquez bennyvasquez merged commit 63d4455 into AlmaLinux:master Jun 10, 2024
@bennyvasquez
Copy link
Sponsor Member

Thanks for your patience and work here, @rl2939! So glad to get this merged in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extra Repositories Additional EPEL Info
3 participants