I don't know if I have any points to give here, but if NPN does not make it to 2.4, it's fine with me to remove it from trunk also. Only mod_spdy is affected afaik.
//Stefan > Am 23.04.2015 um 06:56 schrieb Kaspar Brand <httpd-dev.2...@velox.ch>: > >> On 22.04.2015 21:30, Rainer Jung wrote: >>> Am 22.04.2015 um 17:49 schrieb Kaspar Brand: >>> Thanks. In the patch for ssl_private.h, the complete NPN block should >>> actually be dropped - the same block is are already part of >>> ssl_private.h, just 10 lines above. >> >> I've kept the new one and dropped the old one in r1675459. > > It's nitpicking, but... the old one was already correct, actually. All > this stuff is in a larger "#if !defined(OPENSSL_NO_TLSEXT) ..." block, > so there's no need to repeat NO_TLSEXT. > > I'd rather get consensus on completely dropping HAVE_TLS_NPN from trunk > anyway, though, so it's really a very minor point right now ;-) > > Kaspar