[Piuparts-devel] Fwd: Re: RFC: feature/adequate-tags
Holger Levsen
holger at layer-acht.org
Thu May 30 14:27:39 UTC 2013
Hi Andreas,
thanks for the review!
On Donnerstag, 30. Mai 2013, Andreas Beckmann wrote:
> If I understand correctly, adequate produces "non-inheriting" errors (by
> testing only the requested set of packages), so there is no need to fail
> the test (and block other packages from being tested), given we properly
> report these as errors - and your branch will generate nice individual
> reports for pass/ logs.
Jup. That's why there is --fail-if-inadequate if you really want piuparts to
fail if "only" adequate fails.
> This does not work:
>
> - ("inadequate_error.tpl", "due to logfile containing inadequate
> errors"),
> + ("inadequate_exit_issue.tpl", "due to adequate exiting inadequately"),
>
> IIRC the _error.tpl vs. _issue.tpl suffix is currently a significant
> property used in piuparts-report.
Yup, I've changed this to exit_issue.tpl. (Re-fetch.)
> So maybe just restore inadequate_error.tpl for the time being to have a
> separate combined report of all adequate output in fail/ and friends -
> the tests din't fail because of adequate anyway so splitting them up for
> fail/ is not really helpful.
I think the current/latest branch (fc4207e) is ok.
cheers,
Holger
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 828 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.alioth.debian.org/pipermail/piuparts-devel/attachments/20130530/4845ea16/attachment-0001.pgp>
More information about the Piuparts-devel
mailing list