Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp
@ 2012-05-10 21:06 Andrei Gherzan
  2012-05-10 21:22 ` Khem Raj
  2012-05-16 17:59 ` Saul Wold
  0 siblings, 2 replies; 6+ messages in thread
From: Andrei Gherzan @ 2012-05-10 21:06 UTC (permalink / raw)
  To: openembedded-core

App/ExportWizard.cpp depends on wizard.h which depends on ui_wizard. The last one
should be already generated before compiling ExportWizard.cpp.

[YOCTO #2297]

Signed-off-by: Andrei Gherzan <andrei@gherzan.ro>
---
 .../ExportWizard-depends-on-ui_wizard.patch        |   18 ++++++++++++++++++
 meta/recipes-qt/qt-apps/fotowall_0.9.bb            |    6 ++++--
 2 files changed, 22 insertions(+), 2 deletions(-)
 create mode 100644 meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch

diff --git a/meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch b/meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch
new file mode 100644
index 0000000..d8b2b23
--- /dev/null
+++ b/meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch
@@ -0,0 +1,18 @@
+App/ExportWizard.cpp depends on wizard.h which depends on ui_wizard. The last one
+should be already generated before compiling ExportWizard.cpp.
+
+Signed-off-by: Andrei Gherzan <andrei@gherzan.ro>
+Upstream-Status: Pending
+
+Index: Fotowall-0.9/App/ExportWizard.cpp
+===================================================================
+--- Fotowall-0.9.orig/App/ExportWizard.cpp	2009-11-30 13:21:31.000000000 +0200
++++ Fotowall-0.9/App/ExportWizard.cpp	2012-05-10 17:12:06.765230830 +0300
+@@ -23,6 +23,7 @@
+ #include "imageloaderqt.h"
+ #include "posterazorcore.h"
+ #include "wizard.h"
++#include "ui_wizard.h"
+ 
+ #include <QDesktopServices>
+ #include <QDesktopWidget>
diff --git a/meta/recipes-qt/qt-apps/fotowall_0.9.bb b/meta/recipes-qt/qt-apps/fotowall_0.9.bb
index 52dd727..3cb79a3 100644
--- a/meta/recipes-qt/qt-apps/fotowall_0.9.bb
+++ b/meta/recipes-qt/qt-apps/fotowall_0.9.bb
@@ -7,12 +7,14 @@ LIC_FILES_CHKSUM = "file://GPL_V2;md5=79808397c3355f163c012616125c9e26 \
                     file://main.cpp;beginline=6;endline=11;md5=b569acc2bf8974a3082b58fc53b9d8dc"
 SECTION = "x11/apps"
 
-PR = "r2"
+PR = "r3"
 
 DEPENDS = "qt4-x11-free"
 RRECOMMENDS_${PN} += "qt4-plugin-imageformat-gif qt4-plugin-imageformat-jpeg qt4-plugin-imageformat-tiff"
 
-SRC_URI = "http://qt-apps.org/CONTENT/content-files/71316-Fotowall-0.9.tar.bz2"
+SRC_URI = "http://qt-apps.org/CONTENT/content-files/71316-Fotowall-0.9.tar.bz2 \
+           file://ExportWizard-depends-on-ui_wizard.patch \
+           "
 
 SRC_URI[md5sum] = "142ef697332e0777c6d22c5bc96cc438"
 SRC_URI[sha256sum] = "e4d0c005d2cb1d7c09438bfc3098eadebc08946e4fbc0655b7fc8b046de3810d"
-- 
1.7.9.5




^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp
  2012-05-10 21:06 [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp Andrei Gherzan
@ 2012-05-10 21:22 ` Khem Raj
  2012-05-10 21:26   ` Andrei Gherzan
  2012-05-16 17:59 ` Saul Wold
  1 sibling, 1 reply; 6+ messages in thread
From: Khem Raj @ 2012-05-10 21:22 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, May 10, 2012 at 2:06 PM, Andrei Gherzan <andrei@gherzan.ro> wrote:
>
> +App/ExportWizard.cpp depends on wizard.h which depends on ui_wizard. The last one
> +should be already generated before compiling ExportWizard.cpp.
> +

then shouldnt wizard.h  be including ui_wizard.h and not the .cpp file ?



^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp
  2012-05-10 21:22 ` Khem Raj
@ 2012-05-10 21:26   ` Andrei Gherzan
  2012-05-10 21:31     ` Khem Raj
  0 siblings, 1 reply; 6+ messages in thread
From: Andrei Gherzan @ 2012-05-10 21:26 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 744 bytes --]

And it does. But this is needed for qmake in Makefile generation. Didn't
find a better fix but this works as expected.

*@g*


On Fri, May 11, 2012 at 12:22 AM, Khem Raj <raj.khem@gmail.com> wrote:

> On Thu, May 10, 2012 at 2:06 PM, Andrei Gherzan <andrei@gherzan.ro> wrote:
> >
> > +App/ExportWizard.cpp depends on wizard.h which depends on ui_wizard.
> The last one
> > +should be already generated before compiling ExportWizard.cpp.
> > +
>
> then shouldnt wizard.h  be including ui_wizard.h and not the .cpp file ?
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 1469 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp
  2012-05-10 21:26   ` Andrei Gherzan
@ 2012-05-10 21:31     ` Khem Raj
  2012-05-13 19:33       ` Andrei Gherzan
  0 siblings, 1 reply; 6+ messages in thread
From: Khem Raj @ 2012-05-10 21:31 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Thu, May 10, 2012 at 2:26 PM, Andrei Gherzan <andrei@gherzan.ro> wrote:
> And it does. But this is needed for qmake in Makefile generation. Didn't
> find a better fix but this works as expected.

I see. So it seems makefile generator does not grok recursive includes ?
is that documented or a bug in the generator



^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp
  2012-05-10 21:31     ` Khem Raj
@ 2012-05-13 19:33       ` Andrei Gherzan
  0 siblings, 0 replies; 6+ messages in thread
From: Andrei Gherzan @ 2012-05-13 19:33 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 662 bytes --]

On Fri, May 11, 2012 at 12:31 AM, Khem Raj <raj.khem@gmail.com> wrote:

> On Thu, May 10, 2012 at 2:26 PM, Andrei Gherzan <andrei@gherzan.ro> wrote:
> > And it does. But this is needed for qmake in Makefile generation. Didn't
> > find a better fix but this works as expected.
>
> I see. So it seems makefile generator does not grok recursive includes ?
> is that documented or a bug in the generator
>
>
Didn't check this but i will. Thanks.
@g

_______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 1302 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp
  2012-05-10 21:06 [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp Andrei Gherzan
  2012-05-10 21:22 ` Khem Raj
@ 2012-05-16 17:59 ` Saul Wold
  1 sibling, 0 replies; 6+ messages in thread
From: Saul Wold @ 2012-05-16 17:59 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 05/11/2012 12:06 AM, Andrei Gherzan wrote:
> App/ExportWizard.cpp depends on wizard.h which depends on ui_wizard. The last one
> should be already generated before compiling ExportWizard.cpp.
>
> [YOCTO #2297]
>
> Signed-off-by: Andrei Gherzan<andrei@gherzan.ro>
> ---
>   .../ExportWizard-depends-on-ui_wizard.patch        |   18 ++++++++++++++++++
>   meta/recipes-qt/qt-apps/fotowall_0.9.bb            |    6 ++++--
>   2 files changed, 22 insertions(+), 2 deletions(-)
>   create mode 100644 meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch
>
> diff --git a/meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch b/meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch
> new file mode 100644
> index 0000000..d8b2b23
> --- /dev/null
> +++ b/meta/recipes-qt/qt-apps/fotowall/ExportWizard-depends-on-ui_wizard.patch
> @@ -0,0 +1,18 @@
> +App/ExportWizard.cpp depends on wizard.h which depends on ui_wizard. The last one
> +should be already generated before compiling ExportWizard.cpp.
> +
> +Signed-off-by: Andrei Gherzan<andrei@gherzan.ro>
> +Upstream-Status: Pending
> +
> +Index: Fotowall-0.9/App/ExportWizard.cpp
> +===================================================================
> +--- Fotowall-0.9.orig/App/ExportWizard.cpp	2009-11-30 13:21:31.000000000 +0200
> ++++ Fotowall-0.9/App/ExportWizard.cpp	2012-05-10 17:12:06.765230830 +0300
> +@@ -23,6 +23,7 @@
> + #include "imageloaderqt.h"
> + #include "posterazorcore.h"
> + #include "wizard.h"
> ++#include "ui_wizard.h"
> +
> + #include<QDesktopServices>
> + #include<QDesktopWidget>
> diff --git a/meta/recipes-qt/qt-apps/fotowall_0.9.bb b/meta/recipes-qt/qt-apps/fotowall_0.9.bb
> index 52dd727..3cb79a3 100644
> --- a/meta/recipes-qt/qt-apps/fotowall_0.9.bb
> +++ b/meta/recipes-qt/qt-apps/fotowall_0.9.bb
> @@ -7,12 +7,14 @@ LIC_FILES_CHKSUM = "file://GPL_V2;md5=79808397c3355f163c012616125c9e26 \
>                       file://main.cpp;beginline=6;endline=11;md5=b569acc2bf8974a3082b58fc53b9d8dc"
>   SECTION = "x11/apps"
>
> -PR = "r2"
> +PR = "r3"
>
>   DEPENDS = "qt4-x11-free"
>   RRECOMMENDS_${PN} += "qt4-plugin-imageformat-gif qt4-plugin-imageformat-jpeg qt4-plugin-imageformat-tiff"
>
> -SRC_URI = "http://qt-apps.org/CONTENT/content-files/71316-Fotowall-0.9.tar.bz2"
> +SRC_URI = "http://qt-apps.org/CONTENT/content-files/71316-Fotowall-0.9.tar.bz2 \
> +           file://ExportWizard-depends-on-ui_wizard.patch \
> +           "
>
>   SRC_URI[md5sum] = "142ef697332e0777c6d22c5bc96cc438"
>   SRC_URI[sha256sum] = "e4d0c005d2cb1d7c09438bfc3098eadebc08946e4fbc0655b7fc8b046de3810d"

Merged into OE-Core

Thanks
	Sau!



^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-05-16 18:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-05-10 21:06 [PATCH V3] fotowall: Add #include ui_wizard.h to ExportWizard.cpp Andrei Gherzan
2012-05-10 21:22 ` Khem Raj
2012-05-10 21:26   ` Andrei Gherzan
2012-05-10 21:31     ` Khem Raj
2012-05-13 19:33       ` Andrei Gherzan
2012-05-16 17:59 ` Saul Wold

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox