ProfilesCheck: make.defaults checks#525
Conversation
Related: pkgcore#524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
Related: pkgcore#524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
Related: pkgcore#524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
Related: pkgcore#524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
Related: pkgcore#524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
Codecov ReportBase: 80.99% // Head: 80.92% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #525 +/- ##
==========================================
- Coverage 80.99% 80.92% -0.08%
==========================================
Files 56 56
Lines 8141 8194 +53
Branches 1826 1842 +16
==========================================
+ Hits 6594 6631 +37
- Misses 1448 1464 +16
Partials 99 99
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
| self.search_repo = self.options.search_repo | ||
| self.profiles_dir = repo.config.profiles_base | ||
| self.use_expand_groups = frozenset(x.upper() for x in repo.config.use_expand_desc) | ||
| self.use_expand_groups = { |
There was a problem hiding this comment.
Any thoughts on moving the concept of use_expand groups into pkgcore itself? We've got multiple spots that are now hardcoding how to split them. Seems like something where an object representation might be of value.
There was a problem hiding this comment.
I will create a shared impl in pkgcore, but for now I don't won't to be blocked until pkgcore release, so for now duplication of code, until I introduce the new shared code.
There was a problem hiding this comment.
No worries. My comment is more an observation I did something similar in portage_conf for use_expand, so it's probably time one of us cuts a class to hold the rules for it. ;)
Resolves: #524
PMS
A big cluster of the various checks, and results for make.defaults in profiles
Totally it added this results:
33 results in ::gentoo tree
Tests are included for all the changes here. Just a quick review of English is needed.