I am doing this already for our internal release and build for ATS 8.0.1. I don’t know if this helps or not:
https://github.com/bryancall/trafficserver/commit/d4d4bcd33cf60e5d936107561de5d31389c783dc -Bryan > On Jan 4, 2019, at 5:55 PM, Pushkar Pradhan <pprad...@oath.com.INVALID> wrote: > > It seems the API TSHttpTxnSetHttpRetStatus in 7.x was changed to > TSHttpTxnStatusSet in 8.x. > The underlying code seems to be the same, only the name has changed. > > I am writing a plugin that calls this function and that I want to work in > both 7.x and 8.x codelines. > I can wrap the names in a #iifdef like this: > > #ifdef TS_VERSION_MAJOR > 7 > TSHttpTxnStatusSet > #else > TSHttpTxnSetHttpRetStatus > #endif > > Would this be considered bad practice, any potential pitfalls to be aware > of? > Thanks, > -- > pushkar
