From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 08/30] package-index.bb: add support for deb and rpm.
Date: Thu, 19 May 2011 23:33:04 +0100 [thread overview]
Message-ID: <1305844384.3424.566.camel@rex> (raw)
In-Reply-To: <8305336c6b1eb0789af195f28230ecfca178538a.1305756950.git.sgw@linux.intel.com>
Hi,
We're nearly there with this, I'd like just one other tweak please:
On Thu, 2011-05-19 at 12:19 -0700, Saul Wold wrote:
> diff --git a/meta/classes/package_deb.bbclass b/meta/classes/package_deb.bbclass
> index 4faeb4a..f9ef32c 100644
> --- a/meta/classes/package_deb.bbclass
> +++ b/meta/classes/package_deb.bbclass
> @@ -431,3 +431,12 @@ python do_package_write_deb () {
> do_package_write_deb[dirs] = "${PKGWRITEDIRDEB}"
> addtask package_write_deb before do_package_write after do_package
>
> +
> +PACKAGEINDEXES += "package_index_deb;"
> +PACKAGEINDEXDEPS += "dpkg-native:do_populate_sysroot"
> +PACKAGEINDEXDEPS += "apt-native:do_populate_sysroot"
> +
> +package_index_deb() {
> + package_update_index_deb
> +}
Can't we just do +PACKAGEINDEXES += "package_update_index_deb;"
instead of adding the function?
> diff --git a/meta/classes/package_ipk.bbclass b/meta/classes/package_ipk.bbclass
> index 3c2472b..97de17a 100644
> --- a/meta/classes/package_ipk.bbclass
> +++ b/meta/classes/package_ipk.bbclass
> @@ -386,3 +386,12 @@ python do_package_write_ipk () {
> }
> do_package_write_ipk[dirs] = "${PKGWRITEDIRIPK}"
> addtask package_write_ipk before do_package_write after do_package
> +
> +PACKAGEINDEXES += "package_index_ipk;"
> +PACKAGEINDEXDEPS += "opkg-utils-native:do_populate_sysroot"
> +PACKAGEINDEXDEPS += "opkg-native:do_populate_sysroot"
> +
> +package_index_ipk() {
> + package_update_index_ipk
> +}
ditto?
Cheers,
Richard
next prev parent reply other threads:[~2011-05-19 22:36 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-18 22:26 [PATCH 00/30] 18-May-2011 Consolidated Pull Saul Wold
2011-05-18 22:26 ` [PATCH 01/30] diffutils: fix libc overrides Saul Wold
2011-05-18 22:26 ` [PATCH 02/30] findutils: " Saul Wold
2011-05-18 22:26 ` [PATCH 03/30] lsof: " Saul Wold
2011-05-18 22:26 ` [PATCH 04/30] sysstat: " Saul Wold
2011-05-18 22:26 ` [PATCH 05/30] tcp-wrappers: " Saul Wold
2011-05-18 22:26 ` [PATCH 06/30] linux-yocto: fix gcc 4.6.0 compilation failures Saul Wold
2011-05-18 22:26 ` [PATCH 07/30] lsbsetup: Rename this file and add a link for lsb test Saul Wold
2011-05-18 22:26 ` [PATCH 08/30] package-index.bb: add support for deb and rpm Saul Wold
2011-05-19 22:33 ` Richard Purdie [this message]
2011-05-20 1:13 ` Cui, Dexuan
2011-05-18 22:26 ` [PATCH 09/30] extract bitbake config setting into data_define, use more variables replacing hardcoded values Saul Wold
2011-05-18 22:26 ` [PATCH 10/30] Add adt repo checking Saul Wold
2011-05-18 22:26 ` [PATCH 11/30] utils.bbclass: modify create_wrapper to correctly follow symlinks Saul Wold
2011-05-19 19:26 ` Koen Kooi
2011-05-18 22:26 ` [PATCH 12/30] mklibs-native: Update to 0.1.33 Saul Wold
2011-05-18 22:26 ` [PATCH 13/30] usbutils: Update to 0.91 Saul Wold
2011-05-18 22:26 ` [PATCH 14/30] qt4-tools-native: Fix the patch location Saul Wold
2011-05-18 22:26 ` [PATCH 15/30] create-pull-request: alphabetize arguments Saul Wold
2011-05-18 22:26 ` [PATCH 16/30] create-pull-request: whitespace cleanup Saul Wold
2011-05-18 22:26 ` [PATCH 17/30] create-pull-request: use git request-pull and arbitrary remotes Saul Wold
2011-05-18 22:26 ` [PATCH 18/30] create-pull-request: rewrite known private URLs to public URLs Saul Wold
2011-05-18 22:26 ` [PATCH 19/30] create-pull-request: provide an RFC mode via -c argument Saul Wold
2011-05-18 22:26 ` [PATCH 20/30] send-pull-request: whitespace cleanup Saul Wold
2011-05-18 22:26 ` [PATCH 21/30] send-pull-request: remove local mta support Saul Wold
2011-05-18 22:26 ` [PATCH 22/30] send-pull-request: fix greedy auto-cc regex Saul Wold
2011-05-18 22:26 ` [PATCH 23/30] send-pull-request: don't send all patches to everyone even with -a Saul Wold
2011-05-18 22:26 ` [PATCH 24/30] send-pull-request: verify git sendemail config Saul Wold
2011-05-18 22:26 ` [PATCH 25/30] create-pull-request: do not check certificate Saul Wold
2011-05-18 22:26 ` [PATCH 26/30] create-pull-request: add GitHub remote support Saul Wold
2011-05-18 22:26 ` [PATCH 27/30] create-pull-request: add untested oe repository support Saul Wold
2011-05-18 22:26 ` [PATCH 28/30] send-pull-request: streamline git-send-email usage Saul Wold
2011-05-18 22:26 ` [PATCH 29/30] *pull-request: add copyright, license, and descriptions Saul Wold
2011-05-18 22:26 ` [PATCH 30/30] u-boot: update SRCREV to 2011.03 Saul Wold
2011-05-19 22:42 ` [PATCH 00/30] 18-May-2011 Consolidated Pull Richard Purdie
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=1305844384.3424.566.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox