[OE-core] [PATCH 4/9] package_rpm.bbclass: create method for createrepo cmd
Richard Purdie
richard.purdie at linuxfoundation.org
Mon May 23 14:39:07 UTC 2011
A couple of comments:
Firstly the commit message says what you changed, not why...
On Sun, 2011-05-22 at 11:48 -0700, Saul Wold wrote:
> Signed-off-by: Saul Wold <sgw at linux.intel.com>
> ---
> meta/classes/package_rpm.bbclass | 6 +++++-
> 1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/meta/classes/package_rpm.bbclass b/meta/classes/package_rpm.bbclass
> index 1d8c686..666eb54 100644
> --- a/meta/classes/package_rpm.bbclass
> +++ b/meta/classes/package_rpm.bbclass
> @@ -15,6 +15,10 @@ python package_rpm_install () {
> bb.fatal("package_rpm_install not implemented!")
> }
>
> +package_createrepo () {
> + createrepo ${DEPLOY_DIR_RPM}
> +}
> +
> RPMCONF_TARGET_BASE = "${DEPLOY_DIR_RPM}/solvedb"
> RPMCONF_HOST_BASE = "${DEPLOY_DIR_RPM}/solvedb-sdk"
> #
> @@ -816,6 +820,6 @@ python do_package_write_rpm () {
> do_package_write_rpm[dirs] = "${PKGWRITEDIRRPM}"
> addtask package_write_rpm before do_package_write after do_package
>
> -PACKAGEINDEXES += "package_update_index_rpm; createrepo ${DEPLOY_DIR_RPM};"
> +PACKAGEINDEXES += "package_update_index_rpm; package_createrepo"
Secondly this should end in a ;. I'd just add the
package_update_index_rpm call to package_createrepo to be honest.
package_createrepo could also do with rpm in the name somewhere just to
we're reminded its rpm specific in other contexts.
Cheers,
Richard
More information about the Openembedded-core
mailing list