[[EMAIL PROTECTED] - Mon Jun 17 09:54:04 2002]:

> The snapshot doesn't work under DJGPP, but I think that this patch
> will fix it. There were several problems. In Configure, you tried 
to
> use ENV{DJDIR}, but this puts in a DOS style path such as 

That was a mistake, I only changed one of the instances.  Thanks for 
the notification.

> developed. "make depend" gave quite a few warnings, since the 
include
> directives from CFLAG were not utilized.

I've just added $(CFLAG) after every $(MAKEDEPEND).  It's not the 
same change as you propose, but it feels cleaner, since that's how 
we handle the other passed variables.  I hope that isn't a problem.

> I think I finally came up with an acceptable method of putting the
> header and test files for excluded algoritms in "include/openssl" 
and
> "test" respectively. If this is accepted, there should be no need 
to
> run "make depend" after excluding algorithms. I didn't put it in 
the
> patch, but I would consider removing the notice to run "make 
depend",
> which was just added.

I still don't accept this change, and I've figured out why (I had 
forgotten for a while).  There are some paranoid people who not only 
disable the algorithms they don't want, but remove the source for 
them entirely!  So far, we have supported that, and your change 
would made that support much harder.

I suggest a different approach:

- before running make links, remove all files in include/openssl
- pass a variable to make in test saying exactly which algorithms it 
should build tests for, so it doesn't try to build tests that aren't 
there.

Does that sound like an option to you?

(as a side-note, I can recommend taking some vacation when your 
thinking gets stuck.  I did, and GOD is my brain more awake now!)

> I also found a problem I had previously overlooked in
> crypto/engine/hw_aep.c. Which MSDOS compiler needs the section that
> doesn't apply to DJGPP?

I think it's basically all other Windows compilers...

-- 
Richard Levitte
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to