summaryrefslogtreecommitdiffstats
path: root/zarb-ml/mageia-dev/20110511/004620.html
diff options
context:
space:
mode:
Diffstat (limited to 'zarb-ml/mageia-dev/20110511/004620.html')
-rw-r--r--zarb-ml/mageia-dev/20110511/004620.html112
1 files changed, 112 insertions, 0 deletions
diff --git a/zarb-ml/mageia-dev/20110511/004620.html b/zarb-ml/mageia-dev/20110511/004620.html
new file mode 100644
index 000000000..7ef0e2efe
--- /dev/null
+++ b/zarb-ml/mageia-dev/20110511/004620.html
@@ -0,0 +1,112 @@
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
+<HTML>
+ <HEAD>
+ <TITLE> [Mageia-dev] [1266] add missing function ( and no titi, perl_checker didn' t notice this one, but pterjan checker did )
+ </TITLE>
+ <LINK REL="Index" HREF="index.html" >
+ <LINK REL="made" HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20%5B1266%5D%20add%20missing%20function%20%28%20and%20no%20titi%2C%0A%20perl_checker%20didn%27%20t%20notice%20this%20one%2C%20but%20pterjan%20checker%20did%20%29&In-Reply-To=%3CBANLkTikhnh9EPqYNKjWZ7z91dFUbwqJE0Q%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="004615.html">
+ <LINK REL="Next" HREF="004622.html">
+ </HEAD>
+ <BODY BGCOLOR="#ffffff">
+ <H1>[Mageia-dev] [1266] add missing function ( and no titi, perl_checker didn' t notice this one, but pterjan checker did )</H1>
+ <B>Thierry Vignaud</B>
+ <A HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20%5B1266%5D%20add%20missing%20function%20%28%20and%20no%20titi%2C%0A%20perl_checker%20didn%27%20t%20notice%20this%20one%2C%20but%20pterjan%20checker%20did%20%29&In-Reply-To=%3CBANLkTikhnh9EPqYNKjWZ7z91dFUbwqJE0Q%40mail.gmail.com%3E"
+ TITLE="[Mageia-dev] [1266] add missing function ( and no titi, perl_checker didn' t notice this one, but pterjan checker did )">thierry.vignaud at gmail.com
+ </A><BR>
+ <I>Wed May 11 10:26:58 CEST 2011</I>
+ <P><UL>
+ <LI>Previous message: <A HREF="004615.html">[Mageia-dev] [1266] add missing function ( and no titi, perl_checker didn' t notice this one, but pterjan checker did )
+</A></li>
+ <LI>Next message: <A HREF="004622.html">[Mageia-dev] [1266] add missing function ( and no titi, perl_checker didn' t notice this one, but pterjan checker did )
+</A></li>
+ <LI> <B>Messages sorted by:</B>
+ <a href="date.html#4620">[ date ]</a>
+ <a href="thread.html#4620">[ thread ]</a>
+ <a href="subject.html#4620">[ subject ]</a>
+ <a href="author.html#4620">[ author ]</a>
+ </LI>
+ </UL>
+ <HR>
+<!--beginarticle-->
+<PRE>On 11 May 2011 07:28, Michael scherer &lt;<A HREF="https://www.mageia.org/mailman/listinfo/mageia-dev">misc at zarb.org</A>&gt; wrote:
+&gt;&gt;<i> &gt; add missing function ( and no titi, perl_checker didn't notice this one, but
+</I>&gt;&gt;<i> &gt; pterjan checker did )
+</I>&gt;&gt;<i>
+</I>&gt;&gt;<i> you lie:
+</I>&gt;<i>
+</I>&gt;<i> Or maybe I misued perl_checker :/
+</I>&gt;<i>
+</I>&gt;<i> How do you check, just perl_checker file ?
+</I>
+You didn't misued it but you are not used to it :-)
+So you failed to see at end of log that it was aborting
+due to some core perl modules not following
+proper perl_checker style.
+Thus it didn't completed its checks on the file.
+
+So I blacklisted some more modules:
+<A HREF="http://svnweb.mageia.org/soft/build_system/iurt/trunk/.perl_checker?r1=17&amp;r2=1272">http://svnweb.mageia.org/soft/build_system/iurt/trunk/.perl_checker?r1=17&amp;r2=1272</A>
+
+BTW, note that, in the previous years, I already run perl_checker
+on iurt and added comments about dubious stuff in the code...
+
+eg dead code:
+793 # FIXME: (tv) kill this dead code or use it!!
+794 my $_s = sub {
+795 if ($run{main}) {
+796 plog(&quot;dumping cache...&quot;);
+797 dump_cache_par(\%run);
+798 $Data::Dumper::Indent = 0;
+799 $Data::Dumper::Terse = 1;
+800 plog(&quot;Running environment:\n&quot;, Data::Dumper-&gt;Dump([\%run]), &quot;\n&quot;);
+801 plog(&quot;Configuration:\n&quot;, Data::Dumper-&gt;Dump([$config]), &quot;\n&quot;);
+802 }
+803 exit();
+804 };
+
+eg supposed to be doing stuff for arches but not actually using arch value
+(aka a look that is equivalent to one single assignment):
+1425 foreach my $arch (@{$config-&gt;{supported_arch}}) { #FIXME: (tv)
+this loop looks suspiciously broken
+1426 $ok &amp;&amp;= $cache-&gt;{queue}{$srpm};
+1427 }
+
+and the like
+</PRE>
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+<!--endarticle-->
+ <HR>
+ <P><UL>
+ <!--threads-->
+ <LI>Previous message: <A HREF="004615.html">[Mageia-dev] [1266] add missing function ( and no titi, perl_checker didn' t notice this one, but pterjan checker did )
+</A></li>
+ <LI>Next message: <A HREF="004622.html">[Mageia-dev] [1266] add missing function ( and no titi, perl_checker didn' t notice this one, but pterjan checker did )
+</A></li>
+ <LI> <B>Messages sorted by:</B>
+ <a href="date.html#4620">[ date ]</a>
+ <a href="thread.html#4620">[ thread ]</a>
+ <a href="subject.html#4620">[ subject ]</a>
+ <a href="author.html#4620">[ 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>