Keyring KWallet backend build fails on some architectures

[Carl Chenet]

Hi,

In Debian we have a tool to build the package on different
architectures. It seems the Keyring KWallet backend build fails on some
architectures, given I guess a missing dependency but looking through
the code I'm unable to find it.

Here is the list of the dependencies I'm using to build the Keyring
KWallet backend, please let me know if an obvious one is missing :

'dbus-1' -> /usr/lib/pkgconfig/dbus-1.pc (provided by the libdbus-1-dev
Debian package). The same package for /usr/include/dbus-1.0/dbus/dbus.h

'glib-2.0' -> /usr/lib/pkgconfig/glib-2.0.pc (provided by the
libglib2.0-dev Debian package).

'QtGui' -> /usr/lib/pkgconfig/QtGui.pc (provided by the libqt4-dev
Debian package). The same package
for /usr/include/qt4/QtCore/QCoreApplication
and /usr/include/qt4/QtCore/QString

kwallet.h -> /usr/include/kwallet.h from the Debian package kdelibs5-dev

"Python.h" -> provided by the python-all-dev Debian package
e.g /usr/include/python2.5/Python.h for Python2.5

kde4-config -> provided by the kdelibs-bin Debian package. kde-config is
I guess optional if we don't use qt3.

pkg-config -> provided by the pkg-config Debian package.

This problem is weird because only the KWallet backend does not build,
which means only a dependency related to KDE/KWallet is missing.

Bye,
--
Carl Chenet

[Douglas Thrift]

Hello,

It might be useful to know which architectures this is failing on and
what kind of error messages it is giving.
--
Douglas William Thrift
<dougl...@gmail.com>
<http://douglasthrift.net/>

[chaica]

The most disturbing one is i386 (because I realized the package on
this plateform and had no issues even building it in a chroot). That's
why I guess the Debian autobuilder is really touchy.

> what kind of error messages it is giving.

This is the most annoying, I don't have a error message, the step of
building the KWallet backend build is just skipped. Here is the
output :

running build_ext
building 'gnome_keyring' extension
creating build/temp.linux-i686-2.5
creating build/temp.linux-i686-2.5/keyring
creating build/temp.linux-i686-2.5/keyring/backends
gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -
Wstrict-prototypes -g -O2 -fPIC -I/usr/include/dbus-1.0 -I/usr/lib/
dbus-1.0/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -
I/usr/include/gnome-keyring-1 -I/usr/include/python2.5 -c keyring/
backends/gnome_keyring.c -o build/temp.linux-i686-2.5/keyring/backends/
gnome_keyring.o
gcc -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -g -O2 build/
temp.linux-i686-2.5/keyring/backends/gnome_keyring.o -ldbus-1 -
lpthread -lrt -lgnome-keyring -lglib-2.0 -o build/lib.linux-i686-2.5/
gnome_keyring.so
dh_auto_test
...
after this step, the KWallet backend build is just skipped and the
creation of the package fails because kde_kwallet.so is missing,
that's why the package buid fails. But if everything is ok with the
dependency I gave earlier and no one is missing, I'll have to report a
bug on this builder, because I successfully build the package on the
same archi at home.

The full log is available at :
https://buildd.debian.org/fetch.cgi?&pkg=python-keyring&ver=0.2-2&arch=i386&stamp=1254208107&file=log

> --
> Douglas William Thrift
> <douglas...@gmail.com>
> <http://douglasthrift.net/>

[chaica]

The correct sentence I should get after the message above is :
building 'kde_kwallet' extension

gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -
Wstrict-prototypes -g -O2 -fPIC -I/usr/include/dbus-1.0 -I/usr/lib/
dbus-1.0/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -

I/usr/include/qt4 -I/usr/include/qt4/QtGui -I/usr/include/qt4/QtCore -
I/usr/include/ -I/usr/include/python2.5 -c keyring/backends/
kde_kwallet.cpp -o build/temp.linux-i686-2.5/keyring/backends/
kde_kwallet.o
cc1plus: warning: command line option "-Wstrict-prototypes" is valid
for Ada/C/ObjC but not for C++
keyring/backends/kde_kwallet.cpp: In function 'PyObject*
kde_kwallet_password_get(PyObject*, PyObject*)':
keyring/backends/kde_kwallet.cpp:15: warning: 'password' may be used
uninitialized in this function
g++ -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -g -O2 build/
temp.linux-i686-2.5/keyring/backends/kde_kwallet.o -L/tmp/buildd/.kde/
lib/ -L/usr/lib/ -ldbus-1 -lpthread -lrt -lglib-2.0 -lQtGui -lQtCore -
lkdeui -o build/lib.linux-i686-2.5/kde_kwallet.so -DQT_SHARED

>    dh_auto_test
> ...
> after this step, the KWallet backend build is just skipped and the
> creation of the package fails because kde_kwallet.so is missing,
> that's why the package buid fails. But if everything is ok with the
> dependency I gave earlier and no one is missing, I'll have to report a
> bug on this builder, because I successfully build the package on the
> same archi at home.
>

> The full log is available at :https://buildd.debian.org/fetch.cgi?&pkg=python-keyring&ver=0.2-2&arc...

[Kang Zhang]

Hi Carl,

Can you run following commands and put the result here?

$ pkg-config --exists dbus-1 glib-2.0 QtGui

$ kde4-config --install include

the second command should print a list of dirs, and
kwallet.h is supposed to be in one of them. Can you also
confirm this is true on your computer?

Kang

--
------------------------------------------------------------
Kang Zhang

Computer Science Dept.
Shanghai Jiao Tong University

[chaica]

On Oct 6, 2:34 pm, Kang Zhang <jobo...@gmail.com> wrote:
> Hi Carl,
>
> Can you run following commands and put the result here?
>
>  $ pkg-config --exists dbus-1 glib-2.0 QtGui
>
>  $ kde4-config --install include
>
> the second command should print a list of dirs, and
> kwallet.h is supposed to be in one of them. Can you also
> confirm this is true on your computer?
>
> Kang
>

Please have a look at http://bitbucket.org/kang/python-keyring-lib/issue/9/unable-to-build-kde_kwalletso
for a detailed report about what was going on. Sorry for the mess
before, it was tricky to understand.