Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 646520 - app-portage/portpeek-2.1.25: portage.exception.InvalidAtom: dev-texlive/*
Summary: app-portage/portpeek-2.1.25: portage.exception.InvalidAtom: dev-texlive/*
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Mike Pagano
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-02-03 21:57 UTC by Erik Quaeghebeur
Modified: 2018-02-14 13:31 UTC (History)
1 user (show)

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


Attachments
console input and output with error (pp.txt,6.44 KB, text/plain)
2018-02-12 21:14 UTC, Erik Quaeghebeur
Details
package.use file containing the problematic line (other,986 bytes, text/plain)
2018-02-12 21:20 UTC, Erik Quaeghebeur
Details
other package.use file present in /etc/portage/package.use (required,1.81 KB, text/plain)
2018-02-12 21:21 UTC, Erik Quaeghebeur
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Quaeghebeur 2018-02-03 21:57:56 UTC
portpeek chokes on atoms of the form “dev-texlive/*”, which it calls invalid. I use such atoms successfully in package.use as, e.g., “dev-texlive/* doc”.
Comment 1 Mike Pagano gentoo-dev 2018-02-08 18:35:59 UTC
I don't see doc being a valid use flag for dev-texlive/*
Comment 2 Erik Quaeghebeur 2018-02-08 20:25:00 UTC
(In reply to Mike Pagano from comment #1)
> I don't see doc being a valid use flag for dev-texlive/*
Do you mean that it is not allowed according to some standard? As far as I understand from the manpage, ‘dev-texlive/*’ is a valid atom. Practically speaking it just works. Luckily so, as it'd be extremely bothersome to add the useflag to all packages individually.
Comment 3 Mike Pagano gentoo-dev 2018-02-08 23:45:11 UTC
(In reply to Erik Quaeghebeur from comment #2)
> (In reply to Mike Pagano from comment #1)
> > I don't see doc being a valid use flag for dev-texlive/*
> Do you mean that it is not allowed according to some standard? As far as I
> understand from the manpage, ‘dev-texlive/*’ is a valid atom. Practically
> speaking it just works. Luckily so, as it'd be extremely bothersome to add
> the useflag to all packages individually.

Not what I mean.  I don't see 'doc' as a valid use flag of dev-texlive/*

Not sure how to write that any differently.
Comment 4 Erik Quaeghebeur 2018-02-09 07:36:52 UTC
(In reply to Mike Pagano from comment #3)
> I don't see 'doc' as a valid use flag of dev-texlive/*
I then assume because in general not all packages in categories will have the same use flag(s) available? (In this particular case there is 1 out of 47 that does not have ‘doc’.)

If that is why, then still the *atom* itself is not invalid, but the package.use line is. Perhaps portpeek could report that it doesn't investigate such lines instead of raising an exception? Or is there a valid way to express the same thing without having to list all packages covered by the atom separately?

[Strange thing is: I have two installations, both with such a line in package.use; for one I get the exception, for the other portpeek does not complain.]
Comment 5 Mike Pagano gentoo-dev 2018-02-09 14:17:14 UTC
I'm not convinced yet there is anything wrong yet.

Give me the exact output you get and attach both package.use flags.

Here's it working:

$ cat package.use

sys-kernel/* experimental

$ portpeek -s
                                                                                                                                                                
package.use:                                                                                                                                                    
Done                                  

Now compare:

$ cat package.use

sys-kernel/* not_a_useflag_that_is_used_in_any_ebuild_in_sys-kernel_category   

portpeek -s
                                                                                                                                                                
package.use:                                                                                                                                                    
use flag: not_a_useflag_that_is_used_in_any_ebuild_in_sys-kernel_category is invalid for : sys-kernel/gentoo-sources-4.14.14
Done
Comment 6 Erik Quaeghebeur 2018-02-09 22:42:51 UTC
(In reply to Mike Pagano from comment #5)
> Give me the exact output you get and attach both package.use flags.
I will get back with more info (hopefully) once I get back to the system that gave the error.
Comment 7 Erik Quaeghebeur 2018-02-12 21:14:50 UTC
Created attachment 519278 [details]
console input and output with error
Comment 8 Erik Quaeghebeur 2018-02-12 21:20:31 UTC
Created attachment 519280 [details]
package.use file containing the problematic line
Comment 9 Erik Quaeghebeur 2018-02-12 21:21:35 UTC
Created attachment 519282 [details]
other package.use file present in /etc/portage/package.use
Comment 10 Mike Pagano gentoo-dev 2018-02-12 23:47:24 UTC
That helps. I'll take a look.
Comment 11 Mike Pagano gentoo-dev 2018-02-13 16:56:07 UTC
I'm going to release a masked version 2.1.26 soon. Please test and confirm.

FYI, in the future, do not modify the STATUS dropdown box.

I use it for my workflow and I need to decide if something is confirmed (I looked at it, I ack it, it's in my backlog), vs someone else deciding for me.
This may vary with other developers, but I ask that for my bugs.
Comment 12 Erik Quaeghebeur 2018-02-14 09:32:14 UTC
(In reply to Mike Pagano from comment #11)
> I'm going to release a masked version 2.1.26 soon. Please test and confirm.
Tested; fixes the issue. I now get "No installed packages found for dev-texlive/*.", which is true. Thanks!

> FYI, in the future, do not modify the STATUS dropdown box.
Yes, I'll take it as a safe general rule not to mess with those. Sorry for not having thought about that.
Comment 13 Mike Pagano gentoo-dev 2018-02-14 13:31:23 UTC
(In reply to Erik Quaeghebeur from comment #12)


Great. Thanks, appreciate your update and response. Have a good day.