[oe] [PATCH] gsoap_2.7.13 build errors

[Paul Menzel]

Am Montag, den 14.03.2011, 10:56 +0100 schrieb Peter Gsellmann:
> Am Freitag, 11. März 2011, 13:25:15 schrieb Paul Menzel:

> > Am Freitag, den 11.03.2011, 11:38 +0100 schrieb Peter Gsellmann:
> > > There is a compiler error about an impossible cast from QUIRKS* to const QUIRKS*
> > > The patches are from Fedora12 which has the exact same gsoap-version
> > > They can be found at http://archive.fedoraproject.org/pub/archive/fedora/linux/releases/12/Everything/source/SRPMS/gsoap-2.7.13-2.fc12.src.rpm
> > > 
> > > In the native package, the generated binary is installed in STAGING_BINDIR_NATIVE where expected by the gsoap-package
> > > 
> > > Otherwise the gsoap-package uses a soapcpp2 delivered as 32bit-executable in the source-package! This fails on x86_64 hosts.
> > 
> > smaller patches would it make easier to review in my opinion. (If it
> > were possible.)
> The patches are as found in Fedora gsoap-2.7.13-2.fc12.src.rpm . I did not make them. Credits to the fedora team.
> 
> > On what did you test this patch?
> I run a daily build for my target which is Angstrom with some extra packages. Target hardware is at91sam9g20.
> 
> > > diff --git a/recipes/gsoap/gsoap-native_2.7.13.bb b/recipes/gsoap/gsoap-native_2.7.13.bb
> > > index 3e905e7..83c2329 100644
> > > --- a/recipes/gsoap/gsoap-native_2.7.13.bb
> > > +++ b/recipes/gsoap/gsoap-native_2.7.13.bb
> > > @@ -6,8 +6,9 @@ EXTRA_OEMAKE = ""
> > >  
> > >  inherit native
> > >  
> > > +
> > 
> > Not needed?
> > 
> > >  do_install() {
> > > -	:
> > > +	install -m 0755 ${S}/gsoap/src/soapcpp2 ${STAGING_BINDIR_NATIVE}/
> > >  }
> Only soapcpp2 is needed, exactly in this place.

I meant the newly added line before the function.

> > > diff --git a/recipes/gsoap/gsoap_2.7.13.inc b/recipes/gsoap/gsoap_2.7.13.inc
> > > index 877e1d1..c3ab3c4 100644
> > > --- a/recipes/gsoap/gsoap_2.7.13.inc
> > > +++ b/recipes/gsoap/gsoap_2.7.13.inc
> > > @@ -3,7 +3,9 @@ for the development of SOAP Web Services and clients."
> > >  SECTION = "devel"
> > >  LICENSE = "GPL"
> > >  
> > > -SRC_URI = "${SOURCEFORGE_MIRROR}/gsoap2/gsoap_${PV}.tar.gz"
> > > +SRC_URI = "${SOURCEFORGE_MIRROR}/gsoap2/gsoap_${PV}.tar.gz	\
> > > +	file://gsoap-2.7-openssl.patch	\
> > 
> > Use spaces instead of tabulators before »\«?
> > 
> > > +	file://unused_args.patch"
> > 
> > Maybe you can move the quote »"« on a new line, so that future patches
> > will not create a difference for that line.
> I will repost a reworked patch without tabs today

Great! Please remember to tag it as »[PATCH v2]«.


Thanks,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110314/21beee69/attachment-0002.sig>