Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(272)

Unified Diff: chromeos-base/cashew/cashew-9999.ebuild

Issue 5718004: cashew ebuild: add cros-debug support (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/chromiumos-overlay.git@master
Patch Set: Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos-base/cashew/cashew-9999.ebuild
diff --git a/chromeos-base/cashew/cashew-9999.ebuild b/chromeos-base/cashew/cashew-9999.ebuild
index 0c700226de54900efc7878303a62d4e5cf30f7b7..79cfe0befcbc5639abc981918c52199f1c9b80d0 100644
--- a/chromeos-base/cashew/cashew-9999.ebuild
+++ b/chromeos-base/cashew/cashew-9999.ebuild
@@ -3,7 +3,7 @@
EAPI=2
-inherit cros-workon autotools
+inherit cros-debug cros-workon autotools
DESCRIPTION="Chromium OS network usage tracking daemon"
HOMEPAGE="http://src.chromium.org"
@@ -16,7 +16,7 @@ IUSE="test"
RDEPEND="chromeos-base/flimflam
chromeos-base/libchrome
dev-cpp/gflags
- dev-cpp/glog
+ >=dev-cpp/glog-0.3.1
dev-libs/dbus-c++
dev-libs/glib
net-misc/curl"
@@ -25,7 +25,13 @@ DEPEND="${RDEPEND}
test? ( dev-cpp/gtest )"
src_prepare() {
- eautoreconf
+ eautoreconf || die "eautoreconf failed"
+}
+
+src_configure() {
+ # set NDEBUG (or not) based on value of cros-debug USE flag
+ cros-debug-add-NDEBUG
+ econf || die "econf failed"
Daniel Kurtz 2010/12/14 04:14:21 It seems that most other ebuilds don't die on econ
Vince Laviano 2010/12/14 21:16:28 I suspect that die clauses are only needed to retu
}
src_compile() {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698