Bug 184332 - compiling error with Qt4.5
Summary: compiling error with Qt4.5
Status: RESOLVED UPSTREAM
Alias: None
Product: digikam
Classification: Applications
Component: Portability-Compilation (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-14 19:46 UTC by Salvatore Brigaglia
Modified: 2018-03-23 11:43 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 0.2.0


Attachments
the errors i get compiling kipi-plugins (894 bytes, text/plain)
2009-02-14 19:49 UTC, Salvatore Brigaglia
Details
Patch to avoid uic problem (6.69 KB, patch)
2009-02-15 23:47 UTC, Angelo Naselli
Details
Patch (workaround) to avoid uic problem (743 bytes, patch)
2009-02-19 16:34 UTC, Kevin Kofler
Details
31473: Patch (workaround) to avoid uic problem (1.07 KB, patch)
2009-02-19 16:39 UTC, Kevin Kofler
Details
Patch (workaround) to avoid uic problem (1.07 KB, patch)
2009-02-19 16:42 UTC, Kevin Kofler
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Salvatore Brigaglia 2009-02-14 19:46:07 UTC
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

As the description. After upgrading to Qt4.5 it is impossible to get kipi-plugin compiled. Errors attached.
Comment 1 Salvatore Brigaglia 2009-02-14 19:49:32 UTC
Created attachment 31303 [details]
the errors i get compiling kipi-plugins
Comment 2 Christoph Feck 2009-02-15 15:45:09 UTC
I can confirm the error, but it looks like a problem in Qt 4.5 uic; the problem has been reported on qt-preview-feedback. I will keep you informed what the exact cause is.
Comment 3 Angelo Naselli 2009-02-15 23:47:52 UTC
Created attachment 31349 [details]
Patch to avoid uic problem

I've found an ugly workaround to avoid the problem, that should work also with old qt4.x (x<5). I do hope qt fixes the problem but by now this should work.
Comment 4 Nicolas L. 2009-02-16 14:04:20 UTC
works OK for me
Comment 5 Kevin Kofler 2009-02-19 16:34:50 UTC
Created attachment 31473 [details]
Patch (workaround) to avoid uic problem

Here's a cleaned up version of the patch (without irrelevant changes).
Comment 6 Kevin Kofler 2009-02-19 16:39:30 UTC
Created attachment 31474 [details]
31473: Patch (workaround) to avoid uic problem

Oops, I was missing a required line (lost in the mess of unrelated XML reformatting). This one should actually work.
Comment 7 Kevin Kofler 2009-02-19 16:42:28 UTC
Created attachment 31475 [details]
Patch (workaround) to avoid uic problem

Hopefully the third one is the good one. ;-)
Fixed my whitespace, also removed unneeded changes from one of the changed lines.
Comment 8 caulier.gilles 2009-02-19 16:54:26 UTC
And compilation work again with Qt4.4 ?

Gilles
Comment 9 caulier.gilles 2009-02-19 16:55:41 UTC
sorry, i want mean Qt 4.4 and KDE 4.1.x ?

Gilles
Comment 10 Angelo Naselli 2009-02-19 17:46:51 UTC
I believe if the old one worked for you it should this one also :)
The removed lines are the one qt 4.5 assistant added/changed.
In any case the workaround is silly, i added a file with a redefinition
of Phonon::VolumeSlider and used a local file into ui instead of the global one :)

Angelo
Comment 11 Angelo Naselli 2009-02-19 17:50:25 UTC
For the record why "unneeded" changes were so bad? (I'm usually against to change ui files by hands....)
Comment 12 caulier.gilles 2009-02-19 18:09:34 UTC
This is why i hate to work with ui files. why this must require a change like this ?

Gilles Caulier
Comment 13 caulier.gilles 2009-02-23 09:36:02 UTC
Laurent Montel has just commited a patch in svn:

SVN commit 930363 by mlaurent:

Add patch from #184332  to allow to compile it


 M  +3 -3      playbackwidget.ui
 A             volslider.h   [License: UNKNOWN]


--- trunk/extragear/graphics/kipi-plugins/advancedslideshow/playbackwidget.ui #930362:930363
@@ -107,7 +107,7 @@
      </layout>
     </item>
     <item>
-      <widget class="Phonon::VolumeSlider" native="1" name="m_volumeSlider" >
+      <widget class="VolSlider" native="1" name="m_volumeSlider" >
       <property name="sizePolicy" >
        <sizepolicy vsizetype="Preferred" hsizetype="Expanding" >
         <horstretch>0</horstretch>
@@ -132,9 +132,9 @@
 </widget>
 <customwidgets>
  <customwidget>
-   <class>Phonon::VolumeSlider</class>
+   <class>VolSlider</class>
   <extends>QWidget</extends>
-   <header location="global" >Phonon/VolumeSlider</header>
+   <header>volslider.h</header>
   <container>1</container>
  </customwidget>
 </customwidgets>

Please test and give feedback. Thanks in advance

Gilles Cauleir
Comment 14 caulier.gilles 2009-02-23 09:38:34 UTC
all compile fine here with Qt 4.4 and KDE 4.1.3:

[gilles@pc-caulier advancedslideshow]$ make
[ 33%] Building CXX object kipi-plugins/common/libkipiplugins/CMakeFiles/kipiplugins.dir/kpaboutdata.o
Linking CXX shared library ../../../lib/libkipiplugins.so
[ 33%] Built target kipiplugins
[ 33%] Generating ui_playbackwidget.h
Generating slideshow.moc
[ 33%] Generating ui_captiondialog.h
Scanning dependencies of target kipiplugin_advancedslideshow
[ 33%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/common.o
[ 33%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/kbeffect.o
[ 33%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshowkb.o
[ 33%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/imageloadthread.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/maindialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/captiondialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/soundtrackdialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/advanceddialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/playbackwidget.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshow.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshowconfig.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshowgl.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/plugin_advancedslideshow.o
Linking CXX shared module ../../lib/kipiplugin_advancedslideshow.so
[100%] Built target kipiplugin_advancedslideshow

Gilles Caulier
Comment 15 caulier.gilles 2009-02-23 09:51:42 UTC
Compile fine too with KDE 4.2 and Qt 4.4:

[gilles@localhost kipi-plugins]$
[gilles@localhost advancedslideshow]$ make
[ 33%] Built target kipiplugins
[ 33%] Generating ui_playbackwidget.h
Generating slideshow.moc
[ 33%] Generating ui_captiondialog.h
Scanning dependencies of target kipiplugin_advancedslideshow
[ 33%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/common.o
[ 33%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/kbeffect.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshowkb.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/imageloadthread.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/maindialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/captiondialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/soundtrackdialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/advanceddialog.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/playbackwidget.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshow.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshowconfig.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/slideshowgl.o
[ 66%] Building CXX object kipi-plugins/advancedslideshow/CMakeFiles/kipiplugin_advancedslideshow.dir/plugin_advancedslideshow.o
Linking CXX shared module ../../lib/kipiplugin_advancedslideshow.so
[100%] Built target kipiplugin_advancedslideshow

Someone can check with Qt 4.5 now ?

Gilles Caulier
Comment 16 Salvatore Brigaglia 2009-02-23 09:54:55 UTC
(In reply to comment #15)

> Someone can check with Qt 4.5 now ?
> 
> Gilles Caulier

Doing it right now. Will report in 10 minutes.
Comment 17 Salvatore Brigaglia 2009-02-23 10:44:42 UTC
(In reply to comment #16)
> (In reply to comment #15)
> 
> > Someone can check with Qt 4.5 now ?
> > 
> > Gilles Caulier
> 
> Doing it right now. Will report in 10 minutes.

It is compiling now (and working ;)). So i'm closing the bug as fixed.

Thanks!
Comment 18 Rex Dieter 2009-02-23 19:24:20 UTC
This looks like the relevant qt bug:
http://www.qtsoftware.com/developer/task-tracker/index_html?method=entry&id=244998
Comment 19 Angelo Naselli 2009-02-23 23:29:32 UTC
Gilles since it is a qt 4.5 rc1 bug, and should be fixed in final shouldn't be better  leaving this bug open and leaving the patch as a workaround for packagers? I mean that should be fixed upstream soon and mine is a ugly workaround...
Comment 20 caulier.gilles 2009-02-24 06:20:15 UTC
I set fixed by upstream.

Gilles