From: Andre McCurdy <armccurdy@gmail.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH v2 7/8] puzzles: move ASNEEDED over-ride into the puzzles recipe
Date: Wed, 9 May 2018 14:46:47 -0700 [thread overview]
Message-ID: <1525902408-6796-7-git-send-email-armccurdy@gmail.com> (raw)
In-Reply-To: <1525902408-6796-1-git-send-email-armccurdy@gmail.com>
Move the recipe specific over-ride for ASNEEDED into the recipe to
make it more apparent that the over-ride is being applied (and that
it should be re-checked on version updates, etc).
Also misc minor recipe cleanup (re-order variables to follow the OE
style guide, etc).
Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
---
meta/conf/distro/include/as-needed.inc | 1 -
meta/recipes-sato/puzzles/puzzles_git.bb | 35 +++++++++++++++++---------------
2 files changed, 19 insertions(+), 17 deletions(-)
diff --git a/meta/conf/distro/include/as-needed.inc b/meta/conf/distro/include/as-needed.inc
index c67f2f3..f56d432 100644
--- a/meta/conf/distro/include/as-needed.inc
+++ b/meta/conf/distro/include/as-needed.inc
@@ -2,7 +2,6 @@
ASNEEDED = "-Wl,--as-needed"
ASNEEDED_pn-openobex = ""
-ASNEEDED_pn-puzzles = ""
ASNEEDED_pn-rpm = ""
TARGET_LDFLAGS += "${ASNEEDED}"
diff --git a/meta/recipes-sato/puzzles/puzzles_git.bb b/meta/recipes-sato/puzzles/puzzles_git.bb
index 3670ab8..7d08b96 100644
--- a/meta/recipes-sato/puzzles/puzzles_git.bb
+++ b/meta/recipes-sato/puzzles/puzzles_git.bb
@@ -1,20 +1,20 @@
SUMMARY = "Simon Tatham's Portable Puzzle Collection"
HOMEPAGE = "http://www.chiark.greenend.org.uk/~sgtatham/puzzles/"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://LICENCE;md5=da6110d4ed1225a287eab2bf0ac0193b"
DEPENDS = "libxt"
# The libxt requires x11 in DISTRO_FEATURES
REQUIRED_DISTRO_FEATURES = "x11"
-LICENSE = "MIT"
-LIC_FILES_CHKSUM = "file://LICENCE;md5=da6110d4ed1225a287eab2bf0ac0193b"
-
SRC_URI = "git://git.tartarus.org/simon/puzzles.git \
file://fix-compiling-failure-with-option-g-O.patch \
file://0001-Use-labs-instead-of-abs.patch \
file://0001-palisade-Fix-warnings-with-clang-on-arm.patch \
file://0001-Use-Wno-error-format-overflow-if-the-compiler-suppor.patch \
"
+
UPSTREAM_CHECK_COMMITS = "1"
SRCREV = "c6e0161dd475415316ed66dc82794d68e52f0025"
PE = "2"
@@ -24,24 +24,13 @@ S = "${WORKDIR}/git"
inherit autotools distro_features_check pkgconfig
-CFLAGS_append = " -Wno-deprecated-declarations"
-
PACKAGECONFIG ??= "gtk3"
PACKAGECONFIG[gtk2] = "--with-gtk=2,,gtk+,"
PACKAGECONFIG[gtk3] = "--with-gtk=3,,gtk+3,"
-PACKAGES += "${PN}-extra"
-FILES_${PN} = ""
-FILES_${PN}-extra = "${prefix}/bin ${datadir}/applications"
+CFLAGS_append = " -Wno-deprecated-declarations"
-python __anonymous () {
- var = d.expand("FILES_${PN}")
- data = d.getVar(var, False)
- for name in ("bridges", "fifteen", "inertia", "map", "samegame", "slant"):
- data = data + " ${bindir}/%s" % name
- data = data + " ${datadir}/applications/%s.desktop" % name
- d.setVar(var, data)
-}
+ASNEEDED = ""
do_configure_prepend () {
cd ${S}
@@ -74,3 +63,17 @@ STOP
fi
done
}
+
+PACKAGES += "${PN}-extra"
+
+FILES_${PN} = ""
+FILES_${PN}-extra = "${prefix}/bin ${datadir}/applications"
+
+python __anonymous () {
+ var = d.expand("FILES_${PN}")
+ data = d.getVar(var, False)
+ for name in ("bridges", "fifteen", "inertia", "map", "samegame", "slant"):
+ data = data + " ${bindir}/%s" % name
+ data = data + " ${datadir}/applications/%s.desktop" % name
+ d.setVar(var, data)
+}
--
1.9.1
next prev parent reply other threads:[~2018-05-09 21:47 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-09 21:46 [PATCH v2 1/8] tune-corei7.inc: minor comment tweak to align with tune-core2.inc Andre McCurdy
2018-05-09 21:46 ` [PATCH v2 2/8] babeltrace: move ASNEEDED over-ride into the babeltrace recipe Andre McCurdy
2018-05-09 22:04 ` Khem Raj
2018-05-09 22:52 ` Andre McCurdy
2018-05-09 21:46 ` [PATCH v2 3/8] console-tools: move ASNEEDED over-ride into the console-tools recipe Andre McCurdy
2018-05-09 21:46 ` [PATCH v2 4/8] distcc: move ASNEEDED over-ride into the distcc recipe Andre McCurdy
2018-05-09 21:46 ` [PATCH v2 5/8] icu: move ASNEEDED over-ride into icu.inc Andre McCurdy
2018-05-09 21:46 ` [PATCH v2 6/8] pciutils: move ASNEEDED over-ride into the pciutils recipe Andre McCurdy
2018-05-09 21:46 ` Andre McCurdy [this message]
2018-05-09 21:46 ` [PATCH v2 8/8] rpm: move ASNEEDED over-ride into the rpm recipe Andre McCurdy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1525902408-6796-7-git-send-email-armccurdy@gmail.com \
--to=armccurdy@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox