https://bugs.freedesktop.org/show_bug.cgi?id=50754

--- Comment #26 from Tapani Pälli <lem...@gmail.com> ---
(In reply to comment #25)
> Created attachment 89621 [details] [review]
> Move LT_INIT where it should so it can test correctly (AM_)C(XX)FLAGS and
> others
> 
> Updated patch against latest git code.
> Based on previous patch with further explanations.
> 
> LT_INIT needs to work with the final content of variables to be able to
> correctly run some tests and determine the host type. So it must be called
> after setting anything we want to set in (AM_)C(XX)FLAGS or (AM_)LDFLAGS.
> 
> Fixing LT_INIT must be done independently from modifying environment
> variables or build variables since LT_INIT will check both variables and
> shadow variables if they are set.
> 
> Tested on my setup and it works. I'd like someone else to also test it.
> 
> Another patch should be made to fix llvm-config, which is another topic.

Yep I verified this on my machine, the patch still works with the comments :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to