[PATCH] Improve the unwind.h inclusion

[Sylvestre Ledru]

Le 12/11/2013 20:50, Richard Smith a écrit :
> On Tue, Nov 12, 2013 at 10:23 AM, Rafael Espíndola
> <rafael.espindola at gmail.com <mailto:rafael.espindola at gmail.com>> wrote:
>
>     On 12 November 2013 13:18, Sylvestre Ledru <sylvestre at debian.org
>     <mailto:sylvestre at debian.org>> wrote:
>     >
>     >   I haven't dig enough in this to answer to your question about
>     unwind.h itself.
>     >
>     >   It is just that I needed this patch when building with the
>     package installed (I used to hardcode the path to the clang library).
>     >   I think that applying this patch might save some time to the
>     other packagers (but it might no longer be relevant).
>
>     The question about unwind.h being complete was just a curiosity, but a
>     just built clang looking at /usr/include/clang/3.4/include seems like
>     a real bug that this might just hide.
>
>
> Right, as I mentioned elsewhere, Debian's local patch to force
> /usr/include/clang/3.4/include into the include path is incorrect. I
> don't think we should take patches upstream to work around your local
> issues. I would suggest that you pick a different solution to your
> packaging issue -- perhaps look for that directory via a
> compiler-relative path, not an absolute one, as we do for
> ../lib/clang/3.4/include. (Or, failing that, apply this patch to your
> local sources, and be aware that there may be more pain in your future...)
I agree and I fixed that:
Now, clang is searching in
@clang-binary@/../include/clang/VERSION/include/ (instead of
/usr/include/clang/VERSION/include/)
(based on Richard's suggestion from a previous thread).

Sylvestre