Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 735134 - app-portage/portage-utils: odd (irrelevant) warning about invalid repo_name
Summary: app-portage/portage-utils: odd (irrelevant) warning about invalid repo_name
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Core - External Interaction (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Fabian Groffen
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-08-01 22:41 UTC by Rick Farina (Zero_Chaos)
Modified: 2023-07-20 06:15 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rick Farina (Zero_Chaos) gentoo-dev 2020-08-01 22:41:26 UTC
When I run scripts during my catalyst builds I often use qfile.  Lately it's been emitting an odd warning:

qfile: ignoring parent with unknown repo in profile: pentoo

Catalyst builds don't use repos.conf, and simply bind mount overlays and configure make.conf with PORTDIR_OVERLAY.  Is this warning meaningful in some way?  Is qfile unable to do it's job properly without repos.conf?

Please make the error more descriptive, or make it go away if it's safe to ignore.
Comment 1 Fabian Groffen gentoo-dev 2020-08-02 07:34:13 UTC
The warning is telling you a reference to repo pentoo was found, but no such repo is defined, hence it cannot load whatever it is pointing to.

In this case it is probably safe to ignore, because qfile doesn't do much with repo data.  I should try and make this more just-in-time loading instead.  Any invocation of q-tools should result in this warning.
Comment 2 Larry the Git Cow gentoo-dev 2020-08-02 07:39:13 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/portage-utils.git/commit/?id=61ac8ae9168b7262a6f68a07da312659181647a0

commit 61ac8ae9168b7262a6f68a07da312659181647a0
Author:     Fabian Groffen <grobian@gentoo.org>
AuthorDate: 2020-08-02 07:37:51 +0000
Commit:     Fabian Groffen <grobian@gentoo.org>
CommitDate: 2020-08-02 07:37:51 +0000

    read_portage_profile: adapt warning to point to which profile
    
    When we fail to resolve a repo match, give the user a better hint by
    pointing to the profile we were reading.
    
    Bug: https://bugs.gentoo.org/735134
    Signed-off-by: Fabian Groffen <grobian@gentoo.org>

 main.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Comment 3 Rick Farina (Zero_Chaos) gentoo-dev 2020-08-02 16:21:40 UTC
the point of my bug is more that I don't think qfile needs to be parsing the profile at all, does it?  So this warning is superfluous and confusing.  It makes it look like it isn't able to do it's job properly and I don't believe that to be the case.
Comment 4 Fabian Groffen gentoo-dev 2020-08-03 06:45:02 UTC
Yes, that's what my "just-in-time" comment was about.  Profiles are loaded as part of q startup (= any applet) for things like PORTDIR and Q_VDB, the latter which is used by qfile.  However, as you correctly point out, qfile (and other applets too) do not need full profiles to be parsed, and I should find a way to do this at a later point.
Comment 5 sfrolov 2023-03-27 18:46:40 UTC
Hitting the same warning in ChromeOS, creating concern among developers, so we would appreciate a fix/workaround to hide this warning.

`--quiet` flag promises to suppress warnings, but doesn't. Would it be easier to fix that?
Comment 6 Larry the Git Cow gentoo-dev 2023-04-21 19:11:10 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/portage-utils.git/commit/?id=b9592db1c04cafcdfaaff8f44ab2d6fc7d6dd5fa

commit b9592db1c04cafcdfaaff8f44ab2d6fc7d6dd5fa
Author:     Fabian Groffen <grobian@gentoo.org>
AuthorDate: 2023-04-21 19:10:23 +0000
Commit:     Fabian Groffen <grobian@gentoo.org>
CommitDate: 2023-04-21 19:10:23 +0000

    main: silence repo warnings when quiet is set
    
    Bug: https://bugs.gentoo.org/735134
    Signed-off-by: Fabian Groffen <grobian@gentoo.org>

 main.c | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)
Comment 7 Matt Turner gentoo-dev 2023-05-25 21:04:54 UTC
Fabian: would you mind tagging a release with this commit in it? It would help us close out this issue on ChromeOS.
Comment 8 Fabian Groffen gentoo-dev 2023-05-26 06:46:30 UTC
yes, of course
Comment 9 Larry the Git Cow gentoo-dev 2023-05-26 06:56:22 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1c077c1f373119e25e4522d7952e34283e3b5be2

commit 1c077c1f373119e25e4522d7952e34283e3b5be2
Author:     Fabian Groffen <grobian@gentoo.org>
AuthorDate: 2023-05-26 06:54:58 +0000
Commit:     Fabian Groffen <grobian@gentoo.org>
CommitDate: 2023-05-26 06:54:58 +0000

    dev-libs/libxml2: fixup for Solaris build-fix
    
    - add missing virtual/libiconv dependency
    - add note + bugref for Solaris workaround
    - remove workaround from live-ebuild, since a fix was applied upstream
    
    Closes: https://bugs.gentoo.org/735134
    Signed-off-by: Fabian Groffen <grobian@gentoo.org>

 dev-libs/libxml2/libxml2-2.11.4.ebuild | 3 +++
 dev-libs/libxml2/libxml2-9999.ebuild   | 5 +----
 2 files changed, 4 insertions(+), 4 deletions(-)
Comment 10 Fabian Groffen gentoo-dev 2023-05-26 06:57:42 UTC
wrong bug :(
Comment 11 Larry the Git Cow gentoo-dev 2023-05-26 08:11:50 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6108fcd3c56baf90d60c8fba516f14a2bb231339

commit 6108fcd3c56baf90d60c8fba516f14a2bb231339
Author:     Fabian Groffen <grobian@gentoo.org>
AuthorDate: 2023-05-26 08:11:03 +0000
Commit:     Fabian Groffen <grobian@gentoo.org>
CommitDate: 2023-05-26 08:11:47 +0000

    app-portage/portage-utils-0.96: version bump
    
    cleanup old overshadowed versions
    
    Closes: https://bugs.gentoo.org/735134
    Signed-off-by: Fabian Groffen <grobian@gentoo.org>

 app-portage/portage-utils/Manifest                 |  3 +-
 .../portage-utils/portage-utils-0.94.3.ebuild      | 75 ----------------------
 ...ils-0.94.4.ebuild => portage-utils-0.96.ebuild} |  0
 3 files changed, 1 insertion(+), 77 deletions(-)
Comment 12 sfrolov 2023-07-13 21:41:55 UTC
Hi Fabian, looks like this bug is still not resolved in 0.96.


$ qlist -q virtual/target-sdk
(null)qlist(null): ignoring parent with unknown repo in profile /mnt/host/source/src/third_party/chromiumos-overlay/profiles/targets/sdk: chromiumos:features/llvm
(null)qlist(null): ignoring parent with unknown repo in profile /mnt/host/source/src/third_party/chromiumos-overlay/profiles/targets/sdk: chromiumos:features/selinux

$ qlist --version
(null)qlist(null): ignoring parent with unknown repo in profile /mnt/host/source/src/third_party/chromiumos-overlay/profiles/targets/sdk: chromiumos:features/llvm
(null)qlist(null): ignoring parent with unknown repo in profile /mnt/host/source/src/third_party/chromiumos-overlay/profiles/targets/sdk: chromiumos:features/selinux
portage-utils-0.96
written for Gentoo by solar, vapier and grobian
Comment 13 Fabian Groffen gentoo-dev 2023-07-14 06:45:50 UTC
reopening
Comment 14 Fabian Groffen gentoo-dev 2023-07-17 17:29:01 UTC
ah, this uses quiet before it's actually handled: the profile code runs before handling arguments.
Comment 15 Larry the Git Cow gentoo-dev 2023-07-18 06:28:24 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/portage-utils.git/commit/?id=68e1d30bcd977396f1db41be800f0a80ff03467a

commit 68e1d30bcd977396f1db41be800f0a80ff03467a
Author:     Fabian Groffen <grobian@gentoo.org>
AuthorDate: 2023-07-18 06:26:52 +0000
Commit:     Fabian Groffen <grobian@gentoo.org>
CommitDate: 2023-07-18 06:26:52 +0000

    main: init quiet early for repo checks
    
    For as long as we will pre-load all profile information at every q
    invocation, respect -q flag so warnings in that code can be suppressed.
    
    Bug: https://bugs.gentoo.org/735134
    Signed-off-by: Fabian Groffen <grobian@gentoo.org>

 main.c | 31 +++++++++++++++++++++++--------
 1 file changed, 23 insertions(+), 8 deletions(-)
Comment 16 Fabian Groffen gentoo-dev 2023-07-18 06:29:16 UTC
I'd appreciate if you could try if this works for you.  Either try git, or apply changesets 360d18ba7cfbbd285f5ed6d97892778685b8a106 and 68e1d30bcd977396f1db41be800f0a80ff03467a.  Thanks
Comment 17 sfrolov 2023-07-18 22:41:43 UTC
I am afraid warnings are still there.

$ qlist --version
qlist: ignoring parent with unknown repo in profile /mnt/host/source/src/third_party/chromiumos-overlay/profiles/targets/sdk: chromiumos:features/llvm
qlist: ignoring parent with unknown repo in profile /mnt/host/source/src/third_party/chromiumos-overlay/profiles/targets/sdk: chromiumos:features/selinux
portage-utils-git
written for Gentoo by solar, vapier and grobian


I double-checked that the following commits are included:

68e1d30bcd977396f1db41be800f0a80ff03467a (HEAD)
Author: Fabian Groffen <grobian@gentoo.org>
Date:   Tue Jul 18 08:26:52 2023 +0200
    main: init quiet early for repo checks
commit 360d18ba7cfbbd285f5ed6d97892778685b8a106
Author: Fabian Groffen <grobian@gentoo.org>
Date:   Mon Jul 17 19:29:55 2023 +0200
    initialize_portage_env: init color and quiet defaults early
Comment 18 Fabian Groffen gentoo-dev 2023-07-19 06:12:50 UTC
yes, but the point was that that you could use -q / --quiet to silence that (comment #5) and that should work now
Comment 19 sfrolov 2023-07-19 16:31:29 UTC
Oh right, that does work. Let's land it

$ qlist -q --version
portage-utils-git
written for Gentoo by solar, vapier and grobian
Comment 20 Fabian Groffen gentoo-dev 2023-07-20 06:00:28 UTC
thanks for confirming
Comment 21 Larry the Git Cow gentoo-dev 2023-07-20 06:15:12 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d4803c05a6deeda9e91c7834a091b2d2dcdd876e

commit d4803c05a6deeda9e91c7834a091b2d2dcdd876e
Author:     Fabian Groffen <grobian@gentoo.org>
AuthorDate: 2023-07-20 06:14:30 +0000
Commit:     Fabian Groffen <grobian@gentoo.org>
CommitDate: 2023-07-20 06:14:30 +0000

    app-portage/portage-utils-0.96.1: version bump for bug #735134
    
    Closes: https://bugs.gentoo.org/735134
    Signed-off-by: Fabian Groffen <grobian@gentoo.org>

 app-portage/portage-utils/Manifest                 |  1 +
 ...ils-0.95.ebuild => portage-utils-0.96.1.ebuild} | 10 +--
 .../portage-utils/portage-utils-0.96.ebuild        | 75 ----------------------
 3 files changed, 6 insertions(+), 80 deletions(-)