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 E180E75870 for ; Tue, 14 Jul 2015 01:16:48 +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.15.1/8.15.1) with ESMTPS id t6E1GjUD013562 (version=TLSv1 cipher=AES128-SHA bits=128 verify=FAIL); Mon, 13 Jul 2015 18:16:45 -0700 (PDT) Received: from [128.224.163.132] (128.224.163.132) by ALA-HCA.corp.ad.wrs.com (147.11.189.50) with Microsoft SMTP Server (TLS) id 14.3.235.1; Mon, 13 Jul 2015 18:16:45 -0700 Message-ID: <55A46390.2090904@windriver.com> Date: Tue, 14 Jul 2015 09:19:12 +0800 From: zhangxiao User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130330 Thunderbird/17.0.5 MIME-Version: 1.0 To: Christopher Larson References: <1436767810-20932-1-git-send-email-xiao.zhang@windriver.com> In-Reply-To: X-Originating-IP: [128.224.163.132] Cc: Patches and discussions about the oe-core layer Subject: Re: [PATCH] logrotate: move binary logrotate back to /usr/sbin 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: Tue, 14 Jul 2015 01:16:49 -0000 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit On 2015年07月13日 22:28, Christopher Larson wrote: > > On Sun, Jul 12, 2015 at 11:10 PM, Zhang Xiao > wrote: > > Upstream-Status: pending > > Signed-off-by: Zhang Xiao > > > > There’s no info on why this is being done, only what’s being done, in > this commit message. Binary logrotate should be installed in /usr/sbin and int fact, at the very beginning, it was there. While with commit a46d3646a3e1781be44: ... logrotate: obey our flags Needed to quiet GNU_HASH warnings, and some minor fixes. ... Part of its modification as below: ... - oe_runmake install DESTDIR=${D} PREFIX=${D} MANDIR=${mandir} + oe_runmake install DESTDIR=${D} PREFIX=${D} MANDIR=${mandir} BINDIR=${bindir} ... moves it to /usr/bin. It is not correct, so move it back to /usr/sbin with ${sbindir}. Xiao > -- > Christopher Larson > clarson at kergoth dot com > Founder - BitBake, OpenEmbedded, OpenZaurus > Maintainer - Tslib > Senior Software Engineer, Mentor Graphics