All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] rpm_5.4.9.bb: fix builds on systems with older libc
@ 2012-07-10 17:14 Matthew McClintock
  2012-07-17 15:57 ` Saul Wold
  2012-07-19 15:40 ` Saul Wold
  0 siblings, 2 replies; 4+ messages in thread
From: Matthew McClintock @ 2012-07-10 17:14 UTC (permalink / raw)
  To: openembedded-core

| dbconvert.o: In function `rpmdb_convert':
| /local/jenkins/jobs/yocto-upstream/workspace/label/master/machine/p4080ds/poky/master/tmp/work/x86_64-linux/rpm-native-5.4.9-r45/rpm-5.4.9/tools/dbconvert.c:126: warning: the use of `tempnam' is dangerous, better use `mkstemp'
| dbconvert.o: In function `main':
| dbconvert.c:(.text+0x923): undefined reference to `htobe32'
| dbconvert.c:(.text+0xaa4): undefined reference to `htole32'
| dbconvert.c:(.text+0xac9): undefined reference to `htole32'

Signed-off-by: Matthew McClintock <msm@freescale.com>
---
 meta/recipes-devtools/rpm/rpm/dbconvert.patch |   27 +++++++++++++++++++++++++
 meta/recipes-devtools/rpm/rpm_5.4.9.bb        |    3 ++-
 2 files changed, 29 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-devtools/rpm/rpm/dbconvert.patch

diff --git a/meta/recipes-devtools/rpm/rpm/dbconvert.patch b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
new file mode 100644
index 0000000..27dba8c
--- /dev/null
+++ b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
@@ -0,0 +1,27 @@
+Upstream-Status: Not applicable
+
+Hack to prevent unneeded demo app from building on older libc
+where it will break the build
+
+Index: rpm-5.4.9/tools/Makefile.am
+===================================================================
+--- rpm-5.4.9.orig/tools/Makefile.am
++++ rpm-5.4.9/tools/Makefile.am
+@@ -24,7 +24,7 @@ AM_CFLAGS = $(OPENMP_CFLAGS)
+ 
+ EXTRA_DIST =	hashtab.h bsdiff.1 bspatch.1 dotgraph.hh
+ 
+-EXTRA_PROGRAMS = augtool cudftool dbconvert debugedit \
++EXTRA_PROGRAMS = augtool cudftool debugedit \
+ 	nix-build nix-channel nix-collect-garbage nix-copy-closure \
+ 	nix-env nix-hash nix-install-package nix-instantiate \
+ 	nix-log2xml nix-prefetch-url nix-pull nix-push nix-store nix-worker \
+@@ -60,7 +60,7 @@ pkgbin_PROGRAMS =	\
+ 	rpmcache rpmdigest rpmrepo rpmspecdump \
+ 	rpmcmp rpmdeps rpmdeps-oecore sqlite3 @WITH_KEYUTILS_RPMKEY@ @WITH_LIBELF_DEBUGEDIT@
+ if WITH_DB
+-pkgbin_PROGRAMS +=	dbconvert
++pkgbin_PROGRAMS +=	
+ endif
+ dist_man_MANS =		rpmgrep.1
+ 
diff --git a/meta/recipes-devtools/rpm/rpm_5.4.9.bb b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
index 6ef1aae..e276cc3 100644
--- a/meta/recipes-devtools/rpm/rpm_5.4.9.bb
+++ b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
@@ -43,7 +43,7 @@ LICENSE = "LGPLv2.1"
 LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=2d5025d4aa3495befef8f17206a5b0a1"
 
 DEPENDS = "libpcre attr acl popt ossp-uuid file"
-PR = "r45"
+PR = "r46"
 
 # 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...
@@ -75,6 +75,7 @@ SRC_URI = "http://www.rpm5.org/files/rpm/rpm-5.4/rpm-5.4.9-0.20120508.src.rpm;ex
 	   file://uclibc-support.patch \
 	   file://rpmatch.patch \
 	   file://fstack-protector-configure-check.patch \
+	   file://dbconvert.patch \
 	  "
 
 SRC_URI[md5sum] = "60d56ace884340c1b3fcac6a1d58e768"
-- 
1.7.10





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] rpm_5.4.9.bb: fix builds on systems with older libc
  2012-07-10 17:14 [PATCH] rpm_5.4.9.bb: fix builds on systems with older libc Matthew McClintock
@ 2012-07-17 15:57 ` Saul Wold
  2012-07-19 15:40 ` Saul Wold
  1 sibling, 0 replies; 4+ messages in thread
From: Saul Wold @ 2012-07-17 15:57 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer; +Cc: Matthew McClintock

On 07/10/2012 10:14 AM, Matthew McClintock wrote:
> | dbconvert.o: In function `rpmdb_convert':
> | /local/jenkins/jobs/yocto-upstream/workspace/label/master/machine/p4080ds/poky/master/tmp/work/x86_64-linux/rpm-native-5.4.9-r45/rpm-5.4.9/tools/dbconvert.c:126: warning: the use of `tempnam' is dangerous, better use `mkstemp'
> | dbconvert.o: In function `main':
> | dbconvert.c:(.text+0x923): undefined reference to `htobe32'
> | dbconvert.c:(.text+0xaa4): undefined reference to `htole32'
> | dbconvert.c:(.text+0xac9): undefined reference to `htole32'
>
> Signed-off-by: Matthew McClintock <msm@freescale.com>
> ---
>   meta/recipes-devtools/rpm/rpm/dbconvert.patch |   27 +++++++++++++++++++++++++
>   meta/recipes-devtools/rpm/rpm_5.4.9.bb        |    3 ++-
>   2 files changed, 29 insertions(+), 1 deletion(-)
>   create mode 100644 meta/recipes-devtools/rpm/rpm/dbconvert.patch
>
Merged into OE-Core

Thanks
	Sau!

> diff --git a/meta/recipes-devtools/rpm/rpm/dbconvert.patch b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
> new file mode 100644
> index 0000000..27dba8c
> --- /dev/null
> +++ b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
> @@ -0,0 +1,27 @@
> +Upstream-Status: Not applicable
> +
> +Hack to prevent unneeded demo app from building on older libc
> +where it will break the build
> +
> +Index: rpm-5.4.9/tools/Makefile.am
> +===================================================================
> +--- rpm-5.4.9.orig/tools/Makefile.am
> ++++ rpm-5.4.9/tools/Makefile.am
> +@@ -24,7 +24,7 @@ AM_CFLAGS = $(OPENMP_CFLAGS)
> +
> + EXTRA_DIST =	hashtab.h bsdiff.1 bspatch.1 dotgraph.hh
> +
> +-EXTRA_PROGRAMS = augtool cudftool dbconvert debugedit \
> ++EXTRA_PROGRAMS = augtool cudftool debugedit \
> + 	nix-build nix-channel nix-collect-garbage nix-copy-closure \
> + 	nix-env nix-hash nix-install-package nix-instantiate \
> + 	nix-log2xml nix-prefetch-url nix-pull nix-push nix-store nix-worker \
> +@@ -60,7 +60,7 @@ pkgbin_PROGRAMS =	\
> + 	rpmcache rpmdigest rpmrepo rpmspecdump \
> + 	rpmcmp rpmdeps rpmdeps-oecore sqlite3 @WITH_KEYUTILS_RPMKEY@ @WITH_LIBELF_DEBUGEDIT@
> + if WITH_DB
> +-pkgbin_PROGRAMS +=	dbconvert
> ++pkgbin_PROGRAMS +=	
> + endif
> + dist_man_MANS =		rpmgrep.1
> +
> diff --git a/meta/recipes-devtools/rpm/rpm_5.4.9.bb b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
> index 6ef1aae..e276cc3 100644
> --- a/meta/recipes-devtools/rpm/rpm_5.4.9.bb
> +++ b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
> @@ -43,7 +43,7 @@ LICENSE = "LGPLv2.1"
>   LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=2d5025d4aa3495befef8f17206a5b0a1"
>
>   DEPENDS = "libpcre attr acl popt ossp-uuid file"
> -PR = "r45"
> +PR = "r46"
>
>   # 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...
> @@ -75,6 +75,7 @@ SRC_URI = "http://www.rpm5.org/files/rpm/rpm-5.4/rpm-5.4.9-0.20120508.src.rpm;ex
>   	   file://uclibc-support.patch \
>   	   file://rpmatch.patch \
>   	   file://fstack-protector-configure-check.patch \
> +	   file://dbconvert.patch \
>   	  "
>
>   SRC_URI[md5sum] = "60d56ace884340c1b3fcac6a1d58e768"
>




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] rpm_5.4.9.bb: fix builds on systems with older libc
  2012-07-10 17:14 [PATCH] rpm_5.4.9.bb: fix builds on systems with older libc Matthew McClintock
  2012-07-17 15:57 ` Saul Wold
@ 2012-07-19 15:40 ` Saul Wold
  2012-07-19 16:02   ` McClintock Matthew-B29882
  1 sibling, 1 reply; 4+ messages in thread
From: Saul Wold @ 2012-07-19 15:40 UTC (permalink / raw)
  To: Matthew McClintock; +Cc: Patches and discussions about the oe-core layer

On 07/10/2012 10:14 AM, Matthew McClintock wrote:
> | dbconvert.o: In function `rpmdb_convert':
> | /local/jenkins/jobs/yocto-upstream/workspace/label/master/machine/p4080ds/poky/master/tmp/work/x86_64-linux/rpm-native-5.4.9-r45/rpm-5.4.9/tools/dbconvert.c:126: warning: the use of `tempnam' is dangerous, better use `mkstemp'
> | dbconvert.o: In function `main':
> | dbconvert.c:(.text+0x923): undefined reference to `htobe32'
> | dbconvert.c:(.text+0xaa4): undefined reference to `htole32'
> | dbconvert.c:(.text+0xac9): undefined reference to `htole32'
>

Matthew,

Some how this patch has introduced a problem with the package-index 
process, as can be seen by the recent autobuilder nightly failures. I 
know this patch seems to remove dbconvert, but that causes problems for 
the package-index task.

We have filed a bug #2753 
(https://bugzilla.yoctoproject.org/show_bug.cgi?id=2753), to track this. 
  Can you take a look into this?

Since it is causing the AB to fail, we would like to get this resolved, 
Robert Yang is also looking at the issue. We will likely revert this 
patch tomorrow if we don't get a resolution sooner.

Thanks
	Sau!

> Signed-off-by: Matthew McClintock <msm@freescale.com>
> ---
>   meta/recipes-devtools/rpm/rpm/dbconvert.patch |   27 +++++++++++++++++++++++++
>   meta/recipes-devtools/rpm/rpm_5.4.9.bb        |    3 ++-
>   2 files changed, 29 insertions(+), 1 deletion(-)
>   create mode 100644 meta/recipes-devtools/rpm/rpm/dbconvert.patch
>
> diff --git a/meta/recipes-devtools/rpm/rpm/dbconvert.patch b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
> new file mode 100644
> index 0000000..27dba8c
> --- /dev/null
> +++ b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
> @@ -0,0 +1,27 @@
> +Upstream-Status: Not applicable
> +
> +Hack to prevent unneeded demo app from building on older libc
> +where it will break the build
> +
> +Index: rpm-5.4.9/tools/Makefile.am
> +===================================================================
> +--- rpm-5.4.9.orig/tools/Makefile.am
> ++++ rpm-5.4.9/tools/Makefile.am
> +@@ -24,7 +24,7 @@ AM_CFLAGS = $(OPENMP_CFLAGS)
> +
> + EXTRA_DIST =	hashtab.h bsdiff.1 bspatch.1 dotgraph.hh
> +
> +-EXTRA_PROGRAMS = augtool cudftool dbconvert debugedit \
> ++EXTRA_PROGRAMS = augtool cudftool debugedit \
> + 	nix-build nix-channel nix-collect-garbage nix-copy-closure \
> + 	nix-env nix-hash nix-install-package nix-instantiate \
> + 	nix-log2xml nix-prefetch-url nix-pull nix-push nix-store nix-worker \
> +@@ -60,7 +60,7 @@ pkgbin_PROGRAMS =	\
> + 	rpmcache rpmdigest rpmrepo rpmspecdump \
> + 	rpmcmp rpmdeps rpmdeps-oecore sqlite3 @WITH_KEYUTILS_RPMKEY@ @WITH_LIBELF_DEBUGEDIT@
> + if WITH_DB
> +-pkgbin_PROGRAMS +=	dbconvert
> ++pkgbin_PROGRAMS +=	
> + endif
> + dist_man_MANS =		rpmgrep.1
> +
> diff --git a/meta/recipes-devtools/rpm/rpm_5.4.9.bb b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
> index 6ef1aae..e276cc3 100644
> --- a/meta/recipes-devtools/rpm/rpm_5.4.9.bb
> +++ b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
> @@ -43,7 +43,7 @@ LICENSE = "LGPLv2.1"
>   LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=2d5025d4aa3495befef8f17206a5b0a1"
>
>   DEPENDS = "libpcre attr acl popt ossp-uuid file"
> -PR = "r45"
> +PR = "r46"
>
>   # 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...
> @@ -75,6 +75,7 @@ SRC_URI = "http://www.rpm5.org/files/rpm/rpm-5.4/rpm-5.4.9-0.20120508.src.rpm;ex
>   	   file://uclibc-support.patch \
>   	   file://rpmatch.patch \
>   	   file://fstack-protector-configure-check.patch \
> +	   file://dbconvert.patch \
>   	  "
>
>   SRC_URI[md5sum] = "60d56ace884340c1b3fcac6a1d58e768"
>




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] rpm_5.4.9.bb: fix builds on systems with older libc
  2012-07-19 15:40 ` Saul Wold
@ 2012-07-19 16:02   ` McClintock Matthew-B29882
  0 siblings, 0 replies; 4+ messages in thread
From: McClintock Matthew-B29882 @ 2012-07-19 16:02 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer, Saul Wold

On Thu, Jul 19, 2012 at 10:40 AM, Saul Wold <sgw@linux.intel.com> wrote:
> On 07/10/2012 10:14 AM, Matthew McClintock wrote:
>>
>> | dbconvert.o: In function `rpmdb_convert':
>> |
>> /local/jenkins/jobs/yocto-upstream/workspace/label/master/machine/p4080ds/poky/master/tmp/work/x86_64-linux/rpm-native-5.4.9-r45/rpm-5.4.9/tools/dbconvert.c:126:
>> warning: the use of `tempnam' is dangerous, better use `mkstemp'
>> | dbconvert.o: In function `main':
>> | dbconvert.c:(.text+0x923): undefined reference to `htobe32'
>> | dbconvert.c:(.text+0xaa4): undefined reference to `htole32'
>> | dbconvert.c:(.text+0xac9): undefined reference to `htole32'
>>
>
> Matthew,
>
> Some how this patch has introduced a problem with the package-index process,
> as can be seen by the recent autobuilder nightly failures. I know this patch
> seems to remove dbconvert, but that causes problems for the package-index
> task.
>
> We have filed a bug #2753
> (https://bugzilla.yoctoproject.org/show_bug.cgi?id=2753), to track this.
> Can you take a look into this?
>
> Since it is causing the AB to fail, we would like to get this resolved,
> Robert Yang is also looking at the issue. We will likely revert this patch
> tomorrow if we don't get a resolution sooner.

The intent of this patch is to disable building one object file. It's
possible I borked something else along the way?

-M

>
> Thanks
>         Sau!
>
>
>> Signed-off-by: Matthew McClintock <msm@freescale.com>
>> ---
>>   meta/recipes-devtools/rpm/rpm/dbconvert.patch |   27
>> +++++++++++++++++++++++++
>>   meta/recipes-devtools/rpm/rpm_5.4.9.bb        |    3 ++-
>>   2 files changed, 29 insertions(+), 1 deletion(-)
>>   create mode 100644 meta/recipes-devtools/rpm/rpm/dbconvert.patch
>>
>> diff --git a/meta/recipes-devtools/rpm/rpm/dbconvert.patch
>> b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
>> new file mode 100644
>> index 0000000..27dba8c
>> --- /dev/null
>> +++ b/meta/recipes-devtools/rpm/rpm/dbconvert.patch
>> @@ -0,0 +1,27 @@
>> +Upstream-Status: Not applicable
>> +
>> +Hack to prevent unneeded demo app from building on older libc
>> +where it will break the build
>> +
>> +Index: rpm-5.4.9/tools/Makefile.am
>> +===================================================================
>> +--- rpm-5.4.9.orig/tools/Makefile.am
>> ++++ rpm-5.4.9/tools/Makefile.am
>> +@@ -24,7 +24,7 @@ AM_CFLAGS = $(OPENMP_CFLAGS)
>> +
>> + EXTRA_DIST =  hashtab.h bsdiff.1 bspatch.1 dotgraph.hh
>> +
>> +-EXTRA_PROGRAMS = augtool cudftool dbconvert debugedit \
>> ++EXTRA_PROGRAMS = augtool cudftool debugedit \
>> +       nix-build nix-channel nix-collect-garbage nix-copy-closure \
>> +       nix-env nix-hash nix-install-package nix-instantiate \
>> +       nix-log2xml nix-prefetch-url nix-pull nix-push nix-store
>> nix-worker \
>> +@@ -60,7 +60,7 @@ pkgbin_PROGRAMS =    \
>> +       rpmcache rpmdigest rpmrepo rpmspecdump \
>> +       rpmcmp rpmdeps rpmdeps-oecore sqlite3 @WITH_KEYUTILS_RPMKEY@
>> @WITH_LIBELF_DEBUGEDIT@
>> + if WITH_DB
>> +-pkgbin_PROGRAMS +=    dbconvert
>> ++pkgbin_PROGRAMS +=
>> + endif
>> + dist_man_MANS =               rpmgrep.1
>> +
>> diff --git a/meta/recipes-devtools/rpm/rpm_5.4.9.bb
>> b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
>> index 6ef1aae..e276cc3 100644
>> --- a/meta/recipes-devtools/rpm/rpm_5.4.9.bb
>> +++ b/meta/recipes-devtools/rpm/rpm_5.4.9.bb
>> @@ -43,7 +43,7 @@ LICENSE = "LGPLv2.1"
>>   LIC_FILES_CHKSUM =
>> "file://COPYING.LIB;md5=2d5025d4aa3495befef8f17206a5b0a1"
>>
>>   DEPENDS = "libpcre attr acl popt ossp-uuid file"
>> -PR = "r45"
>> +PR = "r46"
>>
>>   # 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...
>> @@ -75,6 +75,7 @@ SRC_URI =
>> "http://www.rpm5.org/files/rpm/rpm-5.4/rpm-5.4.9-0.20120508.src.rpm;ex
>>            file://uclibc-support.patch \
>>            file://rpmatch.patch \
>>            file://fstack-protector-configure-check.patch \
>> +          file://dbconvert.patch \
>>           "
>>
>>   SRC_URI[md5sum] = "60d56ace884340c1b3fcac6a1d58e768"
>>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-07-19 16:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-10 17:14 [PATCH] rpm_5.4.9.bb: fix builds on systems with older libc Matthew McClintock
2012-07-17 15:57 ` Saul Wold
2012-07-19 15:40 ` Saul Wold
2012-07-19 16:02   ` McClintock Matthew-B29882

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.