Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 695814

Summary: Kill 'simple filename' mirror fetching for SRC_URI
Product: Gentoo Hosted Projects Reporter: Michał Górny <mgorny>
Component: PMS/EAPIAssignee: PMS/EAPI <pms>
Status: RESOLVED FIXED    
Severity: normal CC: chewi, esigra
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=571594
https://bugs.gentoo.org/show_bug.cgi?id=371413
Whiteboard:
Package list:
Runtime testing required: ---

Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2019-09-28 18:13:15 UTC
8.2.10 SRC_URI has:

> If a simple filename rather than a full URI is provided, the package manager can only use mirrors to download the file.

This is weird, half-broken in Portage (requires specific configuration in make.conf to work) and we really, really prefer people using mirror://gentoo when we want that behavior.

Let's either kill that in a future EAPI, or retroactively.  Since pkgcheck checks for this, we know that the tree is clean (except for two ebuilds just committed by Chewi, that are going to be fixed soonish).
Comment 1 James Le Cuirot gentoo-dev 2019-09-28 19:59:16 UTC
As per our discussion, would it then follow that a simple filename would do what RESTRICT="fetch" does now but just for that file? I guess that would make the option obsolete. Does fetch restriction on a full URI ever make sense?
Comment 2 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2019-09-28 20:03:18 UTC
(In reply to James Le Cuirot from comment #1)
> As per our discussion, would it then follow that a simple filename would do
> what RESTRICT="fetch" does now but just for that file? I guess that would
> make the option obsolete. Does fetch restriction on a full URI ever make
> sense?

Since this can't be made working retroactively (because of Portage implementation), I think we'll just go directly for selective fetch restriction (#371413).
Comment 3 Ulrich Müller gentoo-dev 2019-12-16 13:25:41 UTC
IMHO we should ban this retroactively, not as part of EAPI 8. Especially, if the feature neither works in practice, nor is used in the tree.
Comment 4 Larry the Git Cow gentoo-dev 2019-12-30 14:37:07 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/pms.git/commit/?id=2f8a07cf6b41b8883ad751d0e67705ab3cd264d4

commit 2f8a07cf6b41b8883ad751d0e67705ab3cd264d4
Author:     Michał Górny <mgorny@gentoo.org>
AuthorDate: 2019-12-22 18:52:00 +0000
Commit:     Ulrich Müller <ulm@gentoo.org>
CommitDate: 2019-12-30 14:35:25 +0000

    ebuild-vars: Remove 'simple filename' mirror fetching
    
    The feature of using 'simple filename' to fetch from mirrors
    is ill-defined (which mirrors?).  The Portage implementation works only
    if GENTOO_MIRRORS are explicitly set.  It's not used in any ebuilds.
    Let's remove it retroactively from the specification.
    
    Bug: https://bugs.gentoo.org/695814
    Signed-off-by: Michał Górny <mgorny@gentoo.org>
    Signed-off-by: Ulrich Müller <ulm@gentoo.org>

 ebuild-vars.tex | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)