Sorry, I missed that when rebasing. I'm not sure how. SMP_T_CSTR needs to be changed to SMP_T_STR
On Wed, Apr 9, 2014 at 8:45 AM, Lukas Tribus <[email protected]> wrote: > Hi Willy, hi David, > > > > On Tue, Apr 08, 2014 at 06:48:47PM -0400, David S wrote: > >> Thank you Lukas. > >> Here is the rebased patch. > >> I also made one correction, I had added ssl_fc_unique_id as an ACL > keyword, > >> but that does not make sense. I removed that added line from my patch. > > > > Your first contribution was just merged :-) > > > > > http://git.1wt.eu/web?p=haproxy.git;a=commitdiff;h=c1ad52e8f781c301f6308d27f788354b77da1610 > > > Mmmh, building current master fails with: > src/ssl_sock.c: In function 'smp_fetch_ssl_fc_unique_id': > src/ssl_sock.c:2854:14: error: 'SMP_T_CSTR' undeclared (first use in this > function) > src/ssl_sock.c:2854:14: note: each undeclared identifier is reported only > once for each function it appears in > src/ssl_sock.c: At top level: > src/ssl_sock.c:3589:90: error: 'SMP_T_CSTR' undeclared here (not in a > function) > make: *** [src/ssl_sock.o] Error 1 > > > > It did not fail yesterday, when I used the first patch on top of dev22. > Looks like the second (rebased) patch misses something. > > > > > Regards, > > Lukas > >
