On Tue, Nov 12, 2013 at 10:23 AM, Rafael Espíndola < [email protected]> wrote:
> On 12 November 2013 13:18, Sylvestre Ledru <[email protected]> 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...)
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
