Subject: Re: CVS commit: src/crypto/dist/openssl
To: Perry E. Metzger <perry@piermont.com>
From: None <itojun@iijlab.net>
List: tech-userlevel
Date: 07/15/2003 18:24:54
>> >   Log Message:
>> >   unifdef VMS/WIN16/WIN32 in public headers, at least
>> >is this actually necessary?  "dist" sources should not be changed
>> >to be less portable for no reason...  the only real change i can
>> >see this makes is making it harder for future openssl updates due
>> >to unnecessary conflicts....
>> 	header files shouldn't have #ifdef which easily conflict with what user
>> 	would define by chance.  there's no portability issue, we won't and
>> 	shouldn't use the removed prototypes (and whatever) anyways.
>I agree with Matt that the dist headers should remain the
>same. However, you are right that /usr/include header files should be
>clean. Perhaps unifdef could be called on the untouched files at build
>time, so that clean files are installed even though the originals
>remain untouched?

	i fear possible inconsistencies while building libcrypto (with all
	#ifdef VMS intact) and installing include files (with all #ifdef VMS
	removed) with the above approach.

	another problem is that unifdef(1) does not support "#if defined(foo)"
	form.  it isunderstandable as we can put very complex expression
	using this form, and partially evaluating them would be difficult
	sometimes.

itojun