* [PATCH 0/2] package_rpm and createrepo fixes
@ 2012-07-20 9:49 Robert Yang
2012-07-20 9:49 ` [PATCH 1/2] package_rpm: DEPLOY_DIR_RPM must exist before make index Robert Yang
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Robert Yang @ 2012-07-20 9:49 UTC (permalink / raw)
To: openembedded-core; +Cc: Zhenfeng.Zhao
Hi Saul,
For the error:
dbconfig.c:493: db3New: Assertion `dbOpts != ((void *)0) && *dbOpts != '\0'' failed.
The dbOpts is the value of _dbi_config_ or _dbi_config which are defined
in tmp/sysroots/i686-linux/usr/lib/rpm/macros, I can't reproduce it in
a normal build (on Ubuntu 10.10, Fedora 17 64bit, Ubuntu 12.04 64bit),
the error would happen when manually remove i686-linux/usr/lib/rpm/macros,
(but it should be there since the librpmdb-5.4.so is ready in a normal
build).
These two patches are used for fixing other issues of createrepo.
// Robert
The following changes since commit 64f7feb5c37d06e1985c59106cf2f7cd286fd0b8:
EFI: Make installer EFI aware (2012-07-19 17:49:48 +0100)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib robert/createrepo
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/createrepo
Robert Yang (2):
package_rpm: DEPLOY_DIR_RPM must exist before make index
createrepo 0.4.11: fix the RDEPENDS
meta/classes/package_rpm.bbclass | 4 ++--
.../createrepo/createrepo_0.4.11.bb | 4 +++-
2 files changed, 5 insertions(+), 3 deletions(-)
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] package_rpm: DEPLOY_DIR_RPM must exist before make index
2012-07-20 9:49 [PATCH 0/2] package_rpm and createrepo fixes Robert Yang
@ 2012-07-20 9:49 ` Robert Yang
2012-07-20 9:49 ` [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS Robert Yang
2012-07-21 0:19 ` [PATCH 0/2] package_rpm and createrepo fixes Saul Wold
2 siblings, 0 replies; 7+ messages in thread
From: Robert Yang @ 2012-07-20 9:49 UTC (permalink / raw)
To: openembedded-core; +Cc: Zhenfeng.Zhao
There is a package-index.bb which will update the index, but the
DEPLOY_DIR_RPM may not exist, for example, when "bitbake package-index"
in a fresh build.
Only the package_rpm.bbclass needs this, the package_ipk.bbclass and
package_deb.bbclass doesn't need.
[YOCTO #2753]
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
meta/classes/package_rpm.bbclass | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/classes/package_rpm.bbclass b/meta/classes/package_rpm.bbclass
index d213e40..983be4c 100644
--- a/meta/classes/package_rpm.bbclass
+++ b/meta/classes/package_rpm.bbclass
@@ -22,7 +22,7 @@ RPMCONF_HOST_BASE = "${DEPLOY_DIR_RPM}/solvedb-sdk"
# Update the Packages depsolver db in ${DEPLOY_DIR_RPM}
#
package_update_index_rpm () {
- if [ ! -z "${DEPLOY_KEEP_PACKAGES}" ]; then
+ if [ ! -z "${DEPLOY_KEEP_PACKAGES}" -o ! -e "${DEPLOY_DIR_RPM}" ]; then
return
fi
@@ -1171,6 +1171,6 @@ do_package_write_rpm[dirs] = "${PKGWRITEDIRRPM}"
do_package_write_rpm[umask] = "022"
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; [ ! -e ${DEPLOY_DIR_RPM} ] || createrepo ${DEPLOY_DIR_RPM};"
PACKAGEINDEXDEPS += "rpm-native:do_populate_sysroot"
PACKAGEINDEXDEPS += "createrepo-native:do_populate_sysroot"
--
1.7.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS
2012-07-20 9:49 [PATCH 0/2] package_rpm and createrepo fixes Robert Yang
2012-07-20 9:49 ` [PATCH 1/2] package_rpm: DEPLOY_DIR_RPM must exist before make index Robert Yang
@ 2012-07-20 9:49 ` Robert Yang
2012-07-21 0:18 ` Saul Wold
2012-07-21 0:19 ` [PATCH 0/2] package_rpm and createrepo fixes Saul Wold
2 siblings, 1 reply; 7+ messages in thread
From: Robert Yang @ 2012-07-20 9:49 UTC (permalink / raw)
To: openembedded-core; +Cc: Zhenfeng.Zhao
The libxml2-native and rpm-native should be ready before createrepo
runs since it needs "import libxml2 rpm".
The target createrepo needs libxml2-python when run, but the libxml2
doesn't generate the python module (disabled). We may need to fix it
or rename the recipe to createrepo-native_0.4.11.bb.
This recipe is only for the rpm backend system.
[YOCTO #2753]
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
.../createrepo/createrepo_0.4.11.bb | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/meta/recipes-support/createrepo/createrepo_0.4.11.bb b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
index 060ed59..7a4d059 100644
--- a/meta/recipes-support/createrepo/createrepo_0.4.11.bb
+++ b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
@@ -4,7 +4,9 @@ HOMEPAGE = "http://createrepo.baseurl.org/"
LICENSE = "GPLv2+"
LIC_FILES_CHKSUM = "file://COPYING;md5=18810669f13b87348459e611d31ab760"
-PR = "r3"
+RDEPENDS_${PN}_virtclass-native += "libxml2-native rpm-native"
+
+PR = "r4"
SRC_URI= "http://createrepo.baseurl.org/download/${BP}.tar.gz \
file://fix-native-install.patch \
--
1.7.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS
2012-07-20 9:49 ` [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS Robert Yang
@ 2012-07-21 0:18 ` Saul Wold
2012-07-22 10:37 ` Richard Purdie
2012-07-23 2:21 ` Robert Yang
0 siblings, 2 replies; 7+ messages in thread
From: Saul Wold @ 2012-07-21 0:18 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Zhenfeng.Zhao
On 07/20/2012 02:49 AM, Robert Yang wrote:
> The libxml2-native and rpm-native should be ready before createrepo
> runs since it needs "import libxml2 rpm".
>
> The target createrepo needs libxml2-python when run, but the libxml2
> doesn't generate the python module (disabled). We may need to fix it
> or rename the recipe to createrepo-native_0.4.11.bb.
>
> This recipe is only for the rpm backend system.
>
> [YOCTO #2753]
>
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> ---
> .../createrepo/createrepo_0.4.11.bb | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/meta/recipes-support/createrepo/createrepo_0.4.11.bb b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
> index 060ed59..7a4d059 100644
> --- a/meta/recipes-support/createrepo/createrepo_0.4.11.bb
> +++ b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
> @@ -4,7 +4,9 @@ HOMEPAGE = "http://createrepo.baseurl.org/"
> LICENSE = "GPLv2+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=18810669f13b87348459e611d31ab760"
>
> -PR = "r3"
> +RDEPENDS_${PN}_virtclass-native += "libxml2-native rpm-native"
> +
This should not be required here since the package_rpm.bbclass contains
a line:
PACKAGEINDEXDEPS += "rpm-native:do_populate_sysroot"
We might need to add libxml2-native to the PACKAGEINDEXDEPS
This was merged into OE-0Core, but the real issue turned out to be
sstate across different distros.
Sau!
> +PR = "r4"
>
> SRC_URI= "http://createrepo.baseurl.org/download/${BP}.tar.gz \
> file://fix-native-install.patch \
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/2] package_rpm and createrepo fixes
2012-07-20 9:49 [PATCH 0/2] package_rpm and createrepo fixes Robert Yang
2012-07-20 9:49 ` [PATCH 1/2] package_rpm: DEPLOY_DIR_RPM must exist before make index Robert Yang
2012-07-20 9:49 ` [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS Robert Yang
@ 2012-07-21 0:19 ` Saul Wold
2 siblings, 0 replies; 7+ messages in thread
From: Saul Wold @ 2012-07-21 0:19 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Zhenfeng.Zhao
On 07/20/2012 02:49 AM, Robert Yang wrote:
> Hi Saul,
>
> For the error:
>
> dbconfig.c:493: db3New: Assertion `dbOpts != ((void *)0) && *dbOpts != '\0'' failed.
>
> The dbOpts is the value of _dbi_config_ or _dbi_config which are defined
> in tmp/sysroots/i686-linux/usr/lib/rpm/macros, I can't reproduce it in
> a normal build (on Ubuntu 10.10, Fedora 17 64bit, Ubuntu 12.04 64bit),
> the error would happen when manually remove i686-linux/usr/lib/rpm/macros,
> (but it should be there since the librpmdb-5.4.so is ready in a normal
> build).
>
> These two patches are used for fixing other issues of createrepo.
>
Turns out the real problem is that one machine was running Suse and the
other Fedora and mixing the binaries via nfs and sstate was causing the
real problem.
I made a note about the DEPENDS for createrepo, really should be part of
package_rpm.bbclass
> // Robert
>
> The following changes since commit 64f7feb5c37d06e1985c59106cf2f7cd286fd0b8:
>
> EFI: Make installer EFI aware (2012-07-19 17:49:48 +0100)
>
> are available in the git repository at:
> git://git.pokylinux.org/poky-contrib robert/createrepo
> http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/createrepo
>
> Robert Yang (2):
> package_rpm: DEPLOY_DIR_RPM must exist before make index
> createrepo 0.4.11: fix the RDEPENDS
>
> meta/classes/package_rpm.bbclass | 4 ++--
> .../createrepo/createrepo_0.4.11.bb | 4 +++-
> 2 files changed, 5 insertions(+), 3 deletions(-)
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>
Merged into OE-Core
Thanks
Sau!
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS
2012-07-21 0:18 ` Saul Wold
@ 2012-07-22 10:37 ` Richard Purdie
2012-07-23 2:21 ` Robert Yang
1 sibling, 0 replies; 7+ messages in thread
From: Richard Purdie @ 2012-07-22 10:37 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Zhenfeng.Zhao
On Fri, 2012-07-20 at 17:18 -0700, Saul Wold wrote:
> On 07/20/2012 02:49 AM, Robert Yang wrote:
> > The libxml2-native and rpm-native should be ready before createrepo
> > runs since it needs "import libxml2 rpm".
> >
> > The target createrepo needs libxml2-python when run, but the libxml2
> > doesn't generate the python module (disabled). We may need to fix it
> > or rename the recipe to createrepo-native_0.4.11.bb.
> >
> > This recipe is only for the rpm backend system.
> >
> > [YOCTO #2753]
> >
> > Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> > ---
> > .../createrepo/createrepo_0.4.11.bb | 4 +++-
> > 1 files changed, 3 insertions(+), 1 deletions(-)
> >
> > diff --git a/meta/recipes-support/createrepo/createrepo_0.4.11.bb b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
> > index 060ed59..7a4d059 100644
> > --- a/meta/recipes-support/createrepo/createrepo_0.4.11.bb
> > +++ b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
> > @@ -4,7 +4,9 @@ HOMEPAGE = "http://createrepo.baseurl.org/"
> > LICENSE = "GPLv2+"
> > LIC_FILES_CHKSUM = "file://COPYING;md5=18810669f13b87348459e611d31ab760"
> >
> > -PR = "r3"
> > +RDEPENDS_${PN}_virtclass-native += "libxml2-native rpm-native"
> > +
> This should not be required here since the package_rpm.bbclass contains
> a line:
> PACKAGEINDEXDEPS += "rpm-native:do_populate_sysroot"
>
> We might need to add libxml2-native to the PACKAGEINDEXDEPS
FWIW, having correct RDEPENDS in the recipes is important so this fix is
a good one regardless of whether it fixes the other issue or not.
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS
2012-07-21 0:18 ` Saul Wold
2012-07-22 10:37 ` Richard Purdie
@ 2012-07-23 2:21 ` Robert Yang
1 sibling, 0 replies; 7+ messages in thread
From: Robert Yang @ 2012-07-23 2:21 UTC (permalink / raw)
Cc: Zhenfeng.Zhao, Patches and discussions about the oe-core layer
On 07/21/2012 08:18 AM, Saul Wold wrote:
> On 07/20/2012 02:49 AM, Robert Yang wrote:
>> The libxml2-native and rpm-native should be ready before createrepo
>> runs since it needs "import libxml2 rpm".
>>
>> The target createrepo needs libxml2-python when run, but the libxml2
>> doesn't generate the python module (disabled). We may need to fix it
>> or rename the recipe to createrepo-native_0.4.11.bb.
>>
>> This recipe is only for the rpm backend system.
>>
>> [YOCTO #2753]
>>
>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>> ---
>> .../createrepo/createrepo_0.4.11.bb | 4 +++-
>> 1 files changed, 3 insertions(+), 1 deletions(-)
>>
>> diff --git a/meta/recipes-support/createrepo/createrepo_0.4.11.bb
>> b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
>> index 060ed59..7a4d059 100644
>> --- a/meta/recipes-support/createrepo/createrepo_0.4.11.bb
>> +++ b/meta/recipes-support/createrepo/createrepo_0.4.11.bb
>> @@ -4,7 +4,9 @@ HOMEPAGE = "http://createrepo.baseurl.org/"
>> LICENSE = "GPLv2+"
>> LIC_FILES_CHKSUM = "file://COPYING;md5=18810669f13b87348459e611d31ab760"
>>
>> -PR = "r3"
>> +RDEPENDS_${PN}_virtclass-native += "libxml2-native rpm-native"
>> +
> This should not be required here since the package_rpm.bbclass contains a line:
> PACKAGEINDEXDEPS += "rpm-native:do_populate_sysroot"
>
> We might need to add libxml2-native to the PACKAGEINDEXDEPS
>
Thanks, but we may manually run createrepo, if we just "bitbake createrepo" and
manually run createrepo, it would be failed if rpm-native is not ready.
> This was merged into OE-0Core, but the real issue turned out to be sstate across
> different distros.
>
I had tried to use sstate across different distros (ubuntu 12.04 64bit and FC 17
64 bit),
there were strange errors.
// Robert
> Sau!
>
>
>
>> +PR = "r4"
>>
>> SRC_URI= "http://createrepo.baseurl.org/download/${BP}.tar.gz \
>> file://fix-native-install.patch \
>>
>
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-07-23 2:33 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-20 9:49 [PATCH 0/2] package_rpm and createrepo fixes Robert Yang
2012-07-20 9:49 ` [PATCH 1/2] package_rpm: DEPLOY_DIR_RPM must exist before make index Robert Yang
2012-07-20 9:49 ` [PATCH 2/2] createrepo 0.4.11: fix the RDEPENDS Robert Yang
2012-07-21 0:18 ` Saul Wold
2012-07-22 10:37 ` Richard Purdie
2012-07-23 2:21 ` Robert Yang
2012-07-21 0:19 ` [PATCH 0/2] package_rpm and createrepo fixes Saul Wold
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.