Bug#824426: FTBFS against libical2

2016-05-20 Thread Tino Mettler
tags 824426 +pending

Hi folks,

thanks for all the reports, discussion, and testing feedback. I
submitted an upload request to my sponsor. The current state of the
package is also pushed to the git repository.

Regards,
Tino



Bug#824426: FTBFS against libical2

2016-05-19 Thread Justus-dev
I just tried Tino's 1.5.1 packages, and they appear to work flawlessly,
solving the problem I reported above
(https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=824426#17).

Many thanks,
Justus

-- 
Intelligent and Interactive Systems https://iis.uibk.ac.at/
Institute of Computer Science, University of Innsbruck, Austria



Bug#824426: FTBFS against libical2

2016-05-17 Thread Tino Keitel
On Tue, May 17, 2016 at 09:14:39 +0200, Tino Keitel wrote:

> absolutely. Actually, I have 1.6.1 packages ready for upload.

Sorry, 1.5.1 packages of cause [1].

It's not pushed to the git repository[2] yet, just send me a notice if
someone wants to look into it.

Regards,
Tino

[1] http://tikei.de/debian/syncevolution/

[2] http://anonscm.debian.org/cgit/collab-maint/syncevolution.git



Bug#824426: FTBFS against libical2

2016-05-17 Thread Tino Keitel
On Tue, May 17, 2016 at 09:08:17 +0200, Patrick Ohly wrote:

[...]

> OTOH, Debian itself is still on an old, pre-release version. Tino, do
> you still have plans for getting SyncEvolution updated in Debian?

Hi,

absolutely. Actually, I have 1.6.1 packages ready for upload. I just
wanted to do some more testing regarding SyncML and CalDAV/CardDAV, but
got stuck in too much RL. I'd be glad if someone could do some testing
and confirm that nothing breaks.

Regards,
Tino



Bug#824426: FTBFS against libical2

2016-05-17 Thread Patrick Ohly
On Mon, 2016-05-16 at 10:59 +0200, Andreas Henriksson wrote:
> Hello!
> 
> Some information below which might be useful
> 
> On Sun, May 15, 2016 at 09:37:24PM +0200, Michael Biebl wrote:
> > Source: syncevolution
> > Version: 1.4.99.4-5
> > Severity: serious
> > 
> > Hi,
> > 
> > during the libical transition [1], your package syncevolution was
> > rebuilt and it now FTBFS:
> [...]
> > src/syncevo/.libs/libsyncevolution.so: undefined reference to
> > `ical_tzid_prefix'
> [...]
> 
> I've quickly looked into this issue since it was discovered
> (sorry I missed spotting it before the transition started).
> 
> The global variable referenced above is indeed gone (in general
> libical seems to be moving away from global variables in 2.0.0
> in favour of accessor methods). The new accessor function to
> get the value is called icaltimezone_tzid_prefix(). See:
> http://sources.debian.net/src/libical/2.0.0-0.4/src/libical/icaltimezone.c/?hl=150#L150
> 
> I looked upstream but there seemed to be very little activity in recent time.

There hasn't been much demand for new features lately, so I consider
SyncEvolution fairly stable and in maintenance now. However, making it
compile on newer systems certainly counts as "maintenance" - I'll look
into it.

The problem for me upstream nowadays is mostly around continuing to
provide pre-compiled binaries for different distros. I got stuck on that
the last time I tried preparing a new release, because distros have
diverged in quite a few relevant libraries now (libical, EDS, even
libopenobex). Perhaps I should just give up on that front.

OTOH, Debian itself is still on an old, pre-release version. Tino, do
you still have plans for getting SyncEvolution updated in Debian?

> The libical2 transition has already been done in fedora so for comparison
> they're patching their syncevolution, see:
> http://pkgs.fedoraproject.org/cgit/rpms/syncevolution.git/tree/syncevolution-1.5.1-libical2.patch
> 
> IMNSHO the fedora patch is an ugly hack since duplicating the information
> instead of using accessor method isn't nice IMO. OTOH I've looked at
> syncevolutions usage of the global variable and if we're going to replace
> it we see it in two places. The one failing the build above + in the
> syncevolution homebrew/duplicated evolution(-data-server?) API.
> That one looks too scary to touch for me who has no way to test this
> so I sympathize with fedoras patch and maybe it's better to go that
> route. Also, the get accessor seems to be missing from the libical2.symbols
> for some reason so might need fixes on the libical side to be properly
> exported

That scary code is needed only when compiling upstream binaries that
must work with different EDS versions. For Debian it might be enough to
just call the accessor method.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



Bug#824426: FTBFS against libical2

2016-05-16 Thread Andreas Henriksson
Hello!

Some information below which might be useful

On Sun, May 15, 2016 at 09:37:24PM +0200, Michael Biebl wrote:
> Source: syncevolution
> Version: 1.4.99.4-5
> Severity: serious
> 
> Hi,
> 
> during the libical transition [1], your package syncevolution was
> rebuilt and it now FTBFS:
[...]
> src/syncevo/.libs/libsyncevolution.so: undefined reference to
> `ical_tzid_prefix'
[...]

I've quickly looked into this issue since it was discovered
(sorry I missed spotting it before the transition started).

The global variable referenced above is indeed gone (in general
libical seems to be moving away from global variables in 2.0.0
in favour of accessor methods). The new accessor function to
get the value is called icaltimezone_tzid_prefix(). See:
http://sources.debian.net/src/libical/2.0.0-0.4/src/libical/icaltimezone.c/?hl=150#L150

I looked upstream but there seemed to be very little activity in recent time.
Possibly there's some bug tracker or mailing list post somewhere, but I
didn't search that far.

The libical2 transition has already been done in fedora so for comparison
they're patching their syncevolution, see:
http://pkgs.fedoraproject.org/cgit/rpms/syncevolution.git/tree/syncevolution-1.5.1-libical2.patch

IMNSHO the fedora patch is an ugly hack since duplicating the information
instead of using accessor method isn't nice IMO. OTOH I've looked at
syncevolutions usage of the global variable and if we're going to replace
it we see it in two places. The one failing the build above + in the
syncevolution homebrew/duplicated evolution(-data-server?) API.
That one looks too scary to touch for me who has no way to test this
so I sympathize with fedoras patch and maybe it's better to go that
route. Also, the get accessor seems to be missing from the libical2.symbols
for some reason so might need fixes on the libical side to be properly
exported

Regards,
Andreas Henriksson



Bug#824426: FTBFS against libical2

2016-05-15 Thread Michael Biebl
Source: syncevolution
Version: 1.4.99.4-5
Severity: serious

Hi,

during the libical transition [1], your package syncevolution was
rebuilt and it now FTBFS:


libtool: link: g++ -I/usr/include/glib-2.0
-I/usr/lib/x86_64-linux-gnu/glib-2.0/include -pthread
-I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
-pthread -I/usr/include/gtk-2.0
-I/usr/lib/x86_64-linux-gnu/gtk-2.0/include -I/usr/include/gio-unix-2.0/
-I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/atk-1.0
-I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/libpng16
-I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16
-I/usr/include/pango-1.0 -I/usr/include/harfbuzz
-I/usr/include/pango-1.0 -I/usr/include/glib-2.0
-I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/freetype2
-Wall -Wno-unknown-pragmas -Wno-deprecated-declarations -g -O2 -fPIE
-fstack-protector-strong -Wformat -Werror=format-security
-Wl,-uSyncEvolution_Module_Version -Wl,--export-dynamic
-Wl,-usyncevo_fetch_timezone -fPIE -pie -Wl,-z -Wl,relro -Wl,-z -Wl,now
-Wl,--as-needed -o src/.libs/syncevo-dbus-server -pthread
-L/usr/lib/x86_64-linux-gnu -lcppunit -ldl
./src/dbus/server/.libs/libsyncevodbusserver.a -lpcre -lpcrecpp
src/gdbusxx/.libs/libgdbussyncevo.so
src/syncevo/.libs/libsyncevolution.so -lgthread-2.0 -lnotify
-lgtk-x11-2.0 -lgdk-x11-2.0 -lpangocairo-1.0 -latk-1.0 -lcairo
-lgdk_pixbuf-2.0 -lpangoft2-1.0 -lpango-1.0 -lfontconfig
/usr/lib/x86_64-linux-gnu/libfreetype.so -lgio-2.0 -lgobject-2.0
-lglib-2.0 -pthread
src/syncevo/.libs/libsyncevolution.so: undefined reference to
`ical_tzid_prefix'
collect2: error: ld returned 1 exit status
Makefile:6187: recipe for target 'src/syncevolution' failed
make[3]: *** [src/syncevolution] Error 1
make[3]: *** Waiting for unfinished jobs
src/syncevo/.libs/libsyncevolution.so: undefined reference to
`ical_tzid_prefix'
collect2: error: ld returned 1 exit status
Makefile:6128: recipe for target 'src/syncevo-local-sync' failed
make[3]: *** [src/syncevo-local-sync] Error 1
src/syncevo/.libs/libsyncevolution.so: undefined reference to
`ical_tzid_prefix'
collect2: error: ld returned 1 exit status
Makefile:6069: recipe for target 'src/syncevo-dbus-server' failed
make[3]: *** [src/syncevo-dbus-server] Error 1
make[3]: Leaving directory '/«PKGBUILDDIR»'
Makefile:10646: recipe for target 'all-recursive' failed
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory '/«PKGBUILDDIR»'
Makefile:4226: recipe for target 'all' failed
make[1]: *** [all] Error 2

The full build log is available at [2]

Regards,
Michael

[1] https://release.debian.org/transitions/html/auto-libical.html
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=797074
[2] 
https://buildd.debian.org/status/fetch.php?pkg=syncevolution=amd64=1.4.99.4-5%2Bb1=1462978210




-- System Information:
Debian Release: stretch/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (200, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.5.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)