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

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Reply via email to