On Tue, Jun 6, 2017 at 8:51 AM, Emil Velikov <emil.l.veli...@gmail.com> wrote: > > On 5 June 2017 at 21:31, Aaron Watry <awa...@gmail.com> wrote: > > Otherwise r600g will fail to build when the amdgpu drm library is missing > > > And the code that pulls amdgpu is not used by r600g, which makes this > misplaced duck tape. > As you/Jan don't have the time to properly address this, I'll try to > find some later today.
Thank you. You're right that I don't have a lot of time. If I'm lucky, I get 30 minutes in the average evening to poke at any of my own hobbies, including this. I'd love to be able to spend the whole day ripping the amdgpu dependency out of the parts of amd/common that r600 uses, but that's just not possible this week (or next week), hence the bandaid. Combine that with a lack of understanding of autotools and you can see how we ended up here. What Jan and I sent gets the r600 build working again, without losing functionality, and buys us time to deal with the amdgpu dependency in the amd/common and gallium/radeon parts, which honestly is all I had time to do at the moment without taking a day off of work to rip amd/common apart to fix and test it properly. --Aaron > > > -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev