diff options
Diffstat (limited to 'zarb-ml/mageia-dev/2012-December/020691.html')
| -rw-r--r-- | zarb-ml/mageia-dev/2012-December/020691.html | 123 |
1 files changed, 123 insertions, 0 deletions
diff --git a/zarb-ml/mageia-dev/2012-December/020691.html b/zarb-ml/mageia-dev/2012-December/020691.html new file mode 100644 index 000000000..d717e1d84 --- /dev/null +++ b/zarb-ml/mageia-dev/2012-December/020691.html @@ -0,0 +1,123 @@ +<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN"> +<HTML> + <HEAD> + <TITLE> [Mageia-dev] [soft-commits] [6620] obsoleted by pod-syntax.t + </TITLE> + <LINK REL="Index" HREF="index.html" > + <LINK REL="made" HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20%5Bsoft-commits%5D%20%5B6620%5D%20obsoleted%20by%20pod-syntax.t&In-Reply-To=%3CCAONrEtbyh%2BYY1eqDn9jFGDMf%3DG7WErYj9479qfWpCCcNG7cykQ%40mail.gmail.com%3E"> + <META NAME="robots" CONTENT="index,nofollow"> + <META http-equiv="Content-Type" content="text/html; charset=us-ascii"> + <LINK REL="Previous" HREF="020690.html"> + <LINK REL="Next" HREF="020692.html"> + </HEAD> + <BODY BGCOLOR="#ffffff"> + <H1>[Mageia-dev] [soft-commits] [6620] obsoleted by pod-syntax.t</H1> + <B>Thierry Vignaud</B> + <A HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20%5Bsoft-commits%5D%20%5B6620%5D%20obsoleted%20by%20pod-syntax.t&In-Reply-To=%3CCAONrEtbyh%2BYY1eqDn9jFGDMf%3DG7WErYj9479qfWpCCcNG7cykQ%40mail.gmail.com%3E" + TITLE="[Mageia-dev] [soft-commits] [6620] obsoleted by pod-syntax.t">thierry.vignaud at gmail.com + </A><BR> + <I>Sun Dec 9 17:09:21 CET 2012</I> + <P><UL> + <LI>Previous message: <A HREF="020690.html">[Mageia-dev] [soft-commits] [6620] obsoleted by pod-syntax.t +</A></li> + <LI>Next message: <A HREF="020692.html">[Mageia-dev] [soft-commits] [6620] obsoleted by pod-syntax.t +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#20691">[ date ]</a> + <a href="thread.html#20691">[ thread ]</a> + <a href="subject.html#20691">[ subject ]</a> + <a href="author.html#20691">[ author ]</a> + </LI> + </UL> + <HR> +<!--beginarticle--> +<PRE>On 9 December 2012 17:04, Guillaume Rousse <<A HREF="https://www.mageia.org/mailman/listinfo/mageia-dev">guillomovitch at gmail.com</A>> wrote: +>><i> Yours does nothing: +</I>>><i> t/pod-coverage.t ................................................ +</I>>><i> skipped: Author test, set $ENV{TEST_AUTHOR} to a true value to run +</I>>><i> t/pod-spelling.t ................................................ +</I>>><i> skipped: Author test, set $ENV{TEST_AUTHOR} to a true value to run +</I>>><i> t/pod-syntax.t .................................................. +</I>>><i> skipped: Author test, set $ENV{TEST_AUTHOR} to a true value to run +</I>><i> +</I>><i> They work if you set the variable, as explicitely reported. That's perfectly +</I>><i> useless to run those tests anywhere else as on developper machine, and +</I>><i> especially during package build. And that's a standard practice in perl +</I>><i> community. +</I> +No, I want pod errors to be found prior submitting them. +And I don't want to manually run several tests. +Before all I had to do was to run "make test" and it reported me +any errors before commiting. +Now it doesn't anymore. + +><i> And your claim are quite unfair, given than the previous test didn't +</I>><i> prevented trivial syntax errors to be commited. +</I> +Yes it did. +I provided you the "make test" log proving it. +Yours are not run anymore when running "make test". +What I did fail to mention is that they don't even work with that varible: +[<A HREF="https://www.mageia.org/mailman/listinfo/mageia-dev">root at localhost</A> t]# TEST_AUTHOR=test perl pod-coverage.t +You said to run 0 tests at pod-coverage.t line 21. + +You don't even have tried them, did you? + +><i> Feel free to revert those changes, this kind of discussion isn't worth the +</I>><i> pain. +</I> +Well, if that mean less quality testsuite and if you refuse to answer reviewing, +I'll eventually revert those or at least put back the old working pod test. +</PRE> + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +<!--endarticle--> + <HR> + <P><UL> + <!--threads--> + <LI>Previous message: <A HREF="020690.html">[Mageia-dev] [soft-commits] [6620] obsoleted by pod-syntax.t +</A></li> + <LI>Next message: <A HREF="020692.html">[Mageia-dev] [soft-commits] [6620] obsoleted by pod-syntax.t +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#20691">[ date ]</a> + <a href="thread.html#20691">[ thread ]</a> + <a href="subject.html#20691">[ subject ]</a> + <a href="author.html#20691">[ author ]</a> + </LI> + </UL> + +<hr> +<a href="https://www.mageia.org/mailman/listinfo/mageia-dev">More information about the Mageia-dev +mailing list</a><br> +</body></html> |