Felix Lechner pushed to branch master at lintian / lintian


Commits:
a3a0961e by Felix Lechner at 2020-12-18T11:36:05-08:00
Move all checks into the module name space under ./lib; provide symbolic link.

Traditionally, the module names for checks in Lintian did not match
their location in the file system. In Perl, that leads to several
serious shortcomings that are remedied in this commit series.

Here, checks are moved to lib/Lintian/Check. A symbolic link will be
provided as a convenience in the next commit.

The move prevents modules from being absent due to typos. It also
ensures that all modules are loaded under the names we expect. Some of
these assertions are not a possible without the help of the perlcritic
policy Freenode::PackageMatchesFilename, which we hope to enable
shortly.

Together with the changes to follow, this may break pkg-js-tools and
pkg-perl-tools. The maintainers of those packages will be advised
separately, as has been our custom.

- - - - -


30 changed files:

- debian/lintian.install
- checks/Apache2.pm → lib/Lintian/Check/Apache2.pm
- checks/ApplicationNotLibrary.pm → lib/Lintian/Check/ApplicationNotLibrary.pm
- checks/AppstreamMetadata.pm → lib/Lintian/Check/AppstreamMetadata.pm
- checks/Apt.pm → lib/Lintian/Check/Apt.pm
- checks/Binaries.pm → lib/Lintian/Check/Binaries.pm
- checks/BuildSystems/Automake.pm → lib/Lintian/Check/BuildSystems/Automake.pm
- checks/BuildSystems/Cmake.pm → lib/Lintian/Check/BuildSystems/Cmake.pm
- checks/ChangesFile.pm → lib/Lintian/Check/ChangesFile.pm
- checks/Conffiles.pm → lib/Lintian/Check/Conffiles.pm
- checks/ContinuousIntegration/Salsa.pm → 
lib/Lintian/Check/ContinuousIntegration/Salsa.pm
- checks/ControlFiles.pm → lib/Lintian/Check/ControlFiles.pm
- checks/Cron.pm → lib/Lintian/Check/Cron.pm
- checks/Cruft.pm → lib/Lintian/Check/Cruft.pm
- checks/DebFormat.pm → lib/Lintian/Check/DebFormat.pm
- checks/Debhelper.pm → lib/Lintian/Check/Debhelper.pm
- checks/Debian/Changelog.pm → lib/Lintian/Check/Debian/Changelog.pm
- checks/Debian/Control.pm → lib/Lintian/Check/Debian/Control.pm
- checks/Debian/Copyright.pm → lib/Lintian/Check/Debian/Copyright.pm
- checks/Debian/Copyright/ApacheNotice.pm → 
lib/Lintian/Check/Debian/Copyright/ApacheNotice.pm
- checks/Debian/Copyright/Dep5.pm → lib/Lintian/Check/Debian/Copyright/Dep5.pm
- checks/Debian/Debconf.pm → lib/Lintian/Check/Debian/Debconf.pm
- checks/Debian/DesktopEntries.pm → lib/Lintian/Check/Debian/DesktopEntries.pm
- checks/Debian/Filenames.pm → lib/Lintian/Check/Debian/Filenames.pm
- checks/Debian/Files.pm → lib/Lintian/Check/Debian/Files.pm
- checks/Debian/LineSeparators.pm → lib/Lintian/Check/Debian/LineSeparators.pm
- checks/Debian/LintianOverrides.pm → 
lib/Lintian/Check/Debian/LintianOverrides.pm
- checks/Debian/LintianOverrides/Comments.pm → 
lib/Lintian/Check/Debian/LintianOverrides/Comments.pm
- checks/Debian/ManualPages.pm → lib/Lintian/Check/Debian/ManualPages.pm
- checks/Debian/NotInstalled.pm → lib/Lintian/Check/Debian/NotInstalled.pm


The diff was not included because it is too large.


View it on GitLab: 
https://salsa.debian.org/lintian/lintian/-/commit/a3a0961e2e132e502e4f12f7dcc6998b31b6659f

-- 
View it on GitLab: 
https://salsa.debian.org/lintian/lintian/-/commit/a3a0961e2e132e502e4f12f7dcc6998b31b6659f
You're receiving this email because of your account on salsa.debian.org.


Reply via email to