I'd like to voice a comment,
${prefix}/ts seems somewhat vague unless you were going to rename all the
scripts ts.
My eyes can read quite clear exactly what this header is even if i had no
idea what trafficserver was and just happened to be reading some random
piece of code:
"#include <trafficserver/sdkapi.h>"
Just some thoughts,
- Jason
On Wed, Feb 3, 2010 at 11:08 PM, Leif Hedstrom <zw...@apache.org> wrote:
> So, I think the discussions have had enough time, the consensus is
> definitely that we should clean up what we got. As such, we will move both
> include files to be in the same directory in SVN, we will rename RemapAPI.h
> to remap_api.h, and we will install the two header files in a well defined
> include directory. Two things to vote on remains (please cast votes within
> 72 hours):
>
>
> InkApi.h rename:
> ------------------------
> [ ] api.h
> [ ] ts_api.h
> [ ] sdkapi.h
>
>
> Include directory name
> ---------------------------------
>
> [ ] ${prefix}/ts
> [ ] ${prefix}/trafficserver
>
>
>
> Include directives would hence look like the examples below (depending on
> the outcome of the votes):
>
> #include <ts/remap_api.h>
> #include <ts/api.h> -- or --
> #include <ts/ts_api.h> -- or --
> #include <ts/sdkapi.h>
>
> -- or --
>
> #include <trafficserver/remap_api.h>
> #include <trafficserver/api.h> -- or --
> #include <trafficserver/ts_api.h> -- or --
> #include <trafficserver/sdkapi.h>
>
>
> Cheers!
>
> -- Leif
>
>
