[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#941947: RFS: misspell-fixer/0.1-1 [ITP] -- Tool for fixing common misspellings, typos in source code



On Wed, Jan 08, 2020 at 07:50:11PM +0000, Lajos Veres wrote:
> I have checked this page and it mentions some problems:
> https://qa.debian.org/debcheck.php?dist=unstable&package=misspell-fixer
> 
> I believe it would be easy to disable dh_auto_test and remove kcov
> from the dependencies.
> make test generates a coverage report with kcov, but otherwise kcov is
> not really neded.
> Then I guess the debian/tests/control could run the same tests. Although I
> am not really sure when.

This applies only to some architectures which have few users, it would be a
pity to lose tests for archs that are ok.

A plan maximum would be to make the kcov B-Dependency arch-dependant:
Build-Depends: kcov [amd64 arm64 armel armhf i386 ppc64el powerpc ppc64 x32]
with dh_auto_test having an:
if which kcov >/dev/null; then dh_auto_test; fi

but this requires a bit of testing on different archs.

> Does this make sense?
> (Does this problem has any importance at all?)

The package is arch:all, and no one really is going to build it on some
obscure architecture.  After all, the code has no compiled parts.

Thus, I'd just ignore the problem.

Debcheck has a load of other false positives anyway.


Meow!
-- 
⢀⣴⠾⠻⢶⣦⠀ A MAP07 (Dead Simple) raspberry tincture recipe: 0.5l 95% alcohol,
⣾⠁⢠⠒⠀⣿⡁ 1kg raspberries, 0.4kg sugar; put into a big jar for 1 month.
⢿⡄⠘⠷⠚⠋⠀ Filter out and throw away the fruits (can dump them into a cake,
⠈⠳⣄⠀⠀⠀⠀ etc), let the drink age at least 3-6 months.


Reply to: