[devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Motsyo Gennadi aka Drool]

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

  Просьба не пропускать в репозиторий данный пакет - он почему-то не 
собрался для 64-бит. Хотя alt0 собирался, сменилось только упаковывание 
иконок в спеке...

[-- Attachment #2: E: incoming x86_64 reject: qcat-0.5-alt1.eml --]
[-- Type: message/rfc822, Size: 4064 bytes --]

From: Upravdom Team Robot <upravdom@altlinux.org>
To: "Motsyo Gennadi (http://drool-linux.pisem.net)" <drool@altlinux.org>
Cc: sisyphus-incominger@lists.altlinux.org
Subject: E: incoming x86_64 reject: qcat-0.5-alt1
Date: Thu, 22 Nov 2007 15:37:18 +0300 (MSK)
Message-ID: <20071122123718.962F128260@hint1.office.altlinux.org>


Dear Motsyo Gennadi (http://drool-linux.pisem.net)!
The qcat-0.5-alt1 package has been rejected from incoming for x86_64
architecture.

Possible reason follows:
deviceclass.cpp:267: warning: control reaches end of non-void function
deviceclass.cpp: In member function 'int DeviceClass::checkForISO9660(const QString&)':
deviceclass.cpp:197: warning: control reaches end of non-void function
g++ -c -pipe -pipe -pipe -Wall -O2 -g -D_REENTRANT -Wall -W -DQT_SQL_LIB -DQT_XML_LIB -DQT_GUI_LIB -DQT_CORE_LIB -I/usr/share/qt4/mkspecs/linux-g++-64 -I. -I/usr/include/qt4/QtCore -I/usr/include/qt4/QtCore -I/usr/include/qt4/QtGui -I/usr/include/qt4/QtGui -I/usr/include/qt4/QtXml -I/usr/include/qt4/QtXml -I/usr/include/qt4/QtSql -I/usr/include/qt4/QtSql -I/usr/include/qt4 -I../build -I../build -o ../build/catalogtable.o catalogtable.cpp
catalogtable.h: In constructor 'CatalogTable::CatalogTable(QCatDataModuleClass*, QWidget*)':
catalogtable.h:68: warning: 'CatalogTable::dm' will be initialized after
catalogtable.cpp:37: warning:   base 'QTreeWidget'
catalogtable.cpp:36: warning:   when initialized here
catalogtable.cpp: At global scope:
catalogtable.cpp:80: warning: unused parameter 'prevItem'
catalogtable.cpp:259: warning: unused parameter 'oldSize'
catalogtable.cpp:272: warning: unused parameter 'column'
catalogtable.cpp:318: warning: unused parameter 'event'
catalogtable.cpp:322: warning: unused parameter 'event'
catalogtable.cpp:326: warning: unused parameter 'event'
catalogtable.cpp: In member function 'void CatalogTable::startDrag()':
catalogtable.cpp:344: error: cast from 'QList<QTreeWidgetItem*>*' to 'int' loses precision
make[1]: Leaving directory `/usr/src/RPM/BUILD/qcat-dist-0.5/src'
make[1]: *** [../build/catalogtable.o] Error 1
make: *** [sub-src-make_default] Error 2
make: Leaving directory `/usr/src/RPM/BUILD/qcat-dist-0.5'
error: Bad exit status from /usr/src/tmp/rpm-tmp.49728 (%build)


RPM build errors:
    Bad exit status from /usr/src/tmp/rpm-tmp.49728 (%build)
Command exited with non-zero status 1
54.47user 3.35system 0:58.11elapsed 99%CPU (0avgtext+0avgdata 0maxresident)k
0inputs+0outputs (0major+799939minor)pagefaults 0swaps
hsh-rebuild: rebuild of `qcat-0.5-alt1.src.rpm' failed.


-- 
Rgrds, your upravdom

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Michael Shigorin]

On Thu, Nov 22, 2007 at 02:46:41PM +0200, Motsyo Gennadi aka Drool wrote:
> Просьба не пропускать в репозиторий данный пакет - он почему-то
> не собрался для 64-бит. Хотя alt0 собирался, сменилось только
> упаковывание иконок в спеке...

Не проще ли собирать altN+1 для исправления рассинхронизации?

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Motsyo Gennadi aka Drool]

Michael Shigorin пишет:
> On Thu, Nov 22, 2007 at 02:46:41PM +0200, Motsyo Gennadi aka Drool wrote:
>> Просьба не пропускать в репозиторий данный пакет - он почему-то
>> не собрался для 64-бит. Хотя alt0 собирался, сменилось только
>> упаковывание иконок в спеке...
> 
> Не проще ли собирать altN+1 для исправления рассинхронизации?

	Не уловил мысль... Я ж вроде так и сделал - была alt0, поправил спек и 
сделал alt1... Просто я не понял что изменилось в сборочной среде, 
изменения составили только в спеке:

%_iconsdir/hicolor/*/apps/%name.png

на

%_miconsdir/%name.png
%_niconsdir/%name.png
%_liconsdir/%name.png

	Больше ничего не менял.

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Motsyo Gennadi aka Drool]

Motsyo Gennadi aka Drool пишет:

	Кто может немного разъяснить какие изменения произошли в сборочной 
среде сизифа? Теперь даже имеющаяся там сборка qcat-0.5-alt0 на 64 бита 
не собирается с этим симптомом:

> catalogtable.cpp: In member function 'void CatalogTable::startDrag()':
> catalogtable.cpp:344: error: cast from 'QList<QTreeWidgetItem*>*' to 'int' loses precision
> make[1]: Leaving directory `/usr/src/RPM/BUILD/qcat-dist-0.5/src'
> make[1]: *** [../build/catalogtable.o] Error 1

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Victor Forsyuk]

On Fri, Nov 23, 2007 at 11:01:48AM +0200, Motsyo Gennadi aka Drool <motsyo@gmail.com> wrote:
> Motsyo Gennadi aka Drool пишет:
> 
> 	Кто может немного разъяснить какие изменения произошли в сборочной 
> среде сизифа? Теперь даже имеющаяся там сборка qcat-0.5-alt0 на 64 бита 
> не собирается с этим симптомом:

Почему Вы думаете, что изменилось? Разве Вам приходило сообщение об успешной
сборке qcat-0.5-alt0 на x86_64? На самом деле он и не собирался.
 
> > catalogtable.cpp: In member function 'void CatalogTable::startDrag()':
> > catalogtable.cpp:344: error: cast from 'QList<QTreeWidgetItem*>*' to 'int' loses precision
> > make[1]: Leaving directory `/usr/src/RPM/BUILD/qcat-dist-0.5/src'
> > make[1]: *** [../build/catalogtable.o] Error 1

Так нарисуйте патчик. Он несложный. Вместо 'int' должно быть 'long'.

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Motsyo Gennadi aka Drool]

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

Victor Forsyuk пишет:
> On Fri, Nov 23, 2007 at 11:01:48AM +0200, Motsyo Gennadi aka Drool <motsyo@gmail.com> wrote:
>> Motsyo Gennadi aka Drool пишет:
>>
>> 	Кто может немного разъяснить какие изменения произошли в сборочной 
>> среде сизифа? Теперь даже имеющаяся там сборка qcat-0.5-alt0 на 64 бита 
>> не собирается с этим симптомом:
> 
> Почему Вы думаете, что изменилось? Разве Вам приходило сообщение об успешной
> сборке qcat-0.5-alt0 на x86_64? На самом деле он и не собирался.

	Мда... Результат двух работ и хронического недосыпания. Но честно - не 
помню reject-а :-(

>>> catalogtable.cpp: In member function 'void CatalogTable::startDrag()':
>>> catalogtable.cpp:344: error: cast from 'QList<QTreeWidgetItem*>*' to 'int' loses precision
>>> make[1]: Leaving directory `/usr/src/RPM/BUILD/qcat-dist-0.5/src'
>>> make[1]: *** [../build/catalogtable.o] Error 1
> 
> Так нарисуйте патчик. Он несложный. Вместо 'int' должно быть 'long'.

	Патчик в аттаче. Я Вас правильно понял? И этот патч нужен только для 
x86_64, или и на 32 бита он не помешает? Делать ли в спеке выборочный 
патчинг только для 64 бит?

[-- Attachment #2: qcat-0.5-x86_64.diff --]
[-- Type: text/x-patch, Size: 444 bytes --]

--- qcat-dist-0.5/src/catalogtable.cpp.orig	2007-07-18 21:08:21 +0400
+++ qcat-dist-0.5/src/catalogtable.cpp	2007-11-23 13:25:34 +0300
@@ -341,7 +341,7 @@ void CatalogTable::startDrag()
 		selItems +=":";
 	}
     char ptrCh[5];
-    int listAddr = reinterpret_cast <int> (&selectedItemsList);
+    int listAddr = reinterpret_cast <long> (&selectedItemsList);
     memcpy(&ptrCh, &listAddr, 4);
     ptrCh[4] = 0x0;
     QByteArray ptr(ptrCh);

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Victor Forsyuk]

On Fri, Nov 23, 2007 at 12:42:38PM +0200, Motsyo Gennadi aka Drool <motsyo@gmail.com> wrote:
> 
> >>>catalogtable.cpp: In member function 'void CatalogTable::startDrag()':
> >>>catalogtable.cpp:344: error: cast from 'QList<QTreeWidgetItem*>*' to 
> >>>'int' loses precision
> >>>make[1]: Leaving directory `/usr/src/RPM/BUILD/qcat-dist-0.5/src'
> >>>make[1]: *** [../build/catalogtable.o] Error 1
> >
> >Так нарисуйте патчик. Он несложный. Вместо 'int' должно быть 'long'.
> 
> 	Патчик в аттаче. Я Вас правильно понял? И этот патч нужен только для 

Нет, Вы не поняли смысла ошибки. "cast ... to 'int' loses precision" - иными
словами нельзя присваивать это значение переменной типа 'int'.

> x86_64, или и на 32 бита он не помешает? Делать ли в спеке выборочный 
> патчинг только для 64 бит?

Да, _нужен_ он только на x86_64, что очевидно. На 32 бита он не помешает.

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Motsyo Gennadi aka Drool]

Victor Forsyuk пишет:
>> 	Патчик в аттаче. Я Вас правильно понял? И этот патч нужен только для 
> 
> Нет, Вы не поняли смысла ошибки. "cast ... to 'int' loses precision" - иными
> словами нельзя присваивать это значение переменной типа 'int'.

	Нет, я имел ввиду правильно ли я понял Вашу мысль что именно заменить 
патчем. Собралось и на 32, и на 64, но запуск могу проверить только на 
32 (проверил - работает).

>> x86_64, или и на 32 бита он не помешает? Делать ли в спеке выборочный 
>> патчинг только для 64 бит?
> 
> Да, _нужен_ он только на x86_64, что очевидно. На 32 бита он не помешает.

	Ок.

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[Victor Forsyuk]

On Fri, Nov 23, 2007 at 04:37:08PM +0200, Motsyo Gennadi aka Drool <motsyo@gmail.com> wrote:
> Victor Forsyuk пишет:
> >> 	Патчик в аттаче. Я Вас правильно понял? И этот патч нужен только для 
> > 
> > Нет, Вы не поняли смысла ошибки. "cast ... to 'int' loses precision" - иными
> > словами нельзя присваивать это значение переменной типа 'int'.
> 
> 	Нет, я имел ввиду правильно ли я понял Вашу мысль что именно заменить 
> патчем. Собралось и на 32, и на 64, но запуск могу проверить только на 
> 32 (проверил - работает).

Нет, Вы не поняли. Ладно, посмотрите тогда вот на такой патчик, возможно он
натолкнет Вас на понимание:

http://sisyphus.ru/srpm/Sisyphus/vegastrike/patches/0

Re: [devel] [Fwd: E: incoming x86_64 reject: qcat-0.5-alt1]

[led]

Friday, 23 November 2007 16:15:51 Victor Forsyuk написав:
> On Fri, Nov 23, 2007 at 12:42:38PM +0200, Motsyo Gennadi aka Drool 
<motsyo@gmail.com> wrote:
> > >>>catalogtable.cpp: In member function 'void CatalogTable::startDrag()':
> > >>>catalogtable.cpp:344: error: cast from 'QList<QTreeWidgetItem*>*' to
> > >>>'int' loses precision
> > >>>make[1]: Leaving directory `/usr/src/RPM/BUILD/qcat-dist-0.5/src'
> > >>>make[1]: *** [../build/catalogtable.o] Error 1
> > >
> > >Так нарисуйте патчик. Он несложный. Вместо 'int' должно быть 'long'.
> >
> > 	Патчик в аттаче. Я Вас правильно понял? И этот патч нужен только для
>
> Нет, Вы не поняли смысла ошибки. "cast ... to 'int' loses precision" -
> иными словами нельзя присваивать это значение переменной типа 'int'.
>
> > x86_64, или и на 32 бита он не помешает? Делать ли в спеке выборочный
> > патчинг только для 64 бит?
>
> Да, _нужен_ он только на x86_64, что очевидно. На 32 бита он не помешает.

На 32 битах sizeof(int) == sizeof(long). Так что если код позицирнируется как 
(не-32bit-only), то не только "не помешает", но и "только так и должно быть". 
а вобще надо смотреть код: может там какой-нибудь time_t/size_t/etc. нужно 
применять, а не int/long...

___
Led.