All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-oe][RFC] Need for qmake_base.bbclass?
@ 2011-08-16  7:49 Anders Darander
  2011-08-16  8:59 ` Paul Eggleton
  0 siblings, 1 reply; 3+ messages in thread
From: Anders Darander @ 2011-08-16  7:49 UTC (permalink / raw)
  To: oe-dev


Hi,

I've diff'ed qmake_base.bbclass in meta-oe and oe-core, with the result
at the end of the e-mail.

I'd like to get rid of qmake_base.bbclass in oe-core.
If we merge the second hunk of the diff to oe-core, do we risk getting
any failures/problems?

Locally, I've been running without the meta-oe version for a while, but
I'm only building a subset of all packages (and a subset of Qt as well).

Cheers,
Anders
 

--- ../openembedded-core/meta/classes/qmake_base.bbclass
2011-06-09 22:05:03.030620629 +0200
+++ meta-oe/classes/qmake_base.bbclass  2011-08-16 08:34:54.175700192
+0200
@@ -13,6 +13,9 @@
 export OE_QMAKE_STRIP="echo"
 export OE_QMAKE_RPATH="-Wl,-rpath-link,"
 
+# do not export STRIP to the environment
+STRIP[unexport] = "1"
+
 # default to qte2 via bb.conf, inherit qt3x11 to configure for qt3x11
 
 oe_qmake_mkspecs () {
@@ -47,7 +50,7 @@
 
 qmake_base_do_configure() {
        case ${QMAKESPEC} in
-       *linux-oe-g++|*linux-uclibc-oe-g++|*linux-gnueabi-oe-g++)
+
*linux-oe-g++|*linux-uclibc-oe-g++|*linux-gnueabi-oe-g++|*linux-uclibceabi-oe-g++)
                ;;
        *-oe-g++)
                die Unsupported target ${TARGET_OS} for oe-g++ qmake
spec
@@ -92,11 +95,6 @@
                bbnote "qmake prevar substitution:
${EXTRA_QMAKEVARS_PRE}"
        fi
 
-       # Hack .pro files to use OE utilities
-       find -name '*.pro' \
-            -exec sed -i -e 's,=\s*.*/lrelease,=
${OE_QMAKE_LRELEASE},g' \
-                         -e 's,=\s*.*/lupdate,= ${OE_QMAKE_LUPDATE},g'
'{}' ';'
-
 #bbnote "Calling '${OE_QMAKE_QMAKE} -makefile -spec ${QMAKESPEC} -o
Makefile $QMAKE_VARSUBST_PRE $AFTER $PROFILES $QMAKE_VARSUBST_POST'"
        unset QMAKESPEC || true
        ${OE_QMAKE_QMAKE} -makefile -spec ${QMAKESPEC} -o Makefile
$QMAKE_VARSUBST_PRE $AFTER $PROFILES $QMAKE_VARSUBST_POST || die "Error
calling ${OE_QMAKE_QMAKE} on $PROFILES"

-- 
Anders Darander
ChargeStorm AB



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [meta-oe][RFC] Need for qmake_base.bbclass?
  2011-08-16  7:49 [meta-oe][RFC] Need for qmake_base.bbclass? Anders Darander
@ 2011-08-16  8:59 ` Paul Eggleton
  2011-08-16  9:09   ` Anders Darander
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Eggleton @ 2011-08-16  8:59 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Anders Darander

On Tuesday 16 August 2011 08:49:08 Anders Darander wrote:
> I'd like to get rid of qmake_base.bbclass in oe-core.

Presumably you mean "in meta-oe" here...

> If we merge the second hunk of the diff to oe-core, do we risk getting
> any failures/problems?

Looks pretty reasonable to me, at a glance I can't see it causing any issues.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [meta-oe][RFC] Need for qmake_base.bbclass?
  2011-08-16  8:59 ` Paul Eggleton
@ 2011-08-16  9:09   ` Anders Darander
  0 siblings, 0 replies; 3+ messages in thread
From: Anders Darander @ 2011-08-16  9:09 UTC (permalink / raw)
  To: openembedded-devel@lists.openembedded.org

* Paul Eggleton <paul.eggleton@linux.intel.com> [110816 10:59]:
> On Tuesday 16 August 2011 08:49:08 Anders Darander wrote:
> > I'd like to get rid of qmake_base.bbclass in oe-core.

> Presumably you mean "in meta-oe" here...

Yes, that's right. (I don't know what it is with me and e-mail the
latest couple of days...)

> > If we merge the second hunk of the diff to oe-core, do we risk getting
> > any failures/problems?

> Looks pretty reasonable to me, at a glance I can't see it causing any issues.

Good to hear. If no-one else has any objections, I'll submit a patch
delete qmake_base.bbclass, after submitting and getting my oe-core
patches merged.

Cheers,
Anders

-- 
Anders Darander
ChargeStorm AB



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-08-16  9:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-16  7:49 [meta-oe][RFC] Need for qmake_base.bbclass? Anders Darander
2011-08-16  8:59 ` Paul Eggleton
2011-08-16  9:09   ` Anders Darander

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.