fedora-docs / modularity

#64 Add explanation to Using Modules - Switching module streams section.
Merged by asamalik. Opened by jaruga.
fedora-docs/ jaruga/modularity feature/using-modules  into  master

Download 64.patch

This PR comes from https://pagure.io/fm-orchestrator/issue/1342#comment-582911 .

  • 1st commit is to add missing "sudo". Because "$" prompt means user's prompt. That also aligns with other pages like.
    https://docs.fedoraproject.org/en-US/modularity/making-modules/building-modules-locally/
    1st commit is a preparation about 2nd commit.
  • 2nd commit is main commit.

Below part is because original document does not have "newline at end of file".
As I edited the document by vim, that modification happened.

-Updating a system by running the `dnf update` command causes all packages to be upgraded to their latest version provided by their module stream.
\ No newline at end of file
+Updating a system by running the `dnf update` command causes all packages to be upgraded to their latest version provided by their module stream.

here is typo in taht and also you need to mention that this is entirely UNSUPPORTED. You can do this, but you are on your own if your system breaks because of that.

2 new commits added

  • Add explanation to Using Modules - Switching module streams section.
  • Add "sudo" to "$ dnf .." aligining with other documents.

@ignatenkobrain thanks for the review.

Fixed and rebased.

  • I fixed the typo taht.
  • I added below line at the bottom of the section "Switching module streams".

+NOTE: Switching RPMs is a entirely unsupported operation. You can do this, but you are on your own if your system breaks because of that.

But we already have had below line at the top of the section "Switching module streams". Can we merge these lines?

NOTE: Switching streams is a risky operation that might not be always supported in packages, especially downgrades.

  • Renamed wrong package name rubygem-mysql to rubygem-mysql2
  • Changed an expression "Hint: " to "HINT: ".

ping. Is it possible to merge this PR?

@jaruga Sorry about that, this somehow got lost in my mailbox.

To answer your question with another question... :)

NOTE: Switching RPMs is a entirely unsupported operation. You can do this, but you are on your own if your system breaks because of that.

Why are we guiding users to do something and then saying that it's an "entirely unsupported operation"? That sounds very confusing — should they do that, or not?

I'd remove it and only keep the one that's already there:

NOTE: Switching streams is a risky operation that might not be always supported in packages, especially downgrades.

What do you think?

Why are we guiding users to do something and then saying that it's an "entirely unsupported operation"? That sounds very confusing — should they do that, or not?

Good question :) it should be documented, because switching stream is a necessary operation to use modularity. But the system and developers do not want to support it according to @ignatenkobrain 's opinion.

I'd remove it and only keep the one that's already there:

NOTE: Switching streams is a risky operation that might not be always supported in packages, especially downgrades.

Yes, I agree with you.

1 new commit added

  • Removed "NOTE: Switching RPMs is a entirely unsupported operation. ...".

I'd remove it and only keep the one that's already there:

I removed the added NOTE as an additional commit.

Thanks, @jaruga!

Pull-Request has been merged by asamalik
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.