From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.windriver.com (mail.windriver.com [147.11.1.11]) by mail.openembedded.org (Postfix) with ESMTP id C60766AC7B for ; Wed, 3 Sep 2014 02:39:22 +0000 (UTC) Received: from ALA-HCA.corp.ad.wrs.com (ala-hca.corp.ad.wrs.com [147.11.189.40]) by mail.windriver.com (8.14.9/8.14.5) with ESMTP id s832dNWe011600 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Tue, 2 Sep 2014 19:39:23 -0700 (PDT) Received: from [128.224.162.187] (128.224.162.187) by ALA-HCA.corp.ad.wrs.com (147.11.189.50) with Microsoft SMTP Server (TLS) id 14.3.174.1; Tue, 2 Sep 2014 19:39:22 -0700 Message-ID: <54067F64.8040309@windriver.com> Date: Wed, 3 Sep 2014 10:39:32 +0800 From: ChenQi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.0 MIME-Version: 1.0 To: "Burton, Ross" References: <87c78f60aa3ef60a850e81fe49b0f4d33a142972.1409655125.git.Qi.Chen@windriver.com> In-Reply-To: X-Originating-IP: [128.224.162.187] Cc: OE-core Subject: Re: [PATCH 12/26] acpid: upgrade to 2.0.22 and add systemd support X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 03 Sep 2014 02:39:22 -0000 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit On 09/02/2014 11:24 PM, Burton, Ross wrote: > On 2 September 2014 11:53, Chen Qi wrote: >> -inherit update-rc.d >> +inherit autotools update-rc.d systemd >> >> INITSCRIPT_NAME = "acpid" >> INITSCRIPT_PARAMS = "defaults" >> >> -# Makefile ignores our CFLAGS, so override it. >> -# >> -EXTRA_OEMAKE = "CFLAGS='-W -Wall -Werror -Wundef -Wshadow ${CFLAGS} $(DEFS)'" >> +SYSTEMD_PACKAGES = "acpid" >> +SYSTEMD_SERVICE_acpid = "acpid.service" >> >> -do_compile () { >> - oe_runmake 'CC=${CC} -D_GNU_SOURCE' 'CROSS=${HOST_PREFIX}' >> -} > In general it would be nice if the commit mentioned "upstream now uses > autotools" to make it clear that changes to the build were > intentional. > > Ross > > Got it! Thanks, Chen Qi