Ticket #301 (closed defect: wontfix)

Opened 10 years ago

Last modified 8 years ago

Fix QA / qualudis

Reported by: ciaranm Owned by: ciaranm
Priority: MajorRelease Milestone:
Component: core/paludis Version: scm
Keywords: Cc:
Blocked By: Blocking:
Distribution: Gentoo

Description

Fix QA / qualudis.

Attachments

qa-checks.txt Download (2.0 KB) - added by dleverton 10 years ago.
comparison of old QA checks to new ones - not guaranteed complete or accurate

Change History

comment:1 Changed 10 years ago by ciaranm

  • Milestone changed from PackageID breakages to Before 0.26.0

comment:2 Changed 10 years ago by ciaranm

  • Blocking set to 445

comment:3 Changed 10 years ago by anonymous

  • Milestone Before 0.26.0 deleted

Milestone Before 0.26.0 deleted

comment:4 Changed 10 years ago by dleverton

Besides the missing checks (some of which may not be so useful), the old qualudis had options to enable or disable individual checks and/or individual arches, and displaying the maintainer and/or herd of affected packages.

comment:5 Changed 10 years ago by Philantrop

Checks I'd like to see ported because I found them useful:

defaults_check - granted, I should see things like redundant src_compile but I don't always. :) dep_flags_check - has some useful checks. ebuild_name_check - experience shows that this check is important even though it shouldn't be... files_dir_size_check - very useful to see if one should roll a patch tarball. gpg_check - Manifest signing is not (yet) mandatory but it was good to have this check. has_misc_files_check - this is, IMHO, a must-have as it's very basic QA. header_check - useful for beginners to check they got the headers right. iuse_check - this is, IMHO, a must-have as it's very basic QA. parse_deps_check - this is, IMHO, a must-have as it's very basic QA. restrict_check - again, basic QA. The no* values should be removed, though, as they aren't valid, AFAIK. variable_assigns_check - I still see this from time to time so I think it would be good to have this check.

Nice to have:

ebuild_count_check

The other checks not explicitly mentioned were useful sometimes, too. Thus, in an ideal world, they all would be ported for 0.26-final. :)

Changed 10 years ago by dleverton

comparison of old QA checks to new ones - not guaranteed complete or accurate

comment:6 follow-up: ↓ 7 Changed 10 years ago by ciaranm

What's still left?

comment:7 in reply to: ↑ 6 Changed 10 years ago by dleverton

Replying to ciaranm:

What's still left?

Some of the checks, the ability to run in a package or category directory, the ability to disable certain checks/arches/profiles, and reporting the maintainers/herds of the affected packages.

I could live with doing 0.26 without those (sorry Philantrop) but I will add them back sooner or later (maybe we could do a 0.26.x?)

comment:8 Changed 10 years ago by ciaranm

  • Blocking 445 deleted

comment:9 Changed 8 years ago by ciaranm

  • Status changed from new to closed
  • Distribution set to Gentoo
  • Resolution set to wontfix
Note: See TracTickets for help on using tickets.