[MPlayer-dev-eng] [PATCH] build: Check if SCTP is available

Alexander Strasser eclipse7 at gmx.net
Sun May 13 22:04:56 CEST 2012


Hi Reimar,

Reimar Döffinger wrote:
> On 13 May 2012, at 18:33, Alexander Strasser <eclipse7 at gmx.net> wrote:
[...]
> >  I did miss it is available in MPlayer's configure. But I found it
> > now under the name header_check.
> 
> IMO please commit.
> Though you could replace the innermost if by &&, i.e.
> header_check ... && sctp=yes

  Committed.

  I did see the opportunity to get rid of this "if" but as it is
a matter of personal taste I wasn't sure what to do; so I left it
as it was before in Johan's patch. I was under the impression his
patch was strongly modelled after another existing check.

  As the matter is quite annoying and multiple users (rightly)
complained until now I did not want to edit the tested patch.

  If you insist on avoiding the "if" I can change it locally and
commit after I tested it.

  Alexander


More information about the MPlayer-dev-eng mailing list