Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Denis Smirnov]

Upravdom Team Robot wrote:
> Dear Denis Smirnov !
> The perl-DBD-SQLite-1.11-alt1.src.rpm package has been rejected from incoming.
>
> Possible reason follows:
> You have no permission to publish perl-DBD-SQLite-1.11-alt1.src.rpm
> Please consult   at    if you have any questions.
>   
Мне думается что at@ не будет возражать.

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 491 bytes --]

On Fri, Apr 14, 2006 at 04:32:42PM +0400, Denis Smirnov wrote:
> Upravdom Team Robot wrote:
> > Dear Denis Smirnov !
> > The perl-DBD-SQLite-1.11-alt1.src.rpm package has been rejected from incoming.
> >
> > Possible reason follows:
> > You have no permission to publish perl-DBD-SQLite-1.11-alt1.src.rpm
> > Please consult   at    if you have any questions.
> Мне думается что at@ не будет возражать.

А что там с sqlite3?  Я нему несколько патчей сделал, но для старой
версии.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Денис Смирнов]

On Fri, Apr 14, 2006 at 04:33:59PM +0400, Алексей Турбин wrote:

>>> Possible reason follows:
>>> You have no permission to publish perl-DBD-SQLite-1.11-alt1.src.rpm
>>> Please consult   at    if you have any questions.
>> Мне думается что at@ не будет возражать.
AT> А что там с sqlite3?  Я нему несколько патчей сделал, но для старой
AT> версии.

perl-DBD-SQLite не работает с sqlite3 из сизифа, потому я собрал его с
встроеным sqlite3.

У нас из-за этого сейчас sqlgrey неработоспособен вообще.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 519 bytes --]

On Fri, Apr 14, 2006 at 06:40:30PM +0400, Денис Смирнов wrote:
> On Fri, Apr 14, 2006 at 04:33:59PM +0400, Алексей Турбин wrote:
> 
> >>> Possible reason follows:
> >>> You have no permission to publish perl-DBD-SQLite-1.11-alt1.src.rpm
> >>> Please consult   at    if you have any questions.
> >> Мне думается что at@ не будет возражать.
> AT> А что там с sqlite3?  Я нему несколько патчей сделал, но для старой
> AT> версии.
> 
> perl-DBD-SQLite не работает с sqlite3 из сизифа,

А почему?


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Денис Смирнов]

On Fri, Apr 14, 2006 at 06:43:23PM +0400, Dmitry V. Levin wrote:

>> perl-DBD-SQLite не работает с sqlite3 из сизифа,
DVL> А почему?

Очень хороший вопрос. Моя медитация над кодом ни к чему не привела. В
редхате поступили так же как я.

Судя по всему у них слегка изменилось API, но внятного описания я в их
changelog'е не нашел.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
 * Vitls сегодня отмечает 3-х ление участия в проекте Linux Ru Net 

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1997 bytes --]

On Fri, Apr 14, 2006 at 07:33:38PM +0400, Денис Смирнов wrote:
> On Fri, Apr 14, 2006 at 06:43:23PM +0400, Dmitry V. Levin wrote:
> 
> >> perl-DBD-SQLite не работает с sqlite3 из сизифа,
> DVL> А почему?
> 
> Очень хороший вопрос. Моя медитация над кодом ни к чему не привела. В
> редхате поступили так же как я.

А как в редхате поступили?

$ cvs update perl-DBD-SQLite
cvs update: nothing known about perl-DBD-SQLite
$ cat CVS/R*
devel
:pserver:anonymous@cvs.fedora.redhat.com:/cvs/dist
$

> Судя по всему у них слегка изменилось API, но внятного описания я в их
> changelog'е не нашел.

В общем я разобрался в чём дело.  После вот этого патча всё работает:

--- DBD-SQLite-1.09/dbdimp.c-	2005-06-20 17:53:01 +0400
+++ DBD-SQLite-1.09/dbdimp.c	2006-04-16 02:29:37 +0400
@@ -259,7 +259,7 @@
     imp_sth->retval = SQLITE_OK;
     imp_sth->params = newAV();
 
-    if ((retval = sqlite3_prepare(imp_dbh->db, statement, 0, &(imp_sth->stmt), &extra))
+    if ((retval = sqlite3_prepare(imp_dbh->db, statement, -1, &(imp_sth->stmt), &extra))
         != SQLITE_OK)
     {
         if (imp_sth->stmt) {
@@ -319,7 +319,7 @@
         psv = hv_fetch((HV*)SvRV(sth), "Statement", 9, 0);
         statement = (psv && SvOK(*psv)) ? SvPV_nolen(*psv) : "";
         sqlite_trace(3, "re-prepare statement %s", statement);
-        if ((retval = sqlite3_prepare(imp_dbh->db, statement, 0, &(imp_sth->stmt), &extra))
+        if ((retval = sqlite3_prepare(imp_dbh->db, statement, -1, &(imp_sth->stmt), &extra))
             != SQLITE_OK)
         {
             if (imp_sth->stmt) {

Здесь этот аргумент по смыслу означает strlen(statement); раньше он
просто не использовался, а теперь в sqlite3_prepare() появилась проверка
на >= 0 с некоторой неприятной спецификой (см. sqlite/src/prepare.c).

См. тж. http://www.initd.org/tracker/pysqlite/ticket/143
http://www.sqlite.org/cvstrac/tktview?tn=1554

В общем пока предлагаю до понедельника ничего не делать. :)

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Денис Смирнов]

On Sun, Apr 16, 2006 at 02:40:45AM +0400, Алексей Турбин wrote:

>> Очень хороший вопрос. Моя медитация над кодом ни к чему не привела. В
>> редхате поступили так же как я.
AT> А как в редхате поступили?

Мне lakostis@ URL давал, я его потерял. Смысл был в том, что сборка с
sqlite3 в комплекте с perl-DBD-SQLite3 решала проблемы :)

>> Судя по всему у них слегка изменилось API, но внятного описания я в их
>> changelog'е не нашел.
AT> В общем я разобрался в чём дело.  После вот этого патча всё работает:

Ура! :)

AT> Здесь этот аргумент по смыслу означает strlen(statement); раньше он
AT> просто не использовался, а теперь в sqlite3_prepare() появилась проверка
на >>= 0 с некоторой неприятной спецификой (см. sqlite/src/prepare.c).
AT> См. тж. http://www.initd.org/tracker/pysqlite/ticket/143
AT> http://www.sqlite.org/cvstrac/tktview?tn=1554
AT> В общем пока предлагаю до понедельника ничего не делать. :)

Лучше поделись подходом -- как ты это нашел?

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
<hiddenman> ааа. это прикол
<AMorozov> ... сказал ежик, слазя с кактуса :-)

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 822 bytes --]

On Sun, Apr 16, 2006 at 03:25:33AM +0400, Денис Смирнов wrote:
> AT> Здесь этот аргумент по смыслу означает strlen(statement); раньше он
> AT> просто не использовался, а теперь в sqlite3_prepare() появилась проверка
> на >>= 0 с некоторой неприятной спецификой (см. sqlite/src/prepare.c).
> AT> См. тж. http://www.initd.org/tracker/pysqlite/ticket/143
> AT> http://www.sqlite.org/cvstrac/tktview?tn=1554
> AT> В общем пока предлагаю до понедельника ничего не делать. :)
> 
> Лучше поделись подходом -- как ты это нашел?

Скачал DBD-SQLite-1.{11,12}, сделал diff. :-|
Там всего несколько строчек отличается, но 1.12 работает с 3.3.
(То есть по большому счету нашёл не я.)

Потом сделал diff sqlite-3.2.6/src/prepare.c sqlite-3.3.4/src/prepare.c.
Потом гуглом sqlite nBytes -- первая ссылка это как раз оно.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Denis Smirnov]

On Sun, Apr 16, 2006 at 03:32:36AM +0400, Алексей Турбин wrote:

AT> Скачал DBD-SQLite-1.{11,12}, сделал diff. :-|

Опа, уже успел выйти. Хорошо :)

AT> Там всего несколько строчек отличается, но 1.12 работает с 3.3.
AT> (То есть по большому счету нашёл не я.)
AT> Потом сделал diff sqlite-3.2.6/src/prepare.c sqlite-3.3.4/src/prepare.c.
AT> Потом гуглом sqlite nBytes -- первая ссылка это как раз оно.

Ясно, спасибо.

Зальешь новый DBD-SQLite?

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1233 bytes --]

On Sun, Apr 16, 2006 at 02:40:45AM +0400, Alexey Tourbin wrote:
> --- DBD-SQLite-1.09/dbdimp.c-	2005-06-20 17:53:01 +0400
> +++ DBD-SQLite-1.09/dbdimp.c	2006-04-16 02:29:37 +0400
> @@ -259,7 +259,7 @@
>      imp_sth->retval = SQLITE_OK;
>      imp_sth->params = newAV();
>  
> -    if ((retval = sqlite3_prepare(imp_dbh->db, statement, 0, &(imp_sth->stmt), &extra))
> +    if ((retval = sqlite3_prepare(imp_dbh->db, statement, -1, &(imp_sth->stmt), &extra))
>          != SQLITE_OK)
>      {
>          if (imp_sth->stmt) {
> @@ -319,7 +319,7 @@
>          psv = hv_fetch((HV*)SvRV(sth), "Statement", 9, 0);
>          statement = (psv && SvOK(*psv)) ? SvPV_nolen(*psv) : "";
>          sqlite_trace(3, "re-prepare statement %s", statement);
> -        if ((retval = sqlite3_prepare(imp_dbh->db, statement, 0, &(imp_sth->stmt), &extra))
> +        if ((retval = sqlite3_prepare(imp_dbh->db, statement, -1, &(imp_sth->stmt), &extra))
>              != SQLITE_OK)
>          {
>              if (imp_sth->stmt) {

Но лучше не этот патч прикладывать, а изменить условие проверки с '>=0'
на '>0' в sqlite/src/prepare.c.  Это хак, но он по-видимому решает
больше проблем (с бинарной совместимостью), чем создает.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 423 bytes --]

On Fri, Apr 14, 2006 at 06:40:30PM +0400, Денис Смирнов wrote:
> AT> А что там с sqlite3?  Я нему несколько патчей сделал, но для старой
> AT> версии.
> 
> perl-DBD-SQLite не работает с sqlite3 из сизифа, потому я собрал его с
> встроеным sqlite3.

А почему?  Может лучше исправить sqlite3?  Короче мне нужно некоторое
время, чтобы войти в курс дела.

> У нас из-за этого сейчас sqlgrey неработоспособен вообще.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Денис Смирнов]

On Fri, Apr 14, 2006 at 06:43:39PM +0400, Алексей Турбин wrote:

>> perl-DBD-SQLite не работает с sqlite3 из сизифа, потому я собрал его с
>> встроеным sqlite3.
AT> А почему?  Может лучше исправить sqlite3?  Короче мне нужно некоторое
AT> время, чтобы войти в курс дела.

Думаю лучше исправить perl-DBD-SQLite, потому как его код видать просто не
обновили для нового sqlite3. Собственно проверяется просто -- берется
нынешний perl-DBD-SQLite, и мы видим, что он не пересобирается, так как не
проходит тесты.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
Не надо оправдывать багу.
		-- mike in community@

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 593 bytes --]

On Fri, Apr 14, 2006 at 07:35:02PM +0400, Денис Смирнов wrote:
> >> perl-DBD-SQLite не работает с sqlite3 из сизифа, потому я собрал его с
> >> встроеным sqlite3.
> AT> А почему?  Может лучше исправить sqlite3?  Короче мне нужно некоторое
> AT> время, чтобы войти в курс дела.
> 
> Думаю лучше исправить perl-DBD-SQLite, потому как его код видать просто не
> обновили для нового sqlite3. Собственно проверяется просто -- берется
> нынешний perl-DBD-SQLite, и мы видим, что он не пересобирается, так как не
> проходит тесты.

У sqlite3 был довольно вменяемый ABI.  Буду смотреть...

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 782 bytes --]

On Fri, Apr 14, 2006 at 07:48:04PM +0400, Alexey Tourbin wrote:
> On Fri, Apr 14, 2006 at 07:35:02PM +0400, Денис Смирнов wrote:
> > >> perl-DBD-SQLite не работает с sqlite3 из сизифа, потому я собрал его с
> > >> встроеным sqlite3.
> > AT> А почему?  Может лучше исправить sqlite3?  Короче мне нужно некоторое
> > AT> время, чтобы войти в курс дела.
> > 
> > Думаю лучше исправить perl-DBD-SQLite, потому как его код видать просто не
> > обновили для нового sqlite3. Собственно проверяется просто -- берется
> > нынешний perl-DBD-SQLite, и мы видим, что он не пересобирается, так как не
> > проходит тесты.
> 
> У sqlite3 был довольно вменяемый ABI.  Буду смотреть...

А!  Ты собрал новый sqlite3!  И что, с новым sqlite3 старый код не
собирается и не работает? :)

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Денис Смирнов]

On Fri, Apr 14, 2006 at 07:58:38PM +0400, Алексей Турбин wrote:

>> У sqlite3 был довольно вменяемый ABI.  Буду смотреть...
AT> А!  Ты собрал новый sqlite3!  И что, с новым sqlite3 старый код не
AT> собирается и не работает? :)

Хуже. Новый код не собирается и не работает. То что ушло в incoming это
1.11. При этом вдумчивое чтение ченджлога не подсказало мне где проблема.
Мы с lakostis@ думали что делать -- откатывать назад sqlite3, либо
собирать perl-DBD-SQLite с имеющейся в нем sqlite3. Остановились на втором
варианте.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 642 bytes --]

On Fri, Apr 14, 2006 at 09:28:56PM +0400, Денис Смирнов wrote:
> >> У sqlite3 был довольно вменяемый ABI.  Буду смотреть...
> AT> А!  Ты собрал новый sqlite3!  И что, с новым sqlite3 старый код не
> AT> собирается и не работает? :)
> 
> Хуже. Новый код не собирается и не работает. То что ушло в incoming это
> 1.11. При этом вдумчивое чтение ченджлога не подсказало мне где проблема.
> Мы с lakostis@ думали что делать -- откатывать назад sqlite3, либо
> собирать perl-DBD-SQLite с имеющейся в нем sqlite3. Остановились на втором
> варианте.

Надо фиксить sqlite3, оно гораздо лучше мозиллы написано и разобраться
там можно. :)

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Денис Смирнов]

On Fri, Apr 14, 2006 at 09:43:41PM +0400, Алексей Турбин wrote:

>> Хуже. Новый код не собирается и не работает. То что ушло в incoming это
>> 1.11. При этом вдумчивое чтение ченджлога не подсказало мне где проблема.
>> Мы с lakostis@ думали что делать -- откатывать назад sqlite3, либо
>> собирать perl-DBD-SQLite с имеющейся в нем sqlite3. Остановились на втором
>> варианте.
AT> Надо фиксить sqlite3, оно гораздо лучше мозиллы написано и разобраться
AT> там можно. :)

Мне пока не ясно кого надо фиксить -- sqlite3 или все-таки perl-DBD-SQLite

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 480 bytes --]

On Fri, Apr 14, 2006 at 04:32:42PM +0400, Denis Smirnov wrote:
> Upravdom Team Robot wrote:
> > Dear Denis Smirnov !
> > The perl-DBD-SQLite-1.11-alt1.src.rpm package has been rejected from incoming.
> >
> > Possible reason follows:
> > You have no permission to publish perl-DBD-SQLite-1.11-alt1.src.rpm
> > Please consult   at    if you have any questions.
> >   
> Мне думается что at@ не будет возражать.

Не факт.  at@ теперь может высказаться сам.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] E: incoming reject: perl-DBD-SQLite-1.11-alt1.src.rpm

[Денис Смирнов]

On Fri, Apr 14, 2006 at 04:55:38PM +0400, Dmitry V. Levin wrote:

>> Мне думается что at@ не будет возражать.
DVL> Не факт.  at@ теперь может высказаться сам.

Ура!

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------