Bug #65041 for SQL-Statement: circular dependency

Sun Jan 23 08:57:50 2011 CHORNY [...] cpan.org - Ticket created

Subject:

circular dependency

Installing with CPAN.pm Show quoted text

---- Unsatisfied dependencies detected during ---- ---- REHSACK/SQL-Statement-1.32.tar.gz ---- DBD::AnyData [requires] DBD::CSV [requires]

---- Unsatisfied dependencies detected during ---- ---- REHSACK/DBD-AnyData-0.110.tar.gz ---- SQL::Statement [requires] AnyData [requires] Recursive dependency detected: SQL::Statement (have: N/A; want: 1.32) => REHSACK/SQL-Statement-1.32.tar.gz => DBD::AnyData (have: N/A; requires: 0.110) => REHSACK/DBD-AnyData-0.110.tar.gz => SQL::Statement (have: N/A; requires: 1.27_02). Cannot resolve. REHSACK/DBD-AnyData-0.110.tar.gz [depend] -- NOT OK

---- Unsatisfied dependencies detected during ---- ---- HMBRAND/DBD-CSV-0.31.tgz ---- SQL::Statement [requires] Recursive dependency detected: SQL::Statement (have: N/A; want: 1.32) => REHSACK/SQL-Statement-1.32.tar.gz => DBD::AnyData (have: N/A; requires: 0.110) => REHSACK/DBD-AnyData-0.110.tar.gz => SQL::Statement (have: N/A; requires: 1.27_02). Cannot resolve. HMBRAND/DBD-CSV-0.31.tgz [depend] -- NOT OK -- Alexandr Ciornii, http://chorny.net

Wed Jan 26 02:23:36 2011 REHSACK [...] cpan.org - Correspondence added

I see the discomfort the test dependencies create - and meanwhile I have an idea how to break free ;) I added them to get a bigger picture what can fail, because I often got reports from others that newer SQL::Statement releases break something in DBD::CSV, DBD::DBM etc.

Wed Jan 26 02:23:40 2011 The RT System itself - Status changed from 'new' to 'open'

Wed Feb 02 17:25:03 2011 dmuey [...] cpan.org - Correspondence added

DBD::CSV is also a circular dependency w/ SQL::Statement

Wed Feb 02 18:09:50 2011 dmuey [...] cpan.org - Correspondence added

Attached is 1.33 dev that addresses the circular dependency on DBD::AnyData and DBD::CSV. tests pass We are testing on our internal CPAN mirror to ensure installation via CPAN works fine

Subject:

SQL-Statement-1.33.tar.gz

Download SQL-Statement-1.33.tar.gz
application/x-gzip 119.5k

Message body not shown because it is not plain text.

Wed Feb 02 18:12:37 2011 dmuey [...] cpan.org - Correspondence added

On Wed Feb 02 18:09:50 2011, DMUEY wrote: Show quoted text

> Attached is 1.33 dev that addresses the circular dependency on > DBD::AnyData and DBD::CSV. > > tests pass > > We are testing on our internal CPAN mirror to ensure installation via > CPAN works fine

Here is the same thing but with a development version pf 1.32_01

Subject:

SQL-Statement-1.32_01.tar.gz

Download SQL-Statement-1.32_01.tar.gz
application/x-gzip 119.5k

Message body not shown because it is not plain text.

Thu Feb 03 01:13:37 2011 REHSACK [...] cpan.org - Correspondence added

The technical removal of the dependency isn't the problem - the problem is to fulfill the requirement which leads to the circular dependency. If you would be able to upload a patch instead of a gzipped tarball, I promise I take a look.

Thu Feb 03 09:04:49 2011 dmuey [...] cpan.org - Correspondence added

Subject:

circular dependency makes SQL::Statement uninstallable via CPAN

On Thu Feb 03 01:13:37 2011, REHSACK wrote: Show quoted text

> The technical removal of the dependency isn't the problem - the problem > is to fulfill the requirement which leads to the circular dependency.

I agree this is a symptom of a larger issue that needs addressed, however it'd be nice to at least be able to install it via CPAN in the meantime, no? Show quoted text

> If you would be able to upload a patch instead of a gzipped tarball, I > promise I take a look.

Attached, thanks

Subject:

rt65041.patch

diff -ruN SQL-Statement-1.32/Changes SQL-Statement-1.33/Changes --- SQL-Statement-1.32/Changes 2011-01-19 00:09:19.000000000 -0600 +++ SQL-Statement-1.33/Changes 2011-02-02 17:07:48.000000000 -0600 @@ -1,5 +1,12 @@ Changes log for Perl extension SQL::Statement +Version 1.33, released February 1, 2001 + +Remove circular dependency on DBD::AnyData as per rt 65041 + +Also noted DBD::CSV was a circular dependency, added info to rt, +and addressed in this dev version + Version 1.32, released January 19th, 2011 ---------------------------------------------- diff -ruN SQL-Statement-1.32/META.yml SQL-Statement-1.33/META.yml --- SQL-Statement-1.32/META.yml 2011-01-19 00:12:13.000000000 -0600 +++ SQL-Statement-1.33/META.yml 2011-02-02 17:08:34.000000000 -0600 @@ -1,6 +1,6 @@ --- #YAML:1.0 name: SQL-Statement -version: 1.32 +version: 1.33 abstract: SQL parsing and processing engine author: - Jeff Zucker <jeff@vpservices.com>, Jens Rehsack <rehsack@cpan.org> @@ -9,8 +9,6 @@ configure_requires: ExtUtils::MakeMaker: 0 build_requires: - DBD::AnyData: 0.110 - DBD::CSV: 0.30 DBD::DBM: 0.06 DBD::File: 0.40 DBD::SQLite: 0 diff -ruN SQL-Statement-1.32/Makefile.PL SQL-Statement-1.33/Makefile.PL --- SQL-Statement-1.32/Makefile.PL 2010-12-31 06:36:49.000000000 -0600 +++ SQL-Statement-1.33/Makefile.PL 2011-02-02 16:46:02.000000000 -0600 @@ -64,8 +64,9 @@ # means: DBI >= 1.616 'DBI::DBD::SqlEngine' => '0.03', 'DBD::File' => '0.40', - 'DBD::AnyData' => '0.110', - 'DBD::CSV' => '0.30', +# rt65041 we only need this for tests +# 'DBD::AnyData' => '0.110', +# 'DBD::CSV' => '0.30', 'DBD::DBM' => '0.06', 'DBD::SQLite' => '0', 'MLDBM' => '0', diff -ruN SQL-Statement-1.32/lib/SQL/Statement.pm SQL-Statement-1.33/lib/SQL/Statement.pm --- SQL-Statement-1.32/lib/SQL/Statement.pm 2011-01-15 05:13:02.000000000 -0600 +++ SQL-Statement-1.33/lib/SQL/Statement.pm 2011-02-02 17:07:48.000000000 -0600 @@ -30,7 +30,7 @@ #use locale; -$VERSION = '1.32'; +$VERSION = '1.33'; sub new {

Mon Feb 07 17:13:47 2011 REHSACK [...] cpan.org - Correspondence added

I'd say this is fixed in 1.33 together with Bundle::Test::SQL::Statement and Bundle::DBD::DBM.

Mon Feb 07 17:13:48 2011 REHSACK [...] cpan.org - Status changed from 'open' to 'resolved'

Mon Feb 07 17:14:40 2011 REHSACK [...] cpan.org - Severity Critical changed to Important

Mon Feb 07 17:14:40 2011 REHSACK [...] cpan.org - Fixed in 1.33 added