[vlc-devel] [PATCH 2/6] configure: add check for attributes

[Filip Roséen]

Hi Rémi,

On 2017-02-26 21:50, Rémi Denis-Courmont wrote:

> Le sunnuntaina 26. helmikuuta 2017, 20.45.38 EET Filip Roséen a écrit :
> > Hi Rémi,
> > 
> > On 2017-02-26 21:27, Rémi Denis-Courmont wrote:
> > > As has been noted many times over, the VLC headers CANNOT depend on
> > > configure output (since they are installed by the plugin SDK).
> > > 
> > > That is why all public headers use compiler/version checks.
> > > 
> > > So obvious nack.
> > 
> > What do you propose as an alternative?
> 
> Me? I do not propose anything. I propose to leave things as they are, if you 
> will.

Acknowledged.

> I am definitely not letting the headers get broken again just because of some 
> buggy compiler version.

 - So you consider every version of `clang` that was released prior to
   `5.0.0` to be buggy?

I found the part related to nightly builds in my previous reply (that
is not present in the email which this is a reply to) to be of much
relevance:

 - https://mailman.videolan.org/pipermail/vlc-devel/2017-February/111756.html
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170226/c2b17dea/attachment.html>