Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 438506 - qa-scripts: packages-inheriting-eclasses.py failing due to missing ['_eclasses_'] in some packages
Summary: qa-scripts: packages-inheriting-eclasses.py failing due to missing ['_eclasse...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Infrastructure
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2012-10-15 19:49 UTC by Michał Górny
Modified: 2018-02-04 13:19 UTC (History)
1 user (show)

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


Attachments
Patch fixing the issue (0001-p-i-e.py-handle-missing-_eclasses_-list-correctly.patch,896 bytes, patch)
2012-10-15 19:49 UTC, Michał Górny
Details | Diff
A simpler patch ;) (0001-p-i-e.py-handle-missing-_eclasses_-correctly.patch,778 bytes, patch)
2012-10-17 11:30 UTC, Michał Górny
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2012-10-15 19:49:46 UTC
Created attachment 326640 [details, diff]
Patch fixing the issue

The p-i-e.py list is not updated for some time. It seems that lately pkgcore has changed to not to have any ['_eclasses_'] key whenever no eclass is being inherited.

Attaching a patch fixing it. Please push it to the repository as I don't have the write access. TIA.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2012-10-17 11:30:31 UTC
Created attachment 326764 [details, diff]
A simpler patch ;)
Comment 2 Dane Smith (RETIRED) gentoo-dev 2012-10-17 13:03:34 UTC
You're all set Michał. Thanks for the patch!
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2012-10-18 06:42:49 UTC
(In reply to comment #2)
> You're all set Michał. Thanks for the patch!

Err, something's wrong :P.

http://qa-reports.gentoo.org/output/eclass-usage/

gives 404.
Comment 4 Dane Smith (RETIRED) gentoo-dev 2012-10-19 14:27:06 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > You're all set Michał. Thanks for the patch!
> 
> Err, something's wrong :P.
> 
> http://qa-reports.gentoo.org/output/eclass-usage/
> 
> gives 404.

So it does. It's working fine for me locally, so I have no idea.

Infra: Any information or input here? Is this still running? If yes, can we by chance get a log? The README indicates it should be running daily, so we should see fixed output by now. Thanks!
Comment 5 Theo Chatzimichos (RETIRED) archtester gentoo-dev Security 2012-10-25 15:13:16 UTC
I ran it by hand and it seemed to be fixed. I'll double check our deployment though, maybe we messed up something during the migration of the services when the old box died
Comment 6 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2018-02-04 13:19:52 UTC
This is working for some time already.