[Mesa-dev] [PATCH 2/3] meson: Add lmsensors support
emil.l.velikov at gmail.com
Mon Dec 4 18:14:15 UTC 2017
On 4 December 2017 at 18:06, Dylan Baker <dylan at pnwbakers.com> wrote:
> Quoting Emil Velikov (2017-12-04 05:20:03)
>> On 29 November 2017 at 00:56, Dylan Baker <dylan at pnwbakers.com> wrote:
>> > Signed-off-by: Dylan Baker <dylanx.c.baker at 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.
> 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
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.
More information about the mesa-dev