Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: neutral (hermes.apache.org: local policy)
Message-ID: <00c701c50d6b$6a522fc0$e6ec6f83@robinson.cam.ac.uk>
From: "Max Bowsher" <maxb@ukf.net>
To: <dev@apr.apache.org>
References: <20050207224412.3014.qmail@minotaur.apache.org>
 <993D604F73642A745DC4107C@dhcp20-a.ics.uci.edu>
Subject: Re: svn commit: r151766 - apr/apr/trunk/include/apr_general.h
 apr/apr/trunk/include/apr_release.h apr/apr/trunk/include/apr_version.h
Date: Mon, 7 Feb 2005 23:19:00 -0000
MIME-Version: 1.0
Content-Type: text/plain;
	format=flowed;
	charset="iso-8859-1";
	reply-type=response
Content-Transfer-Encoding: 7bit

Justin Erenkrantz wrote:
> --On Monday, February 7, 2005 10:44 PM +0000 wrowe@apache.org wrote:
>
>> Author: wrowe
>> Date: Mon Feb  7 14:44:09 2005
>> New Revision: 151766
>>
>> URL: http://svn.apache.org/viewcvs?view=rev&rev=151766
>> Log:
>>
>>   In order to remove the win32ver.awk generation of .rc files for apr,
>>   it's necessary to have a 'vanilla' c preprocessor file which doesn't
>>   have all the extra apr.h or other function declaration cruft.
>>
>>   ap_release.h is the httpd project's convention, and it seems to fit
>>   the bill.  Provide "big red warnings" (tm) to not use this new file
>>   directly, if the user wants to retain compatibility with APR 0.x/1.x.
>
> I don't get it.  What exactly are you trying to do?  Why must you add a
> header file that is exactly the same as what was in apr_version.h?  You
> also just broke the Unix build which expects the version to be in
> apr_version.h.
>
> I really dislike the ap_release.h strategy used by httpd.  I also don't
> think it makes sense for APR in any form.  Please discuss this first on
> dev@apr so we can minimize breakage.  -- justin

To suggest an alternative:

Put everything back where it was, and wrap the bits you don't want in:

#ifndef RC_INVOKED
...
#endif

(Or another symbol, if my guess that you want to directly #include it from 
the resource compiler is wrong).

Max.