+1 to #1, #2 and #4.
For #3, why not call it ts_api.h. api.h is too generic and ts_api would be
consistent with remap_api.
Thanks.
________________________________
From: Leif Hedstrom <zw...@apache.org>
To: trafficserver-dev@incubator.apache.org
Sent: Tue, February 2, 2010 7:37:35 AM
Subject: [vote] API include files
Hi,
we started this discussion a long time ago, but I'd like to propose a name
change for the two include files. Please comment / vote on these four
suggestions:
1. During installation, we'll place the two include files in
${prefix}/include/ts
2. The RemapAPI.h file gets renamed to remap_api.h
3. The InkAPI.h file gets renamed to just api.h
4. In the source tree, we move remap_api.h to proxy/api/include
(which is where InkAPI.h / api.h is).
I'd like to get this done for the 2.0.0a release, so please comment (and vote).
I know this breaks "internal" plugin builds for Yahoo, but we can easily work
around that if necessary (just make symlinks in our internal package file).
The changes above also means we need to modify documentation, and examples, of
course (i.e. #include <ts/api.h> ). Any future/additional "dev" include files
should obviously also go into ${prefix}/include/ts/ .
Thanks,
-- Leif
