From: Martin Jansa <martin.jansa@gmail.com>
To: Alexander Kanavin <alexander.kanavin@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 11/26] nss: reorder SRC_URI lines to fix upstream version check
Date: Thu, 30 Jul 2015 16:33:53 +0200 [thread overview]
Message-ID: <20150730143353.GG2457@jama> (raw)
In-Reply-To: <85e7e6014242705a3e1e06ec5862358e95641a3a.1438262915.git.alexander.kanavin@linux.intel.com>
[-- Attachment #1: Type: text/plain, Size: 2103 bytes --]
On Thu, Jul 30, 2015 at 04:31:40PM +0300, Alexander Kanavin wrote:
> The source tarball should be listed first
>
> Signed-off-by: Alexander Kanavin <alexander.kanavin@linux.intel.com>
> ---
> meta/recipes-support/nss/nss.inc | 2 +-
> meta/recipes-support/nss/nss_3.17.3.bb | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-support/nss/nss.inc b/meta/recipes-support/nss/nss.inc
> index 665e4e6..b21dc39 100644
> --- a/meta/recipes-support/nss/nss.inc
> +++ b/meta/recipes-support/nss/nss.inc
> @@ -12,7 +12,7 @@ LICENSE = "MPL-2.0 | (MPL-2.0 & GPL-2.0+) | (MPL-2.0 & LGPL-2.1+)"
>
> LIC_FILES_CHKSUM = "file://nss/lib/freebl/mpi/doc/LICENSE;md5=491f158d09d948466afce85d6f1fe18f \
> file://nss/lib/freebl/mpi/doc/LICENSE-MPL;md5=6bf96825e3d7ce4de25621ae886cc859"
> -SRC_URI = "\
> +SRC_URI += "\
> file://nss-fix-support-cross-compiling.patch \
> file://nss-no-rpath-for-cross-compiling.patch \
> file://nss-fix-incorrect-shebang-of-perl.patch \
> diff --git a/meta/recipes-support/nss/nss_3.17.3.bb b/meta/recipes-support/nss/nss_3.17.3.bb
> index e7d0780..b22eb95 100644
> --- a/meta/recipes-support/nss/nss_3.17.3.bb
> +++ b/meta/recipes-support/nss/nss_3.17.3.bb
> @@ -1,7 +1,7 @@
> -require nss.inc
> -
> -SRC_URI += "\
> +SRC_URI = "\
> http://ftp.mozilla.org/pub/mozilla.org/security/nss/releases/NSS_3_17_3_RTM/src/${BP}.tar.gz \
> "
Why don't you move this to .inc file and replace "3_17_3_RTM" with some
variable set in .bb, it would be easier to read than this reorder and
require .inc later..
> SRC_URI[md5sum] = "fba7489e1b26f2a0bfe5527430fd61e1"
> SRC_URI[sha256sum] = "f4d5e9035a2f84f25f35c283de3b0ff60d72e918748de25eaf017ed201fa21d5"
> +
> +require nss.inc
> --
> 2.1.4
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]
next prev parent reply other threads:[~2015-07-30 14:33 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-30 13:31 [PATCH 00/26] Upstream version check fixes for various recipes Alexander Kanavin
2015-07-30 13:31 ` [PATCH 01/26] libproxy: update SRC_URI to not use Google Code Alexander Kanavin
2015-07-30 14:29 ` Martin Jansa
2015-07-30 14:29 ` Burton, Ross
2015-07-30 15:12 ` Alexander Kanavin
2015-07-30 15:20 ` Burton, Ross
2015-07-30 13:31 ` [PATCH 02/26] fotowall: update SRC_URI to use github instead of qt-apps.org Alexander Kanavin
2015-07-30 14:29 ` Martin Jansa
2015-07-30 15:14 ` Alexander Kanavin
2015-07-30 13:31 ` [PATCH 03/26] distcc: update SRC_URI to not use Google Code Alexander Kanavin
2015-07-30 14:30 ` Martin Jansa
2015-07-30 13:31 ` [PATCH 04/26] adt-installer: " Alexander Kanavin
2015-07-30 13:31 ` [PATCH 05/26] ossp-uuid: " Alexander Kanavin
2015-07-30 13:31 ` [PATCH 06/26] eglinfo: change version from 1.0 to 1.0.0 to match the upstream git tag Alexander Kanavin
2015-07-30 14:31 ` Martin Jansa
2015-07-30 15:02 ` Burton, Ross
2015-07-30 15:18 ` Alexander Kanavin
2015-07-30 13:31 ` [PATCH 07/26] gptfdisk: use tarballs instead of git for fetching source code Alexander Kanavin
2015-07-30 13:31 ` [PATCH 08/26] libmad: use ftp instead of sourceforge for consistency with libid3tag Alexander Kanavin
2015-07-30 13:31 ` [PATCH 09/26] lsb: set PV to 1.4 to match upstream tarball version Alexander Kanavin
2015-07-30 14:32 ` Martin Jansa
2015-07-30 15:22 ` Alexander Kanavin
2015-08-07 7:21 ` Khem Raj
2015-08-07 12:09 ` Alexander Kanavin
2015-08-07 19:22 ` Khem Raj
2015-07-30 13:31 ` [PATCH 10/26] linux-yocto-dev: only numeric characters are allowed in package versions taken from git Alexander Kanavin
2015-07-30 13:31 ` [PATCH 11/26] nss: reorder SRC_URI lines to fix upstream version check Alexander Kanavin
2015-07-30 14:33 ` Martin Jansa [this message]
2015-07-30 15:04 ` Burton, Ross
2015-07-30 16:04 ` Alexander Kanavin
2015-07-30 13:31 ` [PATCH 12/26] pseudo: fix upstream SRC_URI to enable " Alexander Kanavin
2015-07-30 13:31 ` [PATCH 13/26] oprofile: reorder SRC_URI lines to fix " Alexander Kanavin
2015-07-30 14:34 ` Martin Jansa
2015-07-30 15:41 ` Alexander Kanavin
2015-07-30 13:31 ` [PATCH 14/26] puzzles: switch svn for git, as upstream has done the same Alexander Kanavin
2015-07-30 14:35 ` Martin Jansa
2015-07-30 16:06 ` Alexander Kanavin
2015-08-07 7:19 ` Khem Raj
2015-08-07 12:08 ` Alexander Kanavin
2015-07-30 13:31 ` [PATCH 15/26] syslinux: drop v6.xx/ from directory path in SRC_URI Alexander Kanavin
2015-07-30 13:31 ` [PATCH 16/26] shared-mime-info: reorder SRC_URI lines to fix upstream version check Alexander Kanavin
2015-07-30 13:31 ` [PATCH 17/26] tremor: switch SRC_URI from svn to tarballs on debian mirrors Alexander Kanavin
2015-07-30 14:35 ` Martin Jansa
2015-07-30 13:31 ` [PATCH 18/26] x264: only numeric characters are allowed in package versions taken from git Alexander Kanavin
2015-07-30 13:31 ` [PATCH 19/26] xf86-video-omapfb: switch SRC_URI to debian git repo Alexander Kanavin
2015-07-30 21:03 ` Burton, Ross
2015-07-30 13:31 ` [PATCH 20/26] stress: use developer's website for SRC_URI Alexander Kanavin
2015-07-30 17:45 ` Max Krummenacher
2015-07-31 12:17 ` Alexander Kanavin
2015-07-30 13:31 ` [PATCH 21/26] systemd: remove PV from the recipe Alexander Kanavin
2015-07-30 14:36 ` Martin Jansa
2015-07-30 15:32 ` Alexander Kanavin
2015-07-30 16:25 ` Burton, Ross
2015-07-30 16:35 ` Alexander Kanavin
2015-07-31 12:40 ` Pascal Bach
2015-07-31 12:53 ` Alexander Kanavin
2015-07-31 13:03 ` Pascal Bach
2015-07-31 13:03 ` Alexander Kanavin
2015-07-31 12:24 ` Alexander Kanavin
2015-07-30 13:31 ` [PATCH 22/26] unzip, zip: fix PV to allow upstream version check Alexander Kanavin
2015-07-30 13:31 ` [PATCH 23/26] guilt-native: set PV in the recipe " Alexander Kanavin
2015-07-30 13:31 ` [PATCH 24/26] distrodata.bbclass: do not skip -native recipes in checkpkg task Alexander Kanavin
2015-07-30 13:31 ` [PATCH 25/26] distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before comparing versions in checkpkg Alexander Kanavin
2015-07-31 14:54 ` Aníbal Limón
2015-07-31 15:09 ` Alexander Kanavin
2015-07-31 15:31 ` Aníbal Limón
2015-07-31 15:34 ` Aníbal Limón
2015-07-31 19:45 ` alexander.kanavin
2015-07-30 13:31 ` [PATCH 26/26] distrodata.bbclass: drop the manual upstream version check logic Alexander Kanavin
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=20150730143353.GG2457@jama \
--to=martin.jansa@gmail.com \
--cc=alexander.kanavin@linux.intel.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