On 4 December 2017 at 18:06, Dylan Baker <dy...@pnwbakers.com> wrote: > Quoting Emil Velikov (2017-12-04 05:20:03) >> On 29 November 2017 at 00:56, Dylan Baker <dy...@pnwbakers.com> wrote: >> > Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> >> > --- >> > meson.build | 11 +++++++++++ >> > meson_options.txt | 7 +++++++ >> > src/gallium/drivers/etnaviv/meson.build | 2 +- >> > src/gallium/drivers/freedreno/meson.build | 1 + >> > src/gallium/drivers/nouveau/meson.build | 2 +- >> > src/gallium/drivers/r300/meson.build | 4 +++- >> > src/gallium/targets/dri/meson.build | 2 +- >> I'd stick with the autotools approach and add the link to the >> gallium/aux module. >> There are simply too many combinations where lmsensors can be utilised. >> >> -Emil > > lmsensors is in one of the variables in src/gallium/Automake.inc, I copied the > location from there, if it would be better to link just to aux we can do that > instead. > Memory is failing - forgot I moved that one.
If possible I'd recommend having something like GALLIUM_COMMON_LIB_DEPS - a group of shared libraries that all final binaries link against. Alternatively moving it to src/gallium/auxiliary/meson.build should also work. Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
