#1060 frontend: docs: external dnf isn't needed for modularity now
Merged 4 years ago by praiskup. Opened 4 years ago by praiskup.
Unknown source modularity-docs  into  master

@@ -132,32 +132,18 @@

            </div>

          {% else %}

          <p>

-           To start using modules, you need to have DNF with modularity support. This is available out of the box

-           on the <a href="https://getfedora.org/en/server/prerelease/">Fedora Modular Server</a>, but on regular releases you need to install such DNF from Copr.

+           To start using modules, you need to have DNF with modularity support

+           (the <code>dnf module</code> command, should work on Fedora and

+           RHEL&nbsp;8+).  See the "Module Command" section in

+           <code>man dnf</code>.

          </p>

-         <div class="highlight autumn">

-           <pre>dnf copr enable mhatina/dnf-modularity-stable

- dnf upgrade dnf</pre>

-         </div>

- 

-         <p>Now to install the built module, please run following commands as root:</p>

-         <div class="highlight autumn">

-           <pre><span class="c1"># Add module repofile to your system</span>

- dnf copr enable {{ module.copr.full_name }}

- 

- <span class="c1"># Install the module</span>

- dnf module enable {{ module.name}}:{{ module.stream }}

- dnf module install {{ module.name }}/default</pre>

Hmm, reading this again. Do we really want to remove these three lines?

- 

-         </div>

- 

-         <p>For <code>dnf module</code> documentation, see the "Module Command" section in <code>man dnf</code>.

- 

-         <hr>

-         Repofiles:

+         <p>

+           Please install related repo file into <code>/usr/yum.repos.d/</code>

+           directory on your system: <br />

          {% for chroot in unique_chroots  %}

            <a href="{{ copr_url('coprs_ns.generate_module_repo_file', copr, name_release=chroot.name_release, module_nsv=module.nsv) }}" class="btn btn-default"><span class="pficon pficon-save"></span> {{ chroot.name_release }}</a>

          {% endfor %}

+         </p>

  

          {% endif %}

        </div>

Resolves: rhbz#1760150

rebased onto 4f98a3838ee2fd42ab44cc3a0e675fa207dce5db

4 years ago

rebased onto 4f98a3838ee2fd42ab44cc3a0e675fa207dce5db

4 years ago

+1

Also, please note, that

# Add module repofile to your system
dnf copr enable orion/cobbler

doesn't actually provide the module repofile, just the regular one for the Copr project. You need to manually download one of the repofiles provided a few lines below.

so do you suggest some update?

rebased onto 57bd79d404273259712298dd7e76a10e4c9f6578

4 years ago

rebased onto 51c910c

4 years ago

@orion WDYT? I think it was pretty easy to forget about the module repofile.

@frostyx should we enhance dnf copr command so it supports copr modules?

Seems reasonable to me. I'm still getting started with modules as well. I did almost overlook the module repo the first time.

I see, can you have another look?

I like this

@frostyx should we enhance dnf copr command so it supports copr modules?

We really should, but there will be some things necessary to do before. I would go with this PR until then.

+1

Hmm, reading this again. Do we really want to remove these three lines?

Commit bbe4b26 fixes this pull-request

Pull-Request has been merged by praiskup

4 years ago