On 01/04, Thomas Gummerer wrote: > On 12/18, Lars Schneider wrote: > > [snip] > > For now I think that looks good. Maybe we could define additional test > > configurations with an environment variable. That could be an array variable > > defined in the lib-travis.ci "case" statement: > > https://github.com/git/git/blob/1229713f78cd2883798e95f33c19c81b523413fd/ci/lib-travisci.sh#L42-L65 > > That sounds like a good idea. I'll try to see if I can come up with > something.
On second thought I'd prefer to just leave it as is for now, and leave defining additional test configurations for a future iteration. Having it configurable makes it a bit uglier, and I'm not even sure how many configurations we can just test at runtime vs. having to compile with the flag set. So I'd like to punt on that for now, and introduce more configurability once we actually need it :) > > - Lars
