Ilya,

On 5/8/21 9:19 PM, Илья Шипицин wrote:
I've created "proof of concept" OT support for CI

https://github.com/chipitsine/haproxy/runs/2535689580

Tim, are you ok with that approach ? If yes, I will polish things a bit.

If the build time is reduced appropriately then I am fine with that. Please order the 'apt' dependencies alphabetically please.

I'm not very happy with unconditionally adding OT flags
https://github.com/chipitsine/haproxy/commit/dcf1812b17bd2907939eaa8d3310378d419d90c5#diff-b525f86b1f4925509959f857496dff18a9c4ed34fcc7f49357c5a6d00fb64d17R91
Tim, do you know how to add it using ${{ contains(matrix.FLAGS, 'USE_OT=1')
}} condition ?


You can simply add these to the 'flags' list within matrix.py, similarly to WURFL_INC and WURFL_LIB also being part of that list.

Best regards
Tim Düsterhus

Reply via email to