Bug 1091640 - RFE: Release specific additional repos
Summary: RFE: Release specific additional repos
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Copr
Classification: Community
Component: backend
Version: unspecified
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
Assignee: Valentin Gologuzov
QA Contact:
URL:
Whiteboard:
: 1114792 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-04-26 20:54 UTC by Heiko Adams
Modified: 2015-12-01 06:04 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-03-05 16:15:58 UTC
Embargoed:


Attachments (Terms of Use)
possible fix: add backend expanded variables to url syntaxe (2.54 KB, patch)
2014-12-19 10:25 UTC, Pavel Raiskup
no flags Details | Diff

Description Heiko Adams 2014-04-26 20:54:24 UTC
I've a project that requires the Gnome 3.12 COPR of rhughes and those packages should be build for Fedora 19,20,Rawhide and EPEL-7 but if I add
> http://copr-be.cloud.fedoraproject.org/results/rhughes/f$releasever-gnome-3-12/fedora-$releasever-$basearch/
as additional required repo builds for EPEL-7 and RawHide are failing because of their $releasever variable content. So there should be a way to define those additional repos seperately for EPEL, Fedora-Releases and RawHide

Comment 1 Miroslav Suchý 2014-04-28 06:59:39 UTC
Since bug 1056039 releasever works for rawhide and epel (this is already in Copr).

The problematic part is "fedora" of "fedora-$releasever-$basearch" because such repo will not work for epel and vice versa.

Current workaround is to define one project for EPELs and another for Fedora. Albait that is sub-optimal.

Hmm I will think about it... Maybe whole "fedora-$releasever-$basearch" replace it by "$root". Or provide varibable which will expand to "fedora" or "epel" so we will have "$distribution-$releasever-$basearch".
Right now I prefer the first option, because that variable is already defined in mock config:
  config_opts['root'] = 'epel-6-x86_64'
so the change would be just tell mock to substitute this variable.

Comment 2 Miroslav Suchý 2014-08-03 20:13:02 UTC
*** Bug 1114792 has been marked as a duplicate of this bug. ***

Comment 3 Milan Bouchet-Valat 2014-10-04 17:05:00 UTC
I'd find this useful too.

Comment 4 Pavel Raiskup 2014-12-19 10:25:58 UTC
Created attachment 971075 [details]
possible fix: add backend expanded variables to url syntaxe

Consider we have chroot == 'epel-7-x86_64'.  Than the attached patch is adding
$chroot variable (expanded to full chroot id, that is 'epel-7-x86_64'),
$distid (expanded to 'epel' only) and $distver (expanded to '7').

Comment 5 Valentin Gologuzov 2015-02-18 15:18:35 UTC
I'm marking this bug as duplicate of https://bugzilla.redhat.com/show_bug.cgi?id=1119300 since primary use case is the same, and it provides simpler solution for user.

*** This bug has been marked as a duplicate of bug 1119300 ***

Comment 6 Pavel Raiskup 2015-02-18 16:25:04 UTC
Thanks a lot for your fix.  Without testing, it will resolve my actual issues!

I am reopening this bug because it is not completely the same RFE, decreasing
priority..

By implementing of $distid/$distver, or something similiar, you could still
enhance this feature a bit;  you may e.g. link your Copr buildroot with
external non-copr-hosted repositories which do not exactly match the
directory structure DISTID-DISTVERSION-ARCH.

Comment 7 Valentin Gologuzov 2015-02-18 23:47:11 UTC
I've think a bit and implemented variable expansion. Changes in commit 490c4b0c92. It was actually just a few lines. On the other hand /edit pages becomes cumbersome with newly added features and help labels (.
Update was deployed to the dev instance.

Comment 8 Pavel Raiskup 2015-02-19 07:21:36 UTC
Perfect, thank you!  Tested dev instance and it works as documented.

(In reply to Valentin Gologuzov from comment #7)
> On the other hand /edit pages
> becomes cumbersome with newly added features and help labels (.

I don't think so.  All content is fine, usable, needed.  Its rarely visited
page anyway.

Comment 9 Valentin Gologuzov 2015-03-05 16:15:58 UTC
Deployed into the fedora cloud
Version:
 copr-backend  1.58-1
 copr-frontend 1.55-1


Note You need to log in before you can comment on or make changes to this bug.