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 11 - Build issue: Kile-kde3 FTBFS
Summary: Build issue: Kile-kde3 FTBFS
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdeaddons (show other bugs)
Version: 3.5.10
Hardware: All Linux
: P4 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks: 7
  Show dependency treegraph
 
Reported: 2008-12-12 14:39 CST by Timothy Pearson
Modified: 2013-12-08 20:22 CST (History)
4 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:


Attachments
07_kde-config.diff (936 bytes, patch)
2008-12-22 00:32 CST, Wouter Deconinck
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Pearson 2008-12-12 14:39:37 CST
Kile-kde3 FTBFS with:
make[4]: Entering directory `/build/buildd/kile-kde3-2.0.2/obj-x86_64-linux-gnu/src/kile'
g++ -Wno-long-long -Wundef -ansi -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align -Wchar-subscripts -Wall -W -Wpointer-arith -DNDEBUG -DNO_DEBUG -O2 -g -O2 -g -Wall -O2 -Wformat-security -Wmissing-format-attribute -Wno-non-virtual-dtor -fno-exceptions -fno-check-new -fno-common -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL -DQT_NO_COMPAT -DQT_NO_TRANSLATION -DQT_THREAD_SUPPORT  -D_REENTRANT -Wl,--as-needed  /build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp   -o tips
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:2: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:7: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:13: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:18: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:23: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:28: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:35: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:41: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:47: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:52: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:58: error: expected constructor, destructor, or type conversion before '(' token
/build/buildd/kile-kde3-2.0.2/./src/kile/tips.cpp:64: error: expected constructor, destructor, or type conversion before '(' token
make[4]: *** [tips] Error 1
make[4]: Leaving directory `/build/buildd/kile-kde3-2.0.2/obj-x86_64-linux-gnu/src/kile'
make[3]: *** [install-recursive] Error 1
make[3]: Leaving directory `/build/buildd/kile-kde3-2.0.2/obj-x86_64-linux-gnu/src/kile'
make[2]: *** [install-recursive] Error 1
make[2]: Leaving directory `/build/buildd/kile-kde3-2.0.2/obj-x86_64-linux-gnu/src'
make[1]: *** [install-recursive] Error 1
make[1]: Leaving directory `/build/buildd/kile-kde3-2.0.2/obj-x86_64-linux-gnu'
make: *** [common-install-impl] Error 2
Comment 1 Stefan Endrullis 2008-12-18 17:25:36 CST
I could not find this package ("kile-kde3"). Am I missing something or has it been removed?
Comment 2 Timothy Pearson 2008-12-18 17:56:23 CST
It was never added for public download to the PPA due to the FTBFS (I don't have control over that, it a Launchpad rule).  If you want to work on the problem, I will be glad to send you the source code--just let me know!
Comment 3 Wouter Deconinck 2008-12-19 10:54:02 CST
I'd be interested in taking a look at this.  I couldn't find the tips.cpp file in the kile source packages in intrepid (2.0.1) or in the official kile packages (2.0.1, 2.0.2, 2.0.3).  I don't have an x86_64 architecture to test on, but would cross compile to get it built cleanly.
Comment 4 Timothy Pearson 2008-12-19 14:32:22 CST
I have uploaded the source here: http://apt.pearsoncomputing.net/kile.uncompiled.tar.gz

As far as I can tell, it fails to build on all platforms, not just amd64.

If you get it working, just send the source back to me and I will upload to the PPA.

Thanks!
Comment 5 Stefan Endrullis 2008-12-20 05:05:37 CST
Thanks. I've also tried to build kile with gcc 4.3.2, but I got the same compiling error.

Sorry, but I'm not going to search for a solution for this bug, because for me it doesn't matter if I'm using kile or kate. Even the compilation feature of kile is not as good as a perfect makefile. And I like to use inotify_wait to run an automatic compilation after one of the documents changed, which is not possible in kile, I think.
Comment 6 Wouter Deconinck 2008-12-21 20:26:07 CST
I am confused now.  I have no problems at all with the source that you uploaded (without any modifications to it).  This is all I do
	wget http://apt.pearsoncomputing.net/kile.uncompiled.tar.gz
	tar -zxvf kile.uncompiled.tar.gz
	cd kile.uncompiled/kile-2.0.1
	dpkg-buildpackage
and this produces a deb package (in kile.uncopiled) without any errors concerning tips.cpp (no tips.cpp is ever compiled).  The package then installs cleanly with dpkg -i, and runs fine (I only opened an existing project, compiled and viewed the result).  Tips in English work too.  Are there any other build commands or targets that are expected by launchpad ppa?  Is there an easy way to reduce my local build environment to just what is available on launchpad (using a chroot jail).  I'm not an expert on debian packaging, as you can see...

This problem with tips.cpp was actually encountered before:
In https://bugs.launchpad.net/ubuntu/+source/kile/+bug/110779 a patch 07_remove_tips_cpp.diff was added, but this was included upstream in the 2.0 series (see Changelog of the ubuntu kile package).  The only thing the patch does is remove the tips.cpp file right after it is created by preparetips.  This still happens in the Makefiles that are included in your uploaded source archive, so no tips.cpp should ever exist long enough to be compiled.
Comment 7 Timothy Pearson 2008-12-21 22:21:47 CST
(In reply to comment #6)
> I am confused now.  I have no problems at all with the source that you uploaded
> (without any modifications to it).  This is all I do
>     wget http://apt.pearsoncomputing.net/kile.uncompiled.tar.gz
>     tar -zxvf kile.uncompiled.tar.gz
>     cd kile.uncompiled/kile-2.0.1
>     dpkg-buildpackage
> and this produces a deb package (in kile.uncopiled) without any errors
> concerning tips.cpp (no tips.cpp is ever compiled).  The package then installs
> cleanly with dpkg -i, and runs fine (I only opened an existing project,
> compiled and viewed the result).  Tips in English work too.  Are there any
> other build commands or targets that are expected by launchpad ppa?  Is there
> an easy way to reduce my local build environment to just what is available on
> launchpad (using a chroot jail).  I'm not an expert on debian packaging, as you
> can see...
> 
> This problem with tips.cpp was actually encountered before:
> In https://bugs.launchpad.net/ubuntu/+source/kile/+bug/110779 a patch
> 07_remove_tips_cpp.diff was added, but this was included upstream in the 2.0
> series (see Changelog of the ubuntu kile package).  The only thing the patch
> does is remove the tips.cpp file right after it is created by preparetips. 
> This still happens in the Makefiles that are included in your uploaded source
> archive, so no tips.cpp should ever exist long enough to be compiled.

I don't understand it either.  It worked for me as well, but look at what happens when the PPA builds it: http://launchpadlibrarian.net/20627427/buildlog_ubuntu-intrepid-i386.kile-kde3_1%3A2.0.2-0ubuntu1_FAILEDTOBUILD.txt.gz
Comment 8 Wouter Deconinck 2008-12-22 00:30:32 CST
It seems that the problem is in the following blurb from your FTBFS log (I didn't get those errors, and dismissed them when I looked at the FTBFS log earlier).

---
XGETTEXT=/usr/bin/kde-xgettext EXTRACTATTR=/usr/bin/extractattr sh ./admin/cvs.sh extract-messages
./src/kile
./admin/cvs.sh: 651: kde-config: not found
./admin/cvs.sh: 651: kde-config: not found
error while opening "/include/kde.pot" for reading: No such file or directory
make exit code: 2
for file in po/*pot; do \
		sed "s/charset=CHARSET/charset=UTF-8/" -i $file; \
	done
sed: can't read po/*pot: No such file or directory
make: [common-install-prehook-impl] Error 2 (ignored)
---

The script admin/cvs.sh expects kde-config in the $PATH to determine the KDE prefix, which is used to find the location of kde.pot (for the environment variable $kdepotpath).  But kde-config is installed under the /usr/kde3 prefix... so it's not found.  Unsetting the kde3 prefix from $PATH reproduces exactly the launchpad error.

admin/cvs.sh goes on to call 'make messages' in src/kile which generates the tips.cpp file, and then tries to run $XGETTEXT with argument $kdepotpath.  Normally as next command tips.cpp would be deleted again, but make fails because it can't find any kde.pot and it does not get to deleting tips.cpp.  Thus tips.cpp fails compilation.


Now, as a solution I introduced an environment variable kdeconfig in admin/cvs.sh.  If kde-config is not found in the path, it just hardwires it to the /usr/kde3/bin location.  It will then find kde.pot, xgettext will succeed and tips.cpp will be deleted.

See the attached patch for admin/cvs.sh.  When applied (and without kde-config in the $PATH) it now succeeds.  I didn't test this on launchpad, but I expect it should work (but that's what they all say ;-) ).
Comment 9 Wouter Deconinck 2008-12-22 00:32:24 CST
Created attachment 1 [details]
07_kde-config.diff

Patch improves search for kde-config in admin/cvs.sh.
Comment 10 Timothy Pearson 2008-12-22 15:54:46 CST
Thank you for the patch--kile is now built properly by the PPA.