Re: [vote] API include files and dirs reorg

2010-02-12 Thread Bryan Call
On Feb 9, 2010, at 6:37 AM, Leif Hedstrom wrote: > Hi all, > > please vote on the following proposals as we have discussed, normal 72h > window. I added an additional option to the install directory (just plain > ".../include), to better match the suggestions (#4) for versioned > directories

Re: [vote] API include files and dirs reorg

2010-02-11 Thread Leif Hedstrom
On 02/11/2010 04:23 PM, John Plevyak wrote: I modified 2 to permit: #include Well, with this bad boy on the table (I thought y'all didn't like short names:), I'll change my vote on #2 to [x] ts.h I'll leave my other votes as is. I have a gut feeling we'll only get a general "consensu

Re: [vote] API include files and dirs reorg

2010-02-11 Thread John Plevyak
I modified 2 to permit: #include This matches a large number of projects which make the main include file name match the directory. john > 1. Include install directory > > [ ] ${prefix}/include > [x] ${prefix}/include/ts > [ ] ${prefix}/include/ats > [ ] ${prefix}/include/trafficserver > >

Re: [vote] API include files and dirs reorg

2010-02-11 Thread Paul Querna
On Thu, Feb 11, 2010 at 12:17 PM, Leif Hedstrom wrote: > I'd like to amend to item #4, as per Paul's suggestion. If you already > voted, and would like to change your vote on #4, please do so. I'll extend > this vote until Tuesday next week. > > 4. Versioning of include dir > > [ ] No versioning,

Re: [vote] API include files and dirs reorg

2010-02-11 Thread Leif Hedstrom
I'd like to amend to item #4, as per Paul's suggestion. If you already voted, and would like to change your vote on #4, please do so. I'll extend this vote until Tuesday next week. 4. Versioning of include dir [ ] No versioning, just plain ${prefix}/include/${name} [ ] Versioning of the includ

Re: [vote] API include files and dirs reorg

2010-02-09 Thread Bryan Call
This is the only real project I could find: XDAQ: http://sourceforge.net/projects/xdaq/ -Bryan Before I vote, Bryan could you enlighten us as to the other project(s) which use ts as an include prefix so that we can evaluate the likelihood of a conflict? Thanx, john On 2/9/2010 6:40 AM, Lei

Re: [vote] API include files and dirs reorg

2010-02-09 Thread John Plevyak
Before I vote, Bryan could you enlighten us as to the other project(s) which use ts as an include prefix so that we can evaluate the likelihood of a conflict? Thanx, john On 2/9/2010 6:40 AM, Leif Hedstrom wrote: > On 02/09/2010 07:37 AM, Leif Hedstrom wrote: >> >> 1. Include install directory

Re: [vote] API include files and dirs reorg

2010-02-09 Thread George Paul
1. Include install directory [X] ${prefix}/include/ats 2. InkAPI.h rename [X] ats.h 3. RemapAPI.h renam [X] remap.h 4. Versioning of include dir [X] Install into a versioned directory like ${prefix}/include/${name}/${version}, and wrappers into ${prefix}/include/${name}. 5. SimpleDBM includ

Re: [vote] API include files and dirs reorg

2010-02-09 Thread Leif Hedstrom
On 02/09/2010 07:37 AM, Leif Hedstrom wrote: 1. Include install directory [ ] ${prefix}/include [x] ${prefix}/include/ts [ ] ${prefix}/include/ats [ ] ${prefix}/include/trafficserver 2. InkAPI.h rename [x] ts_api.h [ ] sdkapi.h [ ] ats.h 3. RemapAPI.h renam [x] remap_api.h [ ] remap.h 4

[vote] API include files and dirs reorg

2010-02-09 Thread Leif Hedstrom
Hi all, please vote on the following proposals as we have discussed, normal 72h window. I added an additional option to the install directory (just plain ".../include), to better match the suggestions (#4) for versioned directories. Cheers, -- Leif 1. Include install directory [ ] ${pref

Re: [vote] API include files

2010-02-02 Thread George Paul
+1 for 1,2 & 4. +1 for 3 if all includes use since 'api.h' is really generic as pointed out. Another possibility is 'sdkapi.h' since it is the TS SDK API. -George On 2/2/10 7:37 AM, Leif Hedstrom wrote: > Hi, > > we started this discussion a long time ago, but I'd like to propose a > name chang

Re: [vote] API include files

2010-02-02 Thread Leif Hedstrom
On 02/02/2010 03:42 PM, Tin Zaw wrote: +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. I was thinking about that too, but all references would be as #include or possibly #include As for vs , it seem

Re: [vote] API include files

2010-02-02 Thread Bryan Call
We should standardize on a naming convention for traffic server directories. Right now we are using "trafficserver". This has less possibility of conflicts then "ts". I vote to put the include files into ${prefix}/include/trafficserver -Bryan On 02/02/2010 07:37 AM, Leif Hedstrom wrote: Hi

Re: [vote] API include files

2010-02-02 Thread Tin Zaw
+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 To: trafficserver-dev@incubator.apache.org Sent: Tue, February 2, 2010 7:37:35 AM Subject: [vote] API

Re: [vote] API include files

2010-02-02 Thread Paul Querna
On Tue, Feb 2, 2010 at 7:37 AM, Leif Hedstrom wrote: > 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 >     $

[vote] API include files

2010-02-02 Thread Leif Hedstrom
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 r