Differential D332

Fix repo links to point to fc21 in docs
ClosedPublic
Actions

Authored by mkrizek on Apr 8 2015, 7:16 AM.

Details

Reviewers
kparal
tflink
jskladan
Commits
rLTRN013fc7d9315a: Fix repo links to point to fc21 in docs
Test Plan

Docs -> none needed.

Diff Detail

Repository
rLTRN libtaskotron
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mkrizek retitled this revision from to Fix repo links to point to fc21 in docs.Apr 8 2015, 7:16 AM
mkrizek updated this object.
mkrizek edited the test plan for this revision. (Show Details)
mkrizek added reviewers: tflink, kparal, jskladan.
kparal accepted this revision.Apr 8 2015, 10:34 AM
This revision is now accepted and ready to land.Apr 8 2015, 10:34 AM
kparal added a comment.Apr 8 2015, 10:39 AM

Actually, the file contains this:

[tflink-taskotron]
name=Copr repo for taskotron owned by tflink
baseurl=http://copr-be.cloud.fedoraproject.org/results/tflink/taskotron/fedora-$releasever-$basearch/
skip_if_unavailable=True
gpgcheck=0
enabled=1

So it is not hardcoded for a specific release, but uses $releasever and $basearch instead. That is great. So what if we simplified the target file name in /etc/yum.repos.d?

Instead of tflink-taskotron-fedora-21.repo we could name it tflink-taskotron.repo or taskotron-copr.repo or just taskotron.repo?

mkrizek added a comment.Apr 8 2015, 10:43 AM
In D332#6041, @kparal wrote:

Actually, the file contains this:

[tflink-taskotron]
name=Copr repo for taskotron owned by tflink
baseurl=http://copr-be.cloud.fedoraproject.org/results/tflink/taskotron/fedora-$releasever-$basearch/
skip_if_unavailable=True
gpgcheck=0
enabled=1

So it is not hardcoded for a specific release, but uses $releasever and $basearch instead. That is great. So what if we simplified the target file name in /etc/yum.repos.d?

Instead of tflink-taskotron-fedora-21.repo we could name it tflink-taskotron.repo or taskotron-copr.repo or just taskotron.repo?

I suspect the files are generated by copr.

kparal added a comment.Apr 8 2015, 10:55 AM
In D332#6042, @mkrizek wrote:

I suspect the files are generated by copr.

I was talking about the files we save into the user system, not the generated ones.

sudo wget http://copr-fe.cloud.fedoraproject.org/coprs/tflink/taskotron/repo/fedora-21/tflink-taskotron-fedora-21.repo -O /etc/yum.repos.d/taskotron-copr.repo

But if we can't control the repo name header:

[tflink-taskotron]

then I'd say we should keep that in the filename as well. So probably rather:

sudo wget http://copr-fe.cloud.fedoraproject.org/coprs/tflink/taskotron/repo/fedora-21/tflink-taskotron-fedora-21.repo -O /etc/yum.repos.d/tflink-taskotron.repo

But that's just an idea how to make it simpler and shorter in the docs, no big deal.

Closed by commit rLTRN013fc7d9315a: Fix repo links to point to fc21 in docs (authored by mkrizek). · Explain WhyApr 9 2015, 7:55 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathPackages
Mdocs/source/index.rst (2 lines)
Mreadme.rst (2 lines)

Diff 909

View Options

docs/source/index.rst

Show First 20 Lines • Show All 63 Lines • ▼ Show 20 Line(s)
64 64​
65Install Libtaskotron 65​Install Libtaskotron
66==================== 66​====================
67 67​
68In order to run any tasks locally, libtaskotron must first be installed. While 68​In order to run any tasks locally, libtaskotron must first be installed. While
69not currently in the main Fedora repositories, it is available in a side repo 69​not currently in the main Fedora repositories, it is available in a side repo
70which is updated as new versions are released. To enable this repo:: 70​which is updated as new versions are released. To enable this repo::
71 71​
72 sudo wget http://copr-fe.cloud.fedoraproject.org/coprs/tflink/taskotron/repo/fedora-20-i386/tflink-taskotron-fedora-20-i386.repo -O /etc/yum.repos.d/tflink-taskotron-fedora-20-i386.repo 72​ sudo wget http://copr-fe.cloud.fedoraproject.org/coprs/tflink/taskotron/repo/fedora-21/tflink-taskotron-fedora-21.repo -O /etc/yum.repos.d/tflink-taskotron-fedora-21.repo
73 73​
74.. TODO - replace this link with a non-copr repo that's more stable over time 74​.. TODO - replace this link with a non-copr repo that's more stable over time
75 75​
76Once the repository is enabled, install libtaskotron with:: 76​Once the repository is enabled, install libtaskotron with::
77 77​
78 sudo yum install libtaskotron 78​ sudo yum install libtaskotron
79 79​
80Note: The permissions on several notable directories (e.g. ``/var/log/taskotron``, 80​Note: The permissions on several notable directories (e.g. ``/var/log/taskotron``,
▲ Show 20 Lines • Show All 45 Lines • Show Last 20 Lines
View Options

readme.rst

Show All 24 Lines
25==================================== 25​====================================
26 26​
27For the moment, libtaskotron can't be fully installed by either pip or rpm and 27​For the moment, libtaskotron can't be fully installed by either pip or rpm and
28needs a bit of both for now. 28​needs a bit of both for now.
29 29​
30Some of the packages required for using libtaskotron are not yet available from 30​Some of the packages required for using libtaskotron are not yet available from
31standard Fedora repos and the taskotron-copr repo is required:: 31​standard Fedora repos and the taskotron-copr repo is required::
32 32​
33 sudo wget http://copr-fe.cloud.fedoraproject.org/coprs/tflink/taskotron/repo/fedora-20-i386/tflink-taskotron-fedora-20-i386.repo -O /etc/yum.repos.d/tflink-taskotron-fedora-20-i386.repo 33​ sudo wget http://copr-fe.cloud.fedoraproject.org/coprs/tflink/taskotron/repo/fedora-21/tflink-taskotron-fedora-21.repo -O /etc/yum.repos.d/tflink-taskotron-fedora-21.repo
34 34​
35Once the copr repo is available, install the necessary packages:: 35​Once the copr repo is available, install the necessary packages::
36 36​
37 koji 37​ koji
38 rpm-python 38​ rpm-python
39 pyOpenSSL 39​ pyOpenSSL
40 gcc 40​ gcc
41 python-pycurl 41​ python-pycurl
▲ Show 20 Lines • Show All 126 Lines • Show Last 20 Lines