From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.chez-thomas.org (mail.mlbassoc.com [65.100.170.105]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 8E1E8E007F8 for ; Wed, 15 Jan 2014 15:42:22 -0800 (PST) Received: by mail.chez-thomas.org (Postfix, from userid 1998) id 34377F811DA; Wed, 15 Jan 2014 16:42:21 -0700 (MST) X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on hermes.chez-thomas.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=4.0 tests=ALL_TRUSTED,BAYES_00 autolearn=unavailable version=3.3.2 Received: from [192.168.1.114] (zeus [192.168.1.114]) by mail.chez-thomas.org (Postfix) with ESMTP id 446D9F811DA; Wed, 15 Jan 2014 16:42:19 -0700 (MST) Message-ID: <52D71CDF.602@mlbassoc.com> Date: Wed, 15 Jan 2014 16:42:23 -0700 From: Gary Thomas User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: Otavio Salvador References: <1389801493-17578-1-git-send-email-gary@mlbassoc.com> <52D6FC06.9010100@mlbassoc.com> In-Reply-To: Cc: "meta-freescale@yoctoproject.org" Subject: Re: [meta-fsl-arm] [PATCH] imx-kobs: Update license checksum X-BeenThere: meta-freescale@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-fsl-* layers List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 Jan 2014 23:42:26 -0000 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit On 2014-01-15 16:26, Otavio Salvador wrote: > On Wed, Jan 15, 2014 at 7:22 PM, Gary Thomas > wrote: > > On 2014-01-15 12:47, Otavio Salvador wrote: > > On Wed, Jan 15, 2014 at 1:58 PM, Gary Thomas >> wrote: > > Update license checksum to follow changes in OE-core > > Signed-off-by: Gary Thomas >> > > > > To avoid using it to happen in future, please rework this as I did in: > > http://git.yoctoproject.org/__cgit/cgit.cgi/meta-fsl-arm/__commit/?id=__a99e70cbb501f992abb8f80218fc5f__b2a4fd6224 > > > > Not my place to make such a change - those are completely different > license files. One refers to exactly GPL-2 while the other says that > the licence may be any of a set. > > > The license, in the LICENSE field, is not changed and this checksum was just to allow the recipe to be parsed. It makes more sense to use the respective license file. The best fix > is, in future, Freescale, include a proper license file on the release tarball of those packages but in meanwhile is the a good interim solution. I don't see this. It looks to me like you are changing the license from file://${COREBASE}/LICENSE to a completely different file file://${COREBASE}/meta/files/common-licenses/GPL-2.0 At least that's what's in the change you quoted (above). I'm only comfortable changing the checksum - not the file being used. > > When we have multiple, we can use the checksums for the specific files as well. > > I only proposed the direct change which matches the changed checksum. > You'll have to make any actual license file changes. > > > I'd like to ask you to just do as I said above. I can ask Lauren to review those patches before commiting those so we are all comfortable with the change. > > What do you think? > > -- > Otavio Salvador O.S. Systems > http://www.ossystems.com.br http://code.ossystems.com.br > Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750 -- ------------------------------------------------------------ Gary Thomas | Consulting for the MLB Associates | Embedded world ------------------------------------------------------------