git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Include xmlparse.h instead of expat.h on QNX

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 11, 2013, 21:53 UTC
Message-ID
<7v38x2ldow.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20130211214948.GB19113@ftbfs.org>
Matt Kraai <kraai@ftbfs.org> writes:
Show 6 quoted lines
>> Assuming that this change is about building with expat1, it would
>> probably be better to do something like this instead, I would think.
>
> expat 1.95.0 through 1.95.8 used expat.h; should I still use
> EXPAT_VERSION = 1 to signify that it should use xmlparse.h, use
> EXPAT_NEEDS_XMLPARSE_H as Jeff suggested, or something else entirely?

Oh, please do not take it as a request to use that exact name (in case you didn't know, I am bad at naming things). It was merely an illustration to show the direction, written without knowing that Peff was essentially giving the same suggestion.

Thanks.

Oh, by the way, please do not deflect an attempt to directly send a response to you with a Mail-Followup-To header.

Previous: Matt Kraai
Message 11 of 11 in “Include xmlparse.h instead of expat.h on QNX”
  1. Include xmlparse.h instead of expat.h on QNXMatt Kraai, Feb 11, 2013
  2. Jeff KingFeb 11, 2013
  3. Matt KraaiFeb 11, 2013
  4. Allow building with xmlparse.hMatt Kraai, Feb 11, 2013
  5. Junio C HamanoFeb 11, 2013
  6. Allow building with xmlparse.hMatt Kraai, Feb 11, 2013
  7. Junio C HamanoFeb 11, 2013
  8. Junio C HamanoFeb 11, 2013
  9. Junio C HamanoFeb 11, 2013
  10. Matt KraaiFeb 11, 2013
  11. Junio C HamanoFeb 11, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.