> Ideally at some point maybe we would make this clearer but not now. Okay, sounds good.
> Have you tested the situations you describe ? That might be a better > way of checking that it's right than the code inspection which is > obviously failing for me now... Yes, I have tested the following situations: (1) ./configure without go installed => go is not enabled (2) ./configure with go version < 1.11.1 => go is not enabled (3) ./configure with go version > 1.11.1 => go is enabled (4) ./configure --enable-golang without go installed => error (5) ./configure --enable-golang with go < 1.11.1 => error (6) ./configure --disable-golang in any case => go is not enabled > > I definitely think we want to fix this for 4.14. So thanks for your > continued help! You're welcome. Thank you again for the review. -NR
