By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 2016

Summary:	Build issue: tqt3 FTBFS with postgresql >= 9.3.3-2
Product:	TDE	Reporter:	Slávek Banko <slavek.banko>
Component:	qt3	Assignee:	Michele Calgaro <michele.calgaro>
Status:	RESOLVED FIXED
Severity:	critical	CC:	bugwatch, michele.calgaro, slavek.banko
Priority:	P5
Version:	R14.0.0 [Trinity]
Hardware:	Other
OS:	Other
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Bug Depends on:
Bug Blocks:	2014
Attachments:	qt3 patch tqt3 patch

Description Slávek Banko 2014-03-22 14:25:41 CDT

In recent updates Postgresql on Debian Jessie was changed in include files - see 314427 in Debian bugs:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=314427

Combining that is used in TQt3 postgres.h (intended for server side) instead of postgres_fe.h (for client side), that is used to catalog/pg_type.h (intended for server side), and that the package libpq-dev is now broken, FTBFS occurs.

Comment 1 Michele Calgaro 2014-03-23 02:04:58 CDT

Confirmed.
Slavek, the bug is a FTBFS but Jessie is a rolling distro. Should we add this to the v14.0.0 bug list? Considering that sooner or later the package would go into Wheezy and that it will take at least 2-3 months of work before v14.0.0 is ready, I think we should.

Comment 2 Slávek Banko 2014-04-06 11:24:20 CDT

The importance of this bug is raised because in the upcoming Ubuntu 14.04 (Trusty) Postgresql has been updated to problematic version.

Comment 3 Michele Calgaro 2014-04-12 07:46:41 CDT

This bug prevents tqt3 from building in Jessie, so it is time it get addressed. A full TDE rebuild is way overdue on my system. I am going to work on this in this coming week.

Comment 4 Michele Calgaro 2014-04-13 22:02:31 CDT

Attached proposed patches for qt3 and tqt3.
I have tested on Debian Jessie and they work well. Based on information about Debian and Ubuntu packages, it should be ok also for all other Debian/Ubuntu distros.
Slavek, Tim, before I push to GIT, please test and confirm (at least on Ubuntu Trusty).

Comment 5 Michele Calgaro 2014-04-13 22:03:05 CDT

Created attachment 2034 [details]
qt3 patch

Comment 6 Michele Calgaro 2014-04-13 22:03:20 CDT

Created attachment 2035 [details]
tqt3 patch

Comment 7 Slávek Banko 2014-04-14 14:40:16 CDT

Excellent - beautifully simple. I tested Ubuntu Trusty and Lucid and both were built fine. I think that you can push the patch.

Comment 8 Michele Calgaro 2014-04-14 20:58:34 CDT

Pushed in commits bd47e75 (qt3) and fcd6bc2 (tqt3)

Comment 9 Slávek Banko 2014-04-16 08:44:42 CDT

Because launchpad on build-farm resolves dependencies a little differently than standard apt, dependencies was fixed in commit d3df6df7.

Comment 10 Michele Calgaro 2014-04-16 08:49:33 CDT

> Because launchpad on build-farm resolves dependencies a little differently
> than standard apt
Interesting, I didn't know that. What is the difference between the build-farm way and standard apt?

Comment 11 Slávek Banko 2014-04-16 15:06:44 CDT

(In reply to Michele Calgaro from comment #10)
> > Because launchpad on build-farm resolves dependencies a little differently
> > than standard apt
> Interesting, I didn't know that. What is the difference between the
> build-farm way and standard apt?

It seems that while pbuilder solves the installation of dependencies by creating an empty package (pbuilder-satisfydepends-dummy.deb) and using a standard installation by apt, launchpad is probably trying to resolve the dependencies itself. And I had already noticed problems if in the alternative dependency is the first package with the condition <= or <<.

Comment 12 Michele Calgaro 2014-04-16 21:47:42 CDT

> And I had already noticed problems if in the alternative dependency is the
> first package with the condition <= or <<.
Thanks for the explanation, Slavek.
I will try to remember this for future patches, so to avoid the same problem.