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

Bug#684220: RFS: tinysvm/0.09-1 [ITP] -- SVM trainer and classifier toolkit



Il 14/11/2012 21:01, Jakub Wilk ha scritto:
> I think README.Debian would be better renamed to README.source.

Renamed.

> Upstream provides tests. Tests love to be run at build time. :)

They should be happy now. :-)

> The test/*.sh files would be more reliable if they used "set -e".

Added set -e to test/*.sh

> You may also want to provide DEP-8 tests.

Are you saying this in general or are you specifically talking about tinysvm?
Are there any benefits in implementing DEP-8 tests replacing tests in the upstream build system? (I just quickly read the "current specification" document linked by
http://dep.debian.net/deps/dep8/) Or are they intended to be run during installation (I did not get the meaning of "as-installed").

Bests,
	Giulio.


Reply to: