[Mesa-maintainers] [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

[Ilia Mirkin]

On Tue, Dec 13, 2016 at 11:08 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 13 December 2016 at 15:25, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> On Thu, Dec 8, 2016 at 2:21 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>>> From: Emil Velikov <emil.velikov at collabora.com>
>>>
>>> Since day 1, Vulkan has depended on --with-egl-platforms to select the
>>> platforms build.
>>>
>>> With earlier commits, we've attributed for that internally by renaming
>>> the [internal] conditionals in our build. At the same time having the
>>> --enable-egl and --with-egl-platforms dependency for Vulkan makes no
>>> sense.
>>>
>>> Since where' on the bridge we want to make the conditional generic, thus
>>> others (such as the gallium VL targets) can honour the respective
>>> platforms.
>>>
>>> Cc: mesa-maintainers at lists.freedesktop.org
>>> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
>>> ---
>>> Should we keep the old configure and error out when anyone provides a
>>> non-default value ? This way they'll have direct feedback how to move
>>> forward. Otherwise they'll likely miss the warning message.
>>
>> I can't tell what problem this is solving. OTOH this is creating the
>> problem that everyone's configure lines will need fixing.
> This need fixing (builder intervention) either way, I'm afraid. Question is
>  a) should we expect people to read the release notes, or
>  b) get their attention with an error message, providing an example
> in order to do the right thing.

(c) leave things alone if this isn't fixing anything other than aesthetics.