Re: DBD::Oracle v1.39_00 is out

2012-02-27 Thread Yanick Champoux

On 02/27/12 02:02, H.Merijn Brand wrote:

On Fri, 24 Feb 2012 16:50:37 -0500, Yanick Champoux
champ...@pythian.com  wrote:

[OTHERS]
- change the shebang line of examples to the more modern '/usr/bin/env
  perl'

Personally I really really hate this change


Fair enough.  This being said, I have to point out that the change 
is only on the scripts in the 'examples/' directory, which are not 
installed as part of the distro.




Perl has been configured to use $Config{startperl}. Use it!

I'm not sure I understand how you would like me to use it.  I know 
that, if we ask nicely, MakeMaker and Module::Build will change the  shebang of 
scripts that are to be installed to the local location of perl, but typically 
things that are not to be installed are left alone. Do you propose that the 
example directory get munged as part of the 'perl Makefile.PL' stage? Or do you 
know of a static shebang that goes straight for $Config{startperl} ?



Joy,
`/anick


--
Yanick Champoux, Senior Perl Developer
The Pythian Group - love your data
http://www.pythian.com


--
--
Pythian proud winner of Oracle North America Titan Award for Exadata 
Solution...watch the video on pythian.com


Re: DBD::Oracle v1.39_00 is out

2012-02-27 Thread H.Merijn Brand
On Mon, 27 Feb 2012 09:57:33 -0500, Yanick Champoux
champ...@pythian.com wrote:

 On 02/27/12 02:02, H.Merijn Brand wrote:
  On Fri, 24 Feb 2012 16:50:37 -0500, Yanick Champoux
  champ...@pythian.com  wrote:
 
   [OTHERS]
- change the shebang line of examples to the more modern '/usr/bin/env 
   perl'
 
  Personally I really really hate this change
 
 Fair enough.  This being said, I have to point out that the change 
 is only on the scripts in the 'examples/' directory, which are not 
 installed as part of the distro.

Understood, but IMHO we/you should not/never promote that usage

  Perl has been configured to use $Config{startperl}. Use it!

 I'm not sure I understand how you would like me to use it. I know that,
 if we ask nicely, MakeMaker and Module::Build will change the shebang
 of scripts that are to be installed to the local location of perl, but
 typically things that are not to be installed are left alone.

In which case I wonder why this change was needed in the first place

 Do you propose that the example directory get munged as part of the
 'perl Makefile.PL' stage?

No, only if these examples are installed as option from Makefile.PL
(many modules allow that)

 Or do you know of a static shebang that goes straight for
 $Config{startperl} ?

No. Maybe #!your_path_to_perl_here
would be the best option :)

Maybe I sounded too harsh in my previous mail, but in the end, that
ticket should just have been marked rejected with exactly the
explanation you just wrote here. If it does not install those scripts
from examples, *ANY* shebang line is valid

-- 
H.Merijn Brand  http://tux.nl   Perl Monger  http://amsterdam.pm.org/
using perl5.00307 .. 5.14   porting perl5 on HP-UX, AIX, and openSUSE
http://mirrors.develooper.com/hpux/http://www.test-smoke.org/
http://qa.perl.org   http://www.goldmark.org/jeff/stupid-disclaimers/


Re: DBD::Oracle v1.39_00 is out

2012-02-26 Thread Yanick Champoux


A new DBD::Oracle trial version is out and on its way to CPAN.

It's a maintenance release with fairly minor bug fixes (full 
changelog below).  As usual, the trial version will be soaked for 2 
weeks and, pending issues, will be promoted to v1.40 for general 
consumption.


Joy,
`/anick


1.39_00   2012-02-24

 [BUG FIXES]
 - TAF supports now conditional to presence of OCI_ATTR_TAF_ENABLED
[RT73798]
 - detect broken Win32::TieRegistry (patch by Rafael Kitover (Caelum))
[RT74544]
 - PL/SQL out values were not utf8 encoded [RT74753]
 (Steve Baldwin + Martin J. Evans)

 [DOCUMENTATION]
 - Mention the release of Oracle Instant Client 64 bit which does not work
on Lion. (Martin J. Evans)
 - fix DBD::Oracle::GetInfo blurb (patch by Julián Moreno Patiño) [rt74000]
 - fix typos. (patch by Julián Moreno Patiño) [rt73999]
 - add troubleshoot doc and diag for error with bequeather. [rt75263]

 [OTHERS]
 - change the shebang line of examples to the more modern '/usr/bin/env 
perl'

[RT74001]



--
--
Pythian proud winner of Oracle North America Titan Award for Exadata 
Solution...watch the video on pythian.com


Re: DBD::Oracle v1.39_00 is out

2012-02-26 Thread H.Merijn Brand
On Fri, 24 Feb 2012 16:50:37 -0500, Yanick Champoux
champ...@pythian.com wrote:

   [OTHERS]
   - change the shebang line of examples to the more modern '/usr/bin/env 
 perl'
  [RT74001]

Personally I really really hate this change

App::Ack was the first I noted to make this horrid change and I always
revert the shebang line to reflect my production perl

I (very) often work on other perls than the one that is installed and
to make testing these with modules (like DBI, DBD::Oracle and many many
more, I temporary change my $PATH to have the perl I am testing first

In that perl, App::Ack is NOT (yet) installed.

For some/many scripts, it doesn't really matter if the script is
installed or not, but for all scripts that invoke modules it *does*
matter.

I certainly do NOT see this change as more modern

Perl has been configured to use $Config{startperl}. Use it!

$ perl -V:startperl
startperl='#!/pro/bin/perl';


-- 
H.Merijn Brand  http://tux.nl   Perl Monger  http://amsterdam.pm.org/
using perl5.00307 .. 5.14   porting perl5 on HP-UX, AIX, and openSUSE
http://mirrors.develooper.com/hpux/http://www.test-smoke.org/
http://qa.perl.org   http://www.goldmark.org/jeff/stupid-disclaimers/