Open MPI logo

Hardware Locality Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |  

This web mail archive is frozen.

This page is part of a frozen web archive of this mailing list.

You can still navigate around this archive, but know that no new mails have been added to it since July of 2016.

Click here to be taken to the new web archives of this list; it includes all the mails that are in this frozen archive plus all new mails that have been sent to the list since it was migrated to the new archives.

Subject: Re: [hwloc-devel] embedding m4 code
From: Brice Goglin (Brice.Goglin_at_[hidden])
Date: 2009-12-05 07:55:27

Jeff Squyres wrote:
> I think I have the first part of the embedding code done -- it builds and compiles hwloc just like today's build system does (but a bunch of the m4 behind the scenes has moved around quite a bit to enable the embedding stuff). :-)
> Could you guys try builds on the supported platforms to see if I broke anything? I only have access to rhel4, rhel5, and osx. The code is in a mercurial branch here:
> I'm working on the actual embedding stuff now, but wanted to ensure that I at least got the first parts right.
> Thanks!

I didn't test it but I read the diff, here are some comments/questions:

1) I don't understand this but I hope you know what you're doing since I
seem to remember that it was related to the failure to build with
autoconf 2.63.

-ACLOCAL_AMFLAGS = -I config -I m4
+# Note that the -I directory must *exactly* match what was specified
+ACLOCAL_AMFLAGS = -I ./config

2) Is there a s/disable/enable/ needed in this comment?

+# Disable building the executables

3) I am surprised to see HWLOC_SYM_PREFIX{,_CAPS}. I seem to remember
you saying it was a mistake in PLPA and you wouldn't do the same in hwloc.

4) There are some configure checks that might need to remain in the main
configure since they may only be used by tools or tests (which embedded
should not build, right?):
* cairo (tools only)
* wchar/libtermcap/curses (maybe tools only)
* libibverbs (tests only)
* libnuma (tests only)
* bash? (Linux test only)
* xmllint? (xml test only)
* HWLOC_CHECK_DIFF_U (tests only)

5) There are a bunch of OMPI that should be replaced with hwloc in the
comments :)