[Mesa-dev] [PATCH 4/4] configure.ac: remove shared/static options

Dan Nicholson dbn.lists at gmail.com
Sat Feb 11 08:56:02 PST 2012


On Fri, Jan 20, 2012 at 12:57 PM, Matt Turner <mattst88 at gmail.com> wrote:
> On Sat, Jan 14, 2012 at 11:16 AM, Matt Turner <mattst88 at gmail.com> wrote:
>> now that libtool provides them.
>>
>> Signed-off-by: Matt Turner <mattst88 at gmail.com>
>> ---
>
> Ping. This is sensible, right?

Sorry, I missed this. I _think_ so. I'm kind of wary of relying on a
bolted on libtool rather than a fully integrated libtool, but I think
for the purpose of these variables it should be fine. Did you actually
check that LT_INIT sets enable_shared and enable_static like we're
using here? I'm pretty sure it does, but I'd appreciate if you could
take a look.

If that's the case,

Reviewed-by: Dan Nicholson <dbn.lists at gmail.com>

--
Dan


More information about the mesa-dev mailing list