search

cookyshi / libjingle

Automatically exported from code.google.com/p/libjingle
0 stars 0 forks source link

PATCH: missing expat_config.h #287

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Patch that fixes a hammer.sh error about missing expat_config.h.

Original issue reported on code.google.com by fabian.p...@gmail.com on 28 Jan 2012 at 3:29

Attachments:

GoogleCodeExporter commented 9 years ago 
I don't think this patch is needed. What platform do yo use? Did you follow the 
build instruction in the README file exactly?

Original comment by jun...@google.com on 10 Feb 2012 at 12:13

GoogleCodeExporter commented 9 years ago 
Sorry for the missing information, I thought it was a generalized problem. I'm 
using OSX (macports) and yes I followed the README

Original comment by fabian.p...@gmail.com on 12 Feb 2012 at 10:52

GoogleCodeExporter commented 9 years ago 
Not able to reproduce this issue using Mac OSX 10.7.2 w/XCode 4.2.1

Original comment by jun...@google.com on 13 Feb 2012 at 7:27

GoogleCodeExporter commented 9 years ago 
Confirmed on Fedora:

________Compiling build/dbg/obj/third_party/expat-2.0.1/lib/xmlparse.o
third_party/expat-2.0.1/lib/xmlparse.c:20:26: error: expat_config.h: No such 
file or directory
third_party/expat-2.0.1/lib/xmlparse.c:81:2: error: #error memmove does not 
exist on this platform, nor is a substitute available
scons: *** [build/dbg/obj/third_party/expat-2.0.1/lib/xmlparse.o] Error 1
scons: building terminated because of errors.

The proposed patch is a good fix.  I opted instead for copying 
third_party/expat-2.0.1/expat_config.h into third_party/expat-2.0.1/lib

Jun, is it possible that you have expat_config.h in your system includes, 
masking the issue?

Original comment by bprodo...@connectify.me on 14 Feb 2012 at 5:17

GoogleCodeExporter commented 9 years ago 
Can anyone plz tell me How to use this patch ?

Original comment by std.a...@gmail.com on 25 Nov 2012 at 1:30