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

Alexandre Demers <alexandre.f.dem...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #65690|0                           |1
        is obsolete|                            |

--- Comment #25 from Alexandre Demers <alexandre.f.dem...@gmail.com> ---
Created attachment 89621
  --> https://bugs.freedesktop.org/attachment.cgi?id=89621&action=edit
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.

-- 
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