Ralph,
*IF* I have new enough autotools to autogen on my FreeBSD platform, I'll
take a shot tonight from the svn trunk. Otherwise I'll need a tarball to
point my testing script at.
-Paul
On Wed, Jan 8, 2014 at 8:10 PM, Ralph Castain <r...@open-mpi.org> wrote:
> Actually, as I look at it, the logic escapes me anyway. Basically, you
> only have two options - use the vfs struct for Sun, and use fs struct for
> everything else. I'm not aware of any other choice, and indeed the list of
> all the systems for the latter actually is intended to amount to "anything
> else".
>
> So I just changed it to an "else" statement in the trunk and scheduled it
> for 1.7.4 if it passes muster - see how this works for you.
>
> Ralph
>
>
> On Jan 8, 2014, at 4:52 PM, Paul Hargrove <phhargr...@lbl.gov> wrote:
>
> This partial make output shows a build failure of the current trunk
> tarball on FreeBSD-9/x86-64:
> CC path.lo
> /home/phargrov/OMPI/openmpi-trunk-freebsd9-amd64/openmpi-1.9a1r30146/opal/util/path.c:
> In function 'opal_path_df':
> /home/phargrov/OMPI/openmpi-trunk-freebsd9-amd64/openmpi-1.9a1r30146/opal/util/path.c:556:
> error: 'buf' undeclared (first use in this function)
> /home/phargrov/OMPI/openmpi-trunk-freebsd9-amd64/openmpi-1.9a1r30146/opal/util/path.c:556:
> error: (Each undeclared identifier is reported only once
> /home/phargrov/OMPI/openmpi-trunk-freebsd9-amd64/openmpi-1.9a1r30146/opal/util/path.c:556:
> error: for each function it appears in.)
> *** [path.lo] Error code 1
>
> Builds of the trunk on NetBSD and OpenBSD fail in the same way.
>
> The code in opal_path_df() seems not to define the variable "buf" unless
> certain defines are present, but then tries to USE the variable
> unconditionally. That unconditional use of "buf" is the proximate cause of
> the failure.
>
> A very-likely-related observation:
> The code in opal/util/path.c seems to expect __BSD to be defined under
> *some* circumstance, but it is not defined by any current
> {Free,Net,Open}BSD:
>
> FreeBSD-9:
> $ touch empty.c
> $ gcc -dM -E empty.c | grep BSD
> #define __FreeBSD_cc_version 900001
> #define __VERSION__ "4.2.1 20070831 patched [FreeBSD]"
> #define __FreeBSD__ 9
>
> NetBSD-6:
> -bash-4.2$ touch empty.c
> -bash-4.2$ gcc -dM -E empty.c | grep BSD
> #define __NetBSD__ 1
>
> OpenBSD-5:
> -bash-4.2$ touch empty.c
> -bash-4.2$ gcc -dM -E empty.c | grep BSD
> #define __OpenBSD__ 1
>
> So, there are TWO issues:
>
> 1) opal_path_df() defines the variable "buf" only conditionally but then
> uses it UNconditionally.
>
> 2) logic which appears to be intended for at least FreeBSD is probably not
> reachable in any build.
>
> -Paul
>
> --
> Paul H. Hargrove phhargr...@lbl.gov
> Future Technologies Group
> Computer and Data Sciences Department Tel: +1-510-495-2352
> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>
>
>
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>
--
Paul H. Hargrove phhargr...@lbl.gov
Future Technologies Group
Computer and Data Sciences Department Tel: +1-510-495-2352
Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
