<div>Matei a cobra, agora vou mostrar o pau.</div><div><br></div><div>2011/2/19 Eden Cardim <span dir="ltr"><<a href="mailto:edencardim@gmail.com">edencardim@gmail.com</a>></span></div><div class="gmail_quote"><div>[...] </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Observa que ele tá reclamando das violações na linha 3, então acho que o<br>
blib/lib/Queue/Base.pm está dessincronizado com esse código que você<br>
mostrou. Tenta re-executar o perlcritic passando o caminho explícito pro<br>
Queue::Base, aqui eu obtive o mesmo resultado do breno:<br></blockquote><div><br></div><div>Repeti aqui e, obtive o mesmo resultado que vocês. Foi aí que eu comecei a olhar para as coisas-que-não-deveriam-influenciar-nisso-mas-não-há-nenhum-outro-bode-para-levar-a-culpa. O primeiro release que eu fiz do Queue::Base foi basicamente o módulo do autor anterior com algumas pequenas alterações de código e eu troquei o Makefile.PL de MakeMaker para Module::Install.</div>
<div><br></div><div>Nos últimos dias, no entanto, eu comecei a fuçar no Dist::Zilla, e o Queue::Base foi a minha cobaia para isso. Na verdade, eu não coloquei o teste de perl critic no modulo. Eu coloquei a linha</div><div>
<br></div><div>[CriticTests]</div><div><br></div><div>no dist.ini, e ele fez o resto. Além disso, eu também estou com o</div><div><br></div><div>[PkgVersion]</div><div><br></div><div>que, and I quote: "add a $VERSION to your packages". O problema é que, o PkgVersion faz isso injetando o código:</div>
<div><br></div><meta http-equiv="content-type" content="text/html; charset=utf-8"><div>------------------ cut here ------------------</div><div><div>BEGIN {</div><div> $Queue::Base::VERSION = '2.1_02';</div><div>
}</div></div><div>------------------ cut here ------------------</div><div><br></div><div>logo após o statement *package* do módulo, e antes do strict e do warning, o que gera o erro no teste do Perl::Critic.</div><div><br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">>>>>> "breno" == breno <<a href="mailto:breno@rio.pm.org">breno@rio.pm.org</a>> writes:<br>
<br>
breno> "blib"?<br>
<br>
>>>>> "Thiago" == Thiago <<a href="mailto:thiagoglauco@ticursos.net">thiagoglauco@ticursos.net</a>> writes:<br>
<br>
Thiago> blib/lib é a pasta onde o Queue::Base esta instalada na maquina dele.<br>
<br>
Apesar de poder instalar o módulo num diretório arbitrariamente chamado<br>
blib se quiser, blib ("build lib") costuma ser o diretório onde os<br>
builders/installers (ExtUtils::MakeMaker, Module::Install,<br>
Module::Build) usam pra fazer o build/staging do módulo antes de<br>
instalar. Então é possível que a source tree e o blib do russo esteja<br>
dessincronizada.<br></blockquote><div><br></div><div>No caso, o blib é o diretório de build (como assim vocês não sabiam??? rsrsrs). Eden, dificilmente estaria dessincronizado nesse ponto, uma vez que o Makefile iria atualizar de acordo com os timestamps.</div>
<div><br></div><div>Logo, o problema é que, o código que é efetivamente gerado (pelo plugin do Dist::Zilla) não é Perl::Critic-friendly. Para reproduzir o problema, testem com o arquivo do CPAN, não com o do git:</div><div>
<br></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8"><div>------------------ cut here ------------------</div><div></div><div>12:08:47 BRST ~ $ cpan</div><div>cpan shell -- CPAN exploration and modules installation (v1.9402)</div>
<div>Enter 'h' for help.</div><div><br></div><div>cpan[1]> look Queue::Base</div><div>...</div><div>cpan[1]> look Queue::BaseCPAN: File::Temp loaded ok (v0.22)</div><div>...</div><div><br></div><div>12:09:45 BRST ~/.local/share/.cpan/build/Queue-Base-2.1-BBsUi1 $ perlcritic --brutal lib/Queue/Base.pm </div>
<div>...</div><div><br></div><div>Code before strictures are enabled at line 4, column 1. See page 429 of PBP. (Severity: 5)</div><div>Code before warnings are enabled at line 4, column 1. See page 431 of PBP. (Severity: 4)</div>
</div><div><meta http-equiv="content-type" content="text/html; charset=utf-8"><div>------------------ cut here ------------------</div><div></div></div><div><br></div><div><br></div></div><br>-- <br><font face="georgia, serif">Alexei Znamensky [russoz_gmail_com] [<a href="http://russoz.wordpress.com" target="_blank">russoz.wordpress.com</a>] [<a href="http://www.flickr.com/photos/alexeiz" target="_blank">www.flickr.com/photos/alexeiz</a>]<br>
<span style="border-collapse:collapse"><div>«Only love / Can bring the rain / That makes you yearn to the sky»</div></span></font><br>