All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] rpm: avoid dependency on perl and python for -native build
Date: Thu, 02 Jun 2011 22:25:46 +0100	[thread overview]
Message-ID: <1307049946.27470.589.camel@rex> (raw)
In-Reply-To: <1307008107.2529.175.camel@phil-desktop>

On Thu, 2011-06-02 at 10:48 +0100, Phil Blundell wrote:
> Update override naming (_native -> _virtclass-native) to disable perl
> and python bindings when building native rpm, and adjust the DEPENDS to
> match.
> 
> Perl bindings were, in fact, already disabled for both native and target
> builds so it's only the python ones that have really changed.
> 
> Signed-off-by: Phil Blundell <philb@gnu.org>
> ---
>  meta/recipes-devtools/rpm/rpm_5.4.0.bb |   10 ++++++----
>  1 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/recipes-devtools/rpm/rpm_5.4.0.bb b/meta/recipes-devtools/rpm/rpm_5.4.0.bb
> index d3f85f2..5c30f1c 100644
> --- a/meta/recipes-devtools/rpm/rpm_5.4.0.bb
> +++ b/meta/recipes-devtools/rpm/rpm_5.4.0.bb
> @@ -42,8 +42,10 @@ HOMEPAGE = "http://rpm5.org/"
>  LICENSE = "LGPL 2.1"
>  LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=2d5025d4aa3495befef8f17206a5b0a1"
>  
> -DEPENDS = "bzip2 zlib python perl db openssl elfutils expat libpcre attr acl popt"
> -PR = "r16"
> +rpmdeps = "bzip2 zlib db openssl elfutils expat libpcre attr acl popt"
> +DEPENDS = "${rpmdeps} python perl"
> +DEPENDS_virtclass-native = "${rpmdeps}"
> +PR = "r17"
>  
>  # rpm2cpio is a shell script, which is part of the rpm src.rpm.  It is needed
>  # in order to extract the distribution SRPM into a format we can extract...
> @@ -92,7 +94,7 @@ WITH_PYTHON = "	--with-python=${PYTHON_BASEVERSION} \
>  		--without-pythonembed \
>  	      "
>  
> -WITH_PYTHON_native = " --without-python"
> +WITH_PYTHON_virtclass-native = " --without-python"
>  
>  # Perl modules are not built, but they could be enabled fairly easily
>  # the perl module creation and installation would need to be patched.
> @@ -100,7 +102,7 @@ WITH_PYTHON_native = " --without-python"
>  #WITH_PERL = "	--with-perl --without-perlembed"
>  WITH_PERL = "	--without-perl"
>  
> -WITH_PERL_native = " --without-perl"
> +WITH_PERL_virtclass-native = " --without-perl"
>  
>  WITH_DB = "--with-db --with-dbsql --without-db-tools-integrated --without-sqlite"
>  

This appears to break the rpm-native build with:

| checking whether to build with Lua library... no
| ++ executing failure action
| ++ mapping --with-pcre=yes to --with-pcre="external:internal:none"
| ++ searching location: external
| ++ searching location: internal
| -- skipping not existing local sub-directory: pcre
| ++ searching location: none
| -- no operation
| checking pcre.h usability... no
| checking pcre.h presence... no
| checking for pcre.h... no
| checking for pcre_compile in -lpcre... no
| checking whether to build with PCRE library... no
| ++ executing failure action
| configure: error: PCRE is mandatory. If you don't have  libpcre then --with-pcre=internal could be used
| ERROR: oe_runconf failed
| ERROR: Function 'do_configure' failed (see /media/build2/builds/rptest/b2/tmp/work/x86_64-linux/rpm-native-5.4.0-r17/temp/log.do_configure.8572 for further information)
| ERROR: Function 'do_configure' failed (see /media/build2/builds/rptest/b2/tmp/work/x86_64-linux/rpm-native-5.4.0-r17/temp/log.do_configure.8572 for further information)
NOTE: package rpm-native-5.4.0-r17: task do_configure: Failed
ERROR: Task 48 (virtual:native:/media/build2/builds/rptest/meta/recipes-devtools/rpm/rpm_5.4.0.bb, do_configure) failed with exit code '1'
ERROR: 'virtual:native:/media/build2/builds/rptest/meta/recipes-devtools/rpm/rpm_5.4.0.bb' failed

:(

Cheers,

Richard





  parent reply	other threads:[~2011-06-02 21:29 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-02  9:48 [PATCH] rpm: avoid dependency on perl and python for -native build Phil Blundell
2011-06-02 13:29 ` Mark Hatle
2011-06-02 13:52 ` Richard Purdie
2011-06-02 21:25 ` Richard Purdie [this message]
2011-06-02 21:37   ` Phil Blundell
2011-06-02 22:26     ` Richard Purdie
2011-06-03 10:32       ` Phil Blundell

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=1307049946.27470.589.camel@rex \
    --to=richard.purdie@linuxfoundation.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.