Felix Lechner pushed to branch master at lintian / lintian


Commits:
07b8b3b0 by Felix Lechner at 2021-10-07T17:20:45-07:00
Bring systemd check closer to current code standards; split off one check.

Further work is needed to separate the check further. Index::Item will probably
gain some INI parsing capability with caching via lazy Moo attributes for use by
multiple checks.

Gbp-Dch: ignore

- - - - -
39071751 by Felix Lechner at 2021-10-07T20:18:31-07:00
Migrate 'setup' entry points in checks to lazy Moo attributes; merge 
'breakdown' into $type.

The 'installable' entry point now always follows 
'visit_control_files' and
'visit_installed_files' so it can be used instead of 
'breakdown_X_files'.

Likewise, 'source' always follows 'visit_orig_files' and 
'visit_patched_files',
but none of them are in this commit.

Lazy Moo attributes can easily replace the old 'setup_X_files' entry 
points.
One just has to be careful when issuing tags in the Moo attributes. It is
possible, but lazy attributes are often evaluated inside conditions. The tags
are then not always issued, and the surrounding conditionals are located far
away.

The last point triggered a partial rewrite of the systemd check, which can be
found in the previous commit.

Gbp-Dch: ignore

- - - - -
78022c9c by Felix Lechner at 2021-10-07T20:22:19-07:00
Drop old hooks from Lintian::Check; no longer used anywhere including 
pkg-js-tools.

The 'installable' entry point now always follows 
'visit_control_files' and
'visit_installed_files' so it can be used instead of 
'breakdown_X_files'.

Likewise, 'source' always follows 'visit_orig_files' and 
'visit_patched_files'.

Please use lazy Moo attributes in lieu of the old 'setup_X_files' entry 
points.

Just be careful when issuing tags in the Moo attributes. It is possible, but
lazy attributes are often evaluated inside conditions. The tags are then not
always issued, and the surrounding conditionals are located far away.

Gbp-Dch: ignore

- - - - -


30 changed files:

- lib/Lintian/Check.pm
- lib/Lintian/Check/Binaries/Prerequisites.pm
- lib/Lintian/Check/Binaries/Prerequisites/Numpy.pm
- lib/Lintian/Check/Binaries/Prerequisites/Perl.pm
- lib/Lintian/Check/Binaries/Prerequisites/Php.pm
- lib/Lintian/Check/Debian/LintianOverrides.pm
- lib/Lintian/Check/Desktop/X11.pm
- lib/Lintian/Check/Documentation.pm
- lib/Lintian/Check/Documentation/Manual.pm
- lib/Lintian/Check/Files/Architecture.pm
- lib/Lintian/Check/Files/Compressed/Gz.pm
- lib/Lintian/Check/Files/Devhelp.pm
- lib/Lintian/Check/Files/Duplicates.pm
- lib/Lintian/Check/Files/EmptyPackage.pm
- lib/Lintian/Check/Files/MultiArch.pm
- lib/Lintian/Check/Files/Permissions.pm
- lib/Lintian/Check/Files/SymbolicLinks/Broken.pm
- lib/Lintian/Check/HugeUsrShare.pm
- lib/Lintian/Check/Languages/Ocaml.pm
- lib/Lintian/Check/Languages/Perl.pm
- lib/Lintian/Check/Languages/Python.pm
- lib/Lintian/Check/Languages/R/Architecture.pm
- lib/Lintian/Check/Languages/R/SiteLibrary.pm
- lib/Lintian/Check/Libraries/Shared/MultiArch.pm
- lib/Lintian/Check/Libraries/Shared/Trigger/Ldconfig.pm
- + lib/Lintian/Check/MaintainerScripts/Systemctl.pm
- lib/Lintian/Check/Systemd.pm
- lib/Lintian/Check/Team/PkgJs/Deprecated.pm
- lib/Lintian/Check/Team/PkgPerl/XsAbi.pm
- profiles/debian/main.profile


The diff was not included because it is too large.


View it on GitLab: 
https://salsa.debian.org/lintian/lintian/-/compare/097c4c4453ca29ce03bf7d814504bb9312b11924...78022c9c57b61b4c76cbc7a1bb051a00712cf8b2

-- 
View it on GitLab: 
https://salsa.debian.org/lintian/lintian/-/compare/097c4c4453ca29ce03bf7d814504bb9312b11924...78022c9c57b61b4c76cbc7a1bb051a00712cf8b2
You're receiving this email because of your account on salsa.debian.org.


Reply via email to