Openembedded Devel Discussions
 help / color / mirror / Atom feed
* [2011.03-maintenance 0/1] pull request 20121003
@ 2012-10-03 12:22 Steffen Sledz
  2012-10-03 12:22 ` [2011.03-maintenance 1/1] subversion-1.6.5: fix QA issue Steffen Sledz
  2012-10-09 14:41 ` [2011.03-maintenance 0/1] pull request 20121003 Tom Rini
  0 siblings, 2 replies; 3+ messages in thread
From: Steffen Sledz @ 2012-10-03 12:22 UTC (permalink / raw)
  To: Tom Rini, openembedded-devel

Hi Tom,

please pull this commit which fixes the QA issue

 ERROR: This autoconf log indicates errors, it looked at host includes.

for subversion by backporting some build related patches and recipe
modifications from oe-core.

Thx,
Steffen

The following changes since commit a35ceaacb2019750dc9f4b4fd5ea927cf2ad47fb:

  gdk-pixbuf: add missing dependencies (2012-09-14 15:07:04 -0700)

are available in the git repository at:

  git://github.com/sledz/oe pull-request-2
  https://github.com/sledz/oe/tree/pull-request-2

Steffen Sledz (1):
  subversion-1.6.5: fix QA issue

 recipes/subversion/files/fix-install-depends.patch |   45 ++++++++++++++++++++
 recipes/subversion/files/libtool2.patch            |   17 ++++++++
 recipes/subversion/subversion_1.6.5.bb             |   21 ++++++---
 3 files changed, 76 insertions(+), 7 deletions(-)
 create mode 100644 recipes/subversion/files/fix-install-depends.patch
 create mode 100644 recipes/subversion/files/libtool2.patch

-- 
1.7.10.4




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

* [2011.03-maintenance 1/1] subversion-1.6.5: fix QA issue
  2012-10-03 12:22 [2011.03-maintenance 0/1] pull request 20121003 Steffen Sledz
@ 2012-10-03 12:22 ` Steffen Sledz
  2012-10-09 14:41 ` [2011.03-maintenance 0/1] pull request 20121003 Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Steffen Sledz @ 2012-10-03 12:22 UTC (permalink / raw)
  To: Tom Rini, openembedded-devel

This patch fixes the QA issue

 ERROR: This autoconf log indicates errors, it looked at host includes.

by backporting some build related patches and recipe modifications
from oe-core.

Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de>
---
 recipes/subversion/files/fix-install-depends.patch |   45 ++++++++++++++++++++
 recipes/subversion/files/libtool2.patch            |   17 ++++++++
 recipes/subversion/subversion_1.6.5.bb             |   21 ++++++---
 3 files changed, 76 insertions(+), 7 deletions(-)
 create mode 100644 recipes/subversion/files/fix-install-depends.patch
 create mode 100644 recipes/subversion/files/libtool2.patch

diff --git a/recipes/subversion/files/fix-install-depends.patch b/recipes/subversion/files/fix-install-depends.patch
new file mode 100644
index 0000000..6f49ed4
--- /dev/null
+++ b/recipes/subversion/files/fix-install-depends.patch
@@ -0,0 +1,45 @@
+install-neon-lib should depend on libsvn_delta's installation
+
+install-neon-lib needs libsvn_delta-1.la which will be regenerated
+during libsvn_delta-1.la's installation, if libsvn_delta-1.la is
+in regenerating and at the same time install-neon-lib links it, the
+error willl happen.
+
+Let install-neon-lib run after libsvn_delta-1.la is installed will fix
+the problem.
+
+Upstream-Status: Pending
+
+Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
+---
+ build-outputs.mk |    2 +-
+ build.conf       |    2 ++
+ 2 files changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/build-outputs.mk b/build-outputs.mk
+--- a/build-outputs.mk
++++ b/build-outputs.mk
+@@ -979,7 +979,7 @@ install-locale: subversion/po/de.mo subversion/po/es.mo subversion/po/fr.mo subv
+ 	$(MKDIR) $(DESTDIR)$(localedir)/zh_TW/LC_MESSAGES
+ 	cd subversion/po ; $(INSTALL_LOCALE) zh_TW.mo $(DESTDIR)$(localedir)/zh_TW/LC_MESSAGES/$(PACKAGE_NAME).mo
+ 
+-install-neon-lib: subversion/libsvn_ra_neon/libsvn_ra_neon-1.la
++install-neon-lib: subversion/libsvn_ra_neon/libsvn_ra_neon-1.la $(SVN_FS_LIB_INSTALL_DEPS)
+ 	$(MKDIR) $(DESTDIR)$(neon_libdir)
+ 	cd subversion/libsvn_ra_neon ; $(INSTALL_NEON_LIB) libsvn_ra_neon-1.la $(DESTDIR)$(neon_libdir)/libsvn_ra_neon-1.la
+ 
+diff --git a/build.conf b/build.conf
+--- a/build.conf
++++ b/build.conf
+@@ -272,6 +272,8 @@ type = ra-module
+ path = subversion/libsvn_ra_neon
+ install = neon-lib
+ libs = libsvn_delta libsvn_subr aprutil apriconv apr neon
++# conditionally add more dependencies
++add-install-deps = $(SVN_FS_LIB_INSTALL_DEPS)
+ msvc-static = yes
+ 
+ # Accessing repositories via DAV through serf
+-- 
+1.7.10.4
+
diff --git a/recipes/subversion/files/libtool2.patch b/recipes/subversion/files/libtool2.patch
new file mode 100644
index 0000000..32f88b7
--- /dev/null
+++ b/recipes/subversion/files/libtool2.patch
@@ -0,0 +1,17 @@
+Upstream-Status: Inappropriate [embedded specific]
+
+Index: subversion-1.5.5/configure.ac
+===================================================================
+--- subversion-1.5.5.orig/configure.ac	2008-08-26 18:27:56.000000000 +0100
++++ subversion-1.5.5/configure.ac	2009-01-07 18:00:47.000000000 +0000
+@@ -153,8 +153,8 @@
+   LIBTOOL="$sh_libtool"
+   SVN_LIBTOOL="$sh_libtool"
+ else
+-  sh_libtool="$abs_builddir/libtool"
+-  SVN_LIBTOOL="\$(SHELL) $sh_libtool"
++  sh_libtool="$abs_builddir/$host_alias-libtool"
++  SVN_LIBTOOL="\$(SHELL) \$(abs_builddir)/$host_alias-libtool"
+   dnl libtoolize requires that the following line not be indented
+ ifdef([LT_INIT], [LT_INIT], [AC_PROG_LIBTOOL])
+ fi
diff --git a/recipes/subversion/subversion_1.6.5.bb b/recipes/subversion/subversion_1.6.5.bb
index 51ec724..2a563ca 100644
--- a/recipes/subversion/subversion_1.6.5.bb
+++ b/recipes/subversion/subversion_1.6.5.bb
@@ -5,23 +5,30 @@ RDEPENDS_${PN} = "neon"
 LICENSE = "Apache BSD"
 HOMEPAGE = "http://subversion.tigris.org/"
 
-PR = "r1"
+PR = "r2"
 
 SRC_URI = "http://subversion.tigris.org/downloads/${P}.tar.bz2 \
-	   file://disable-revision-install.patch"
+	   file://disable-revision-install.patch \
+	   file://libtool2.patch \
+	   file://fix-install-depends.patch \
+	   "
 
 EXTRA_OECONF = "--without-berkeley-db --without-apxs --without-apache \
                 --without-swig --with-apr=${STAGING_BINDIR_CROSS} \
-                --with-apr-util=${STAGING_BINDIR_CROSS}"
+                --with-apr-util=${STAGING_BINDIR_CROSS} \
+		ac_cv_path_RUBY=none"
 
 
 inherit autotools
 
-acpaths = "-I build/ac-macros"
+export LDFLAGS += " -L${STAGING_LIBDIR} "
 
-# FIXME: Ugly hack!
-do_configure_append() {
-	if ! test -f libtool ; then cp -a *-libtool libtool ; fi
+acpaths = "-I build/ -I build/ac-macros/"
+
+do_configure_prepend () {
+	rm -f ${S}/libtool
+	rm -f ${S}/build/libtool.m4
+	sed -i -e 's:with_sasl="/usr/local":with_sasl="${STAGING_DIR}":' ${S}/build/ac-macros/sasl.m4
 }
 
 SRC_URI[md5sum] = "1a53a0e72bee0bf814f4da83a9b6a636"
-- 
1.7.10.4




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

* Re: [2011.03-maintenance 0/1] pull request 20121003
  2012-10-03 12:22 [2011.03-maintenance 0/1] pull request 20121003 Steffen Sledz
  2012-10-03 12:22 ` [2011.03-maintenance 1/1] subversion-1.6.5: fix QA issue Steffen Sledz
@ 2012-10-09 14:41 ` Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2012-10-09 14:41 UTC (permalink / raw)
  To: Steffen Sledz; +Cc: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 1211 bytes --]

On Wed, Oct 03, 2012 at 02:22:29PM +0200, Steffen Sledz wrote:

> Hi Tom,
> 
> please pull this commit which fixes the QA issue
> 
>  ERROR: This autoconf log indicates errors, it looked at host includes.
> 
> for subversion by backporting some build related patches and recipe
> modifications from oe-core.
> 
> Thx,
> Steffen
> 
> The following changes since commit a35ceaacb2019750dc9f4b4fd5ea927cf2ad47fb:
> 
>   gdk-pixbuf: add missing dependencies (2012-09-14 15:07:04 -0700)
> 
> are available in the git repository at:
> 
>   git://github.com/sledz/oe pull-request-2
>   https://github.com/sledz/oe/tree/pull-request-2
> 
> Steffen Sledz (1):
>   subversion-1.6.5: fix QA issue
> 
>  recipes/subversion/files/fix-install-depends.patch |   45 ++++++++++++++++++++
>  recipes/subversion/files/libtool2.patch            |   17 ++++++++
>  recipes/subversion/subversion_1.6.5.bb             |   21 ++++++---
>  3 files changed, 76 insertions(+), 7 deletions(-)
>  create mode 100644 recipes/subversion/files/fix-install-depends.patch
>  create mode 100644 recipes/subversion/files/libtool2.patch

Applied to 2011.03-maintenance, thanks and sorry for the delay.

-- 
Tom

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

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

end of thread, other threads:[~2012-10-09 14:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-03 12:22 [2011.03-maintenance 0/1] pull request 20121003 Steffen Sledz
2012-10-03 12:22 ` [2011.03-maintenance 1/1] subversion-1.6.5: fix QA issue Steffen Sledz
2012-10-09 14:41 ` [2011.03-maintenance 0/1] pull request 20121003 Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox