On Fri, Jun 24, 2011 at 09:29, Jie Zhang wrote:
> [Resend with a smaller patch. The original patch email is pending for
> list moderator's approval because the size exceeding 100KB. I don't
> know when the moderator will see it. So I resend it with a smaller
> patch with trivial stuff removed.]

i can add you to said list if you want

> This patch imports hex.c from libiberty in GCC.

as long as it's GPL-2+, i dont see a problem

> It also move libiberty
> headers from src/lib/ to include/. So if source file in UrJTAG want to
> use functions from libiberty, it can just include libiberty.h and
> don't need to declare the function by itself. This is also how GCC,
> GDB and Binutils use libiberty.

i wonder if we should have it be include/lib/ and then have the
toplevel code take care of putting -I$(top_srcdir)/include/lib.

also, did you update src/lib/Makefile.am and include/Makefile.am to
reflect the header move ?  otherwise `make dist` wont work ...
-mike

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a 
definitive record of customers, application performance, security 
threats, fraudulent activity and more. Splunk takes this data and makes 
sense of it. Business sense. IT sense. Common sense.. 
http://p.sf.net/sfu/splunk-d2d-c1
_______________________________________________
UrJTAG-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/urjtag-development

Reply via email to