Skip to Content.
Sympa Menu

devel - Re: [sympa-developpers] [sympa-commits] sympa[10828] trunk/t/Language.t: [dev] no need to re-add standard @INC, but relative path for src/lib helps executing the tests manually

Subject: Developers of Sympa

List archive

Chronological Thread  
  • From: IKEDA Soji <address@concealed>
  • To: address@concealed
  • Subject: Re: [sympa-developpers] [sympa-commits] sympa[10828] trunk/t/Language.t: [dev] no need to re-add standard @INC, but relative path for src/lib helps executing the tests manually
  • Date: Fri, 6 Jun 2014 23:26:09 +0900

Guillaume,

On Fri, 06 Jun 2014 15:07:30 +0200
Guillaume Rousse <address@concealed> wrote:

> Le 06/06/2014 14:15, IKEDA Soji a écrit :
> > I suppose we may not have to add any of these:
> > executable bit, sh-bang, FindBin and -I option for child processes.
> And you're perfectly right, none of those elements are mandatory. They
> are just useful for some people (me, at least), with a near 0 additional
> complexity cost.
>
> David, Etienne, do you have any opinion about this fundamental issue
> before this turn into yet another holy war ?

"Holy war" has already come! I'm removing FindBin etc. on
sympa-6.2-branch repeatedly and you are adding it on trunk. That's
why I brought this issue up to the list.

I wish to hear everyone else, too.

> For a short summary:
> - shellbang + executable bits allows to run the test scripts as
> 't/Language.t' instead of 'perl t/Language.t', at the cost of one
> additional line of code and a file permission bit.
>
> - Findbin usage allows to run the test scripts as 'perl t/Language.t'
> instead of 'perl -I src/lib t/Language.t' or 'PERL5LIB=src/lib
> t/Language.t', at the cost of two additional line of code.


Regards,

--- Soji

--
株式会社 コンバージョン セキュリティ&OSSソリューション部 池田荘児
〒231-0004 神奈川県横浜市中区元浜町3-21-2 ヘリオス関内ビル7F
e-mail address@concealed TEL 045-640-3550
http://www.conversion.co.jp/




Archive powered by MHonArc 2.6.19+.

Top of Page