Compiler warnngs (was: ANNOUNCE DBI 1.618)

2012-02-26 Thread Tim Bunce
On Sat, Feb 25, 2012 at 02:18:56PM -0800, Jonathan Leffler wrote:
 
The compilation phase has a couple of warnings with the compiler set very 
 fussy...I doubt if they're
significant, but just in case they're fixable and fixworthy:
 
Perl.c: In function �XS_DBD__Perl__dr_data_sources�:
Perl.c:262:7: warning: variable �attr� set but not used 
 [-Wunused-but-set-variable]
Perl.c: In function �XS_DBD__Perl__db__login�:
Perl.c:306:7: warning: variable �attribs� set but not used 
 [-Wunused-but-set-variable]
Perl.c: In function �XS_DBD__Perl__db_last_insert_id�:
Perl.c:538:7: warning: variable �attr� set but not used 
 [-Wunused-but-set-variable]

Perl.xs is a weird kind of hack that exists just to do a basic sanity
check of Driver.xst. Does this change in Perl.xs silence the first
warning?:

-#define dbd_dr_data_sources(drh, imp_drh, attr) 
(drh=drh,imp_drh=imp_drh,Nullav)
+#define dbd_dr_data_sources(drh, imp_drh, attr) (drh=drh,imp_drh=imp_drh,attr)

DBI.xs: In function �XS_DBI_dispatch�:
DBI.xs:282:14: warning: �c� may be used uninitialized in this function 
 [-Wuninitialized]
DBI.xs:227:21: note: �c� was declared here

I think that one's not a problem (and Dave's has reworked the method
cache so it'll change for the next release).

Tim.


DBM/DBD::File/SQL::Statement failures (was: ANNOUNCE DBI 1.618)

2012-02-26 Thread Tim Bunce
On Sat, Feb 25, 2012 at 02:18:56PM -0800, Jonathan Leffler wrote:
On Sat, Feb 25, 2012 at 12:47, Tim Bunce [1]tim.bu...@pobox.com wrote:
 
DBI 1.618 installed fine for me on MacOS X 10.7.3 with Perl 5.14.1

Great.

I'm slightly puzzled by a couple of the tests skipping because 'Not 
 running with SQL::Statement', when
in fact I have SQL::Statement installed.
 
t/52dbm_complex.t ... ok
t/zvg_52dbm_complex.t ... ok
t/zvn_52dbm_complex.t ... skipped: Not running with SQL::Statement
t/zvp_52dbm_complex.t ... ok
t/zvxgn_52dbm_complex.t . skipped: Not running with SQL::Statement
t/zvxgnp_52dbm_complex.t  skipped: Not running with SQL::Statement
t/zvxgp_52dbm_complex.t . ok
t/zvxnp_52dbm_complex.t . skipped: Not running with SQL::Statement
 
Maybe it is the message that needs tweakiing?

Yes, they should be changed to requires SQL::Statement module.

Also cpantesters is reporting some failures relating to the DBM modules:

For example, these two:
http://www.cpantesters.org/cpan/report/ce8c15e4-601c-11e1-8e85-838032b1b899
http://www.cpantesters.org/cpan/report/b684f8b6-5ffa-11e1-9bda-bf61d4dc8887
both fail with:

DBD::DBM::db selectall_arrayref failed: Cannot open 
/home/njh/.local/share/.cpan/build/DBI-1.618-WGnXIY/fred.lck: No such file or 
directory (2) at 
/home/njh/.local/share/.cpan/build/DBI-1.618-WGnXIY/blib/lib/DBD/File.pm line 
730 at 
/home/njh/.local/share/.cpan/build/DBI-1.618-WGnXIY/blib/lib/DBI/DBD/SqlEngine.pm
 line 795 [for Statement select * from Fred] at ./t/51dbm_file.t line 122.
Compilation failed in require at t/zvn_51dbm_file.t line 4.

This one is different, but also DBM/DBD::File related:
http://www.cpantesters.org/cpan/report/01b4e958-6024-11e1-8e85-838032b1b899
t/zvxgnp_50dbm_simple.t . 
No subtests run 

Tim.


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/