Re: Verify AES support when Blumenthal draft is enabled

Robert Story <[email protected]>
Newsgroups gmane.network.net-snmp.devel
Message-ID <[email protected]>
On Wed, 25 Apr 2018 10:28:59 -0600 Bart wrote:
BVA> On 04/25/18 10:04, Keith Mendoza wrote:
BVA> > I have submitted a merge request to verify that when the
BVA> > --enable-blumenthal-aes is used in configure that it checks
BVA> > that OpenSSL's aes.h and evp.h are available. Merge request
BVA> > is at
BVA> > https://sourceforge.net/p/net-snmp/code/merge-requests/14/.
BVA> > [...]
BVA> 
BVA> Hello Keith,
BVA> 
BVA> Are you aware that running something like "brew upgrade
BVA> openssl" brings in a version of openssl on OS/X that is recent
BVA> enough for all Net-SNMP features?

Regardless, configure should be doing the right thing based on what
is currently installed.

BVA> Regarding your pull request:
BVA> I'd like to avoid adding AC_CHECK_HEADERS() calls in
BVA> config_project_with_enable because whether or not these
BVA> succeed depend on the compiler flags (-I) and some compiler
BVA> flags are only set at a later phase.

I agree that header checks inside a feature check is undesirable.
Keith, do you think you could come up with a patch that re-arranges
configure checks that that the desired effect is achieved?

Robert

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.