Skip to content

Remove stale solaris configure references #13206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 29, 2025
+69 −170
Merged

Remove stale solaris configure references #13206

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6dae8ab
Remove stale solaris configure references
rhc54 Apr 22, 2025
f1201ab
Preserve the "hreads" comment, but make it clearer that
rhc54 Apr 22, 2025
f744b78
opal/mca/if: fix "struct sockaddr_in" and OS checks
jsquyres Apr 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 1 addition & 6 deletions autogen.pl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
# Copyright (c) 2020 Amazon.com, Inc. or its affiliates.
# All Rights reserved.
#
# Copyright (c) 2025 Nanook Consulting All rights reserved.
# $COPYRIGHT$
#
# Additional copyrights may follow
Expand Down Expand Up @@ -81,12 +82,6 @@

# Patch program
my $patch_prog = "patch";
# Solaris "patch" doesn't understand unified diffs, and will cause
# autogen.pl to hang with a "File to patch:" prompt. Default to Linux
# "patch", but use "gpatch" on Solaris.
if ($^O eq "solaris") {
$patch_prog = "gpatch";
}

$username = $ENV{USER} || getpwuid($>);
$full_hostname = $ENV{HOSTNAME} || `hostname`;
Expand Down
17 changes: 1 addition & 16 deletions config/ompi_setup_java.m4
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ dnl Copyright (c) 2013 Intel, Inc. All rights reserved.
dnl Copyright (c) 2015-2018 Research Organization for Information Science
dnl and Technology (RIST). All rights reserved.
dnl Copyright (c) 2017 FUJITSU LIMITED. All rights reserved.
dnl Copyright (c) 2025 Nanook Consulting All rights reserved.
dnl $COPYRIGHT$
dnl
dnl Additional copyrights may follow
Expand Down Expand Up @@ -128,17 +129,6 @@ AC_DEFUN([_OMPI_SETUP_JAVA],[
[AC_MSG_RESULT([not found])])])
fi

if test "$ompi_java_found" = "0"; then
# Solaris
ompi_java_dir=/usr/java
AC_MSG_CHECKING([for Java in Solaris locations])
AS_IF([test -d $ompi_java_dir && test -r "$ompi_java_dir/include/jni.h"],
[AC_MSG_RESULT([found ($ompi_java_dir)])
with_jdk_headers=$ompi_java_dir/include
with_jdk_bindir=$ompi_java_dir/bin
ompi_java_found=1],
[AC_MSG_RESULT([not found])])
fi
],
[ompi_java_found=1])

Expand Down Expand Up @@ -188,11 +178,6 @@ EOF
# too. Ugh.
AS_IF([test -d "$with_jdk_headers/linux"],
[OMPI_JDK_CPPFLAGS="$OMPI_JDK_CPPFLAGS -I$with_jdk_headers/linux"])
# Solaris JDK also require -I<blah>/solaris.
# See if that's there, and if so, add a -I for that,
# too. Ugh.
AS_IF([test -d "$with_jdk_headers/solaris"],
[OMPI_JDK_CPPFLAGS="$OMPI_JDK_CPPFLAGS -I$with_jdk_headers/solaris"])
# Darwin JDK also require -I<blah>/darwin.
# See if that's there, and if so, add a -I for that,
# too. Ugh.
Expand Down
69 changes: 0 additions & 69 deletions config/opal_check_os_flavors.m4
View file
Open in desktop

This file was deleted.

3 changes: 2 additions & 1 deletion configure.ac
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
# Copyright (c) 2019 Triad National Security, LLC. All rights
# reserved.
# Copyright (c) 2023 Jeffrey M. Squyres. All rights reserved.
# Copyright (c) 2025 Nanook Consulting All rights reserved.
# $COPYRIGHT$
#
# Additional copyrights may follow
Expand Down Expand Up @@ -995,7 +996,7 @@ AC_CACHE_SAVE
opal_show_title "System-specific tests"

##################################
OPAL_CHECK_OS_FLAVORS
OAC_CHECK_OS_FLAVORS



Expand Down
25 changes: 14 additions & 11 deletions opal/mca/if/bsdx_ipv4/configure.m4
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
# Copyright (c) 2010 Cisco Systems, Inc. All rights reserved.
# Copyright (c) 2015 Research Organization for Information Science
# and Technology (RIST). All rights reserved.
# Copyright (c) 2025 Nanook Consulting All rights reserved.
# Copyright (c) 2025 Jeffrey M. Squyres. All rights reserved.
# $COPYRIGHT$
#
# Additional copyrights may follow
Expand All @@ -22,25 +24,26 @@ AC_DEFUN([MCA_opal_if_bsdx_ipv4_COMPILE_MODE], [
AC_DEFUN([MCA_opal_if_bsdx_ipv4_CONFIG], [
AC_CONFIG_FILES([opal/mca/if/bsdx_ipv4/Makefile])

AC_REQUIRE([OPAL_CHECK_OS_FLAVORS])
AC_REQUIRE([OAC_CHECK_OS_FLAVORS])

# If we found struct sockaddr and we're on any of the BSDs, we're
# If we previously found struct sockaddr_in (we don't repeat the
# AC CHECK_TYPES test here simply because it's cumbersome with all
# the required #includes) and we're on any of the BSDs, we're
# happy. I.e., this:
#if defined(__NetBSD__) || defined(__FreeBSD__) || \
# defined(__OpenBSD__) || defined(__DragonFly__)
AC_MSG_CHECKING([struct sockaddr])
AS_IF([test "$opal_found_sockaddr" = "yes"],
[AC_MSG_RESULT([yes (cached)])
AC_MSG_CHECKING([struct sockaddr_in again])
AS_IF([test "$ac_cv_type_struct_sockaddr_in" = "yes"],
[AC_MSG_RESULT([yes (OPAL cached)])
AC_MSG_CHECKING([NetBSD, FreeBSD, OpenBSD, or DragonFly])
AS_IF([test "$opal_found_netbsd" = "yes" || \
test "$opal_found_freebsd" = "yes" || \
test "$opal_found_openbsd" = "yes" || \
test "$opal_found_dragonfly" = "yes"],
AS_IF([test "$oac_found_netbsd" = "yes" || \
test "$oac_found_freebsd" = "yes" || \
test "$oac_found_openbsd" = "yes" || \
test "$oac_found_dragonfly" = "yes"],
[AC_MSG_RESULT([yes])
$1],
[AC_MSG_RESULT([no])
$2])],
[AC_MSG_RESULT([no (cached)])
[AC_MSG_RESULT([no (OPAL cached)])
$2])
])

28 changes: 16 additions & 12 deletions opal/mca/if/bsdx_ipv6/configure.m4
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
# Copyright (c) 2010 Cisco Systems, Inc. All rights reserved.
# Copyright (c) 2015 Research Organization for Information Science
# and Technology (RIST). All rights reserved.
# Copyright (c) 2025 Nanook Consulting All rights reserved.
# Copyright (c) 2025 Jeffrey M. Squyres. All rights reserved.
# $COPYRIGHT$
#
# Additional copyrights may follow
Expand All @@ -22,26 +24,28 @@ AC_DEFUN([MCA_opal_if_bsdx_ipv6_COMPILE_MODE], [
AC_DEFUN([MCA_opal_if_bsdx_ipv6_CONFIG], [
AC_CONFIG_FILES([opal/mca/if/bsdx_ipv6/Makefile])

AC_REQUIRE([OPAL_CHECK_OS_FLAVORS])
AC_REQUIRE([OAC_CHECK_OS_FLAVORS])

# If we found struct sockaddr and we're on any of the BSDs, we're
# If we previously found struct sockaddr_in6 (we don't repeat the
# AC CHECK_TYPES test here simply because it's cumbersome with all
# the required #includes) and we're on any of the BSDs, we're
# happy. I.e., this:
#if defined( __NetBSD__) || defined(__OpenBSD__) || defined(__FreeBSD__) || \
# defined(__386BSD__) || defined(__bsdi__) || defined(__APPLE__)
AC_MSG_CHECKING([struct sockaddr])
AS_IF([test "$opal_found_sockaddr" = "yes"],
[AC_MSG_RESULT([yes (cached)])
AC_MSG_CHECKING([struct sockaddr_in6 again])
AS_IF([test "$ac_cv_type_struct_sockaddr_in6" = "yes"],
[AC_MSG_RESULT([yes (OPAL cached)])
AC_MSG_CHECKING([some flavor of BSD])
AS_IF([test "$opal_found_netbsd" = "yes" || \
test "$opal_found_freebsd" = "yes" || \
test "$opal_found_openbsd" = "yes" || \
test "$opal_found_386bsd" = "yes" || \
test "$opal_found_bsdi" = "yes" ||
test "$opal_found_apple" = "yes"],
AS_IF([test "$oac_found_netbsd" = "yes" || \
test "$oac_found_freebsd" = "yes" || \
test "$oac_found_openbsd" = "yes" || \
test "$oac_found_386bsd" = "yes" || \
test "$oac_found_bsdi" = "yes" ||
test "$oac_found_apple" = "yes"],
[AC_MSG_RESULT([yes])
$1],
[AC_MSG_RESULT([no])
$2])],
[AC_MSG_RESULT([no (cached)])
[AC_MSG_RESULT([no (OPAL cached)])
$2])
])dnl
13 changes: 8 additions & 5 deletions opal/mca/if/linux_ipv6/configure.m4
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
# Copyright (c) 2010 Cisco Systems, Inc. All rights reserved.
# Copyright (c) 2015 Research Organization for Information Science
# and Technology (RIST). All rights reserved.
# Copyright (c) 2025 Nanook Consulting All rights reserved.
# Copyright (c) 2025 Jeffrey M. Squyres. All rights reserved.
# $COPYRIGHT$
#
# Additional copyrights may follow
Expand All @@ -22,12 +24,13 @@ AC_DEFUN([MCA_opal_if_linux_ipv6_COMPILE_MODE], [
AC_DEFUN([MCA_opal_if_linux_ipv6_CONFIG], [
AC_CONFIG_FILES([opal/mca/if/linux_ipv6/Makefile])

AC_REQUIRE([OPAL_CHECK_OS_FLAVORS])
AC_REQUIRE([OAC_CHECK_OS_FLAVORS])

AC_MSG_CHECKING([if we are on Linux with TCP])
# If we have struct sockaddr and we're on Linux, then we're
# happy.
AS_IF([test "$opal_found_sockaddr" = "yes" && test "$opal_found_linux" = "yes"],
# If we previously found struct sockaddr_in6 (we don't repeat the
# AC CHECK_TYPES test here simply because it's cumbersome with all
# the required #includes) and we're on Linux, we're happy.
AC_MSG_CHECKING([for Linux with struct sockaddr_in6])
AS_IF([test "$ac_cv_type_struct_sockaddr_in6" = "yes" && test "$oac_found_linux" = "yes"],
[AC_MSG_RESULT([yes])
$1],
[AC_MSG_RESULT([no])
Expand Down
20 changes: 12 additions & 8 deletions opal/mca/if/posix_ipv4/configure.m4
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
# Copyright (c) 2010 Cisco Systems, Inc. All rights reserved.
# Copyright (c) 2015 Research Organization for Information Science
# and Technology (RIST). All rights reserved.
# Copyright (c) 2025 Nanook Consulting All rights reserved.
# Copyright (c) 2025 Jeffrey M. Squyres. All rights reserved.
# $COPYRIGHT$
#
# Additional copyrights may follow
Expand All @@ -25,22 +27,24 @@ AC_DEFUN([MCA_opal_if_posix_ipv4_CONFIG], [
OPAL_VAR_SCOPE_PUSH([opal_if_posix_ipv4_happy])
opal_if_posix_ipv4_happy=no

AC_REQUIRE([OPAL_CHECK_OS_FLAVORS])
AC_REQUIRE([OAC_CHECK_OS_FLAVORS])

# If we found struct sockaddr and we're NOT on most of the BSDs,
# we're happy. I.e., if posix but not:
# If we previously found struct sockaddr_in (we don't repeat the
# AC CHECK_TYPES test here simply because it's cumbersome with all
# the required #includes) and we're NOT on most of the BSDs, we're
# happy. I.e., this:
#if defined(__NetBSD__) || defined(__FreeBSD__) || \
# defined(__OpenBSD__) || defined(__DragonFly__)
AC_MSG_CHECKING([struct sockaddr])
AS_IF([test "$opal_found_sockaddr" = "yes"],
[AC_MSG_RESULT([yes (cached)])
AC_MSG_CHECKING([struct sockaddr_in again])
AS_IF([test "$ac_cv_type_struct_sockaddr_in" = "yes"],
[AC_MSG_RESULT([yes (OPAL cached)])
AC_MSG_CHECKING([not NetBSD, FreeBSD, OpenBSD, or DragonFly])
AS_IF([test "$opal_found_netbsd" = "no" && test "$opal_found_freebsd" = "no" && test "$opal_found_openbsd" = "no" && test "$opal_found_dragonfly" = "no"],
AS_IF([test "$oac_found_netbsd" = "no" && test "$oac_found_freebsd" = "no" && test "$oac_found_openbsd" = "no" && test "$oac_found_dragonfly" = "no"],
[AC_MSG_RESULT([yes])
opal_if_posix_ipv4_happy=yes],
[AC_MSG_RESULT([no])]
)],
[AC_MSG_RESULT([no (cached)])]
[AC_MSG_RESULT([no (OPAL cached)])]
)

AS_IF([test "$opal_if_posix_ipv4_happy" = "yes"],
Expand Down
11 changes: 3 additions & 8 deletions opal/mca/shmem/posix/shmem_posix_common_utils.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
* reserved.
* Copyright (c) 2022 IBM Corporation. All rights reserved
*
* Copyright (c) 2025 Nanook Consulting All rights reserved.
* $COPYRIGHT$
*
* Additional copyrights may follow
Expand All @@ -33,15 +34,9 @@
# include <fcntl.h>
#endif /* HAVE_FCNTL_H */
#include <string.h>
#if OPAL_HAVE_SOLARIS && !defined(_POSIX_C_SOURCE)
# define _POSIX_C_SOURCE 200112L /* Required for shm_{open,unlink} decls */
#ifdef HAVE_SYS_MMAN_H
# include <sys/mman.h>
# undef _POSIX_C_SOURCE
#else
# ifdef HAVE_SYS_MMAN_H
# include <sys/mman.h>
# endif /* HAVE_SYS_MMAN_H */
#endif
#endif /* HAVE_SYS_MMAN_H */
#ifdef HAVE_UNISTD_H
# include <unistd.h>
#endif /* HAVE_UNISTD_H */
Expand Down
11 changes: 3 additions & 8 deletions opal/mca/shmem/posix/shmem_posix_module.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
* Copyright (c) 2019 Triad National Security, LLC. All rights
* reserved.
* Copyright (c) 2022 IBM Corporation. All rights reserved
* Copyright (c) 2025 Nanook Consulting All rights reserved.
* $COPYRIGHT$
*
* Additional copyrights may follow
Expand All @@ -30,15 +31,9 @@
#ifdef HAVE_FCNTL_H
# include <fcntl.h>
#endif /* HAVE_FCNTL_H */
#if OPAL_HAVE_SOLARIS && !defined(_POSIX_C_SOURCE)
# define _POSIX_C_SOURCE 200112L /* Required for shm_{open,unlink} decls */
#ifdef HAVE_SYS_MMAN_H
# include <sys/mman.h>
# undef _POSIX_C_SOURCE
#else
# ifdef HAVE_SYS_MMAN_H
# include <sys/mman.h>
# endif /* HAVE_SYS_MMAN_H */
#endif
#endif /* HAVE_SYS_MMAN_H */
#ifdef HAVE_UNISTD_H
# include <unistd.h>
#endif /* HAVE_UNISTD_H */
Expand Down
Loading