Re: Lintian as a static analysis framework

[Niels Thykier <niels@thykier.net>]

On 2011-07-09 11:30, Niels Thykier wrote:
> On 2011-07-08 18:08, Russ Allbery wrote:
>> Niels Thykier <niels@thykier.net> writes:
>> [...]
>>
> 
> So perhaps it is time to have a look at how the Lintian::Collect
> migration is going.  I manually checked each of our checks to estimate
> how we are doing.  The results are 9 are completely converted to
> Lintian::Collect, 13 still uses debfiles, control or/and unpacked and
> the remaining 17 accesses more than that.
>   I decided to group debfiles, control and unpacked using check
> separately, because the use of these directories are rather common.
>   For the remaining checks I listed which parts of the Lab (I noticed)
> they are currently accessing directly.
> 
> 
+++ old/mail
--- new/mail
@@ something, something@@
- Does not access anything from the lab directly (9):
+ Does not access anything from the lab directly (24):
  version-substvars, symlinks, standards-version, java, huge-usr-share,
  filename-length, duplicate-files, description, circular-deps
-
- Uses (only) the debfiles, control or/and unpacked dirs (13):
  watch-file, shared-libs[3], rules[1], nmu, manpages,
  info-files, files, etcfiles, debian-source-dir, debhelper,
  debconf, control-file, conffiles
+ po-debconf, patch-systems
</end>

done to here. :)

Also, turns out I missed some "common_data" users, so I used grep this
time.  The (remaining) checks using common_data are: info-files, menus
and scripts.

> Uses any parts of the lab (15):
>   scripts (unpacked, control-scripts),
>   ocaml (ar-info)
>   menus[2][3] (debfiles, doc-base, control)
>   menu-format (menu, unpacked)
>    - As unpacked is present, there is little point in using
>      menu-files.
>   md5sums (control, md5sums)
>    - Though Lintian::Collect already offers an md5sums, but it
>      is for the package control file, not the one we generate
>      as I recall.
>   init.d (init.d, control)
>   fields (fields)
>   debian-readme (README.Debian)
>   deb-format (*-errors, deb)
>   cruft (*-errors, diffstat, debfiles, unpacked)
>     - Technically, debfiles is redundant if unpacked is present.
>   copyright-file (source, copyright)
>     - The source part can be replaced with a look in $group for
>       the source or the package in question.
>   control-files (control-index)
>   changes-file (changes)
>     - Accesses files outside the lab (as in they are not copied
>       or symlinked into the lab as the changes file itself)
>   changelog-file[3] (NEWS.Debian, changelog)
>   binaries (strings)
> 
> [...]