#6 not supported for perl 5.26

[Cesar Martins]

Hi,
YaSQL not work anymore after I upgrade to perl 5.28... please can anyone update the sources to keep it functional?

[Edoard Matz]

Ran into the same issue recently. Came by today to see if there was a newer version for YaSQL. Not sure if you still have the issue. I've attached a patch with a fix. I've tested my changes on Ubuntu 18.04 and perl 5.28.

In case you are curious this was caused by the below issue which before 5.26 would just caused a warning.
Use of "do" to call subroutines is deprecated

From https://perldoc.perl.org/perl5200delta.html
Incompatible Changes
do can no longer be used to call subroutines

The do SUBROUTINE(LIST) form has resulted in a deprecation warning since Perl v5.0.0, and is now a syntax error.

[Marcin Kasperski]

[Marcin Kasperski]

Just in case somebody incomfortable with applying patches finds this issue: trivial edit is needed to fix the problem, just replace

open(FOUT, $op . $op_text) || do query_err(…)

with

open(FOUT, $op . $op_text) or query_err(…)

(remove word do , I also suggest replacing || with or but this is not essential).
Note that there are two such lines.

Then usual ./configure; make; sudo make install and tool works as expected (of course one must also install DBD::Oracle, for example cpanm --sudo DBD::Oracle)

[DaTa]

We forked that thing and added some stuff, and also fixed that 13 years ago: https://github.com/datamuc/yasqlng/commit/7ab0bd5a79512d51746f63bb60f2eeabb594d575

[Lorinczy Zsigmond]

Off:
I've just created another fork: https://github.com/lzsiga/yasql-fx
Not because of the version-incompatibility, but to solve an actual problem (test: https://github.com/lzsiga/yasql-fx/blob/main/test/test0001.sh )