On Tue, Feb 16, 2016 at 01:28:42AM -0500, Jeff Wieland wrote:> The Solaris privilege code breaks building on Solaris 10. If > you let configure just do its thing, you get the following error > when compiling: > > "sandbox-solaris.c", line 22: #error: "--with-solaris-privs must be used > with the Solaris sandbox"Could you please try this patch? It adds priv_basicset to the test for whether or not we have usable solaris privs support. Note that you will need to run "autoreconf" after applying the patch to rebuild configure. Thanks. diff --git a/configure.ac b/configure.ac index b4c0aaa..5b50b9e 100644 --- a/configure.ac +++ b/configure.ac @@ -897,8 +897,10 @@ mips-sony-bsd|mips-sony-newsos4) AC_MSG_RESULT([no]) fi AC_CHECK_FUNC([setppriv], - [ AC_CHECK_HEADERS([priv.h], [ - SOLARIS_PRIVS="yes" + AC_CHECK_FUNC([priv_basicset], + [ AC_CHECK_HEADERS([priv.h], [ + SOLARIS_PRIVS="yes" + ]) ]) ]) AC_ARG_WITH([solaris-contracts], -- Darren Tucker (dtucker at zip.com.au) GPG key 8FF4FA69 / D9A3 86E9 7EEE AF4B B2D4 37C9 C982 80C7 8FF4 FA69 Good judgement comes with experience. Unfortunately, the experience usually comes from bad judgement.
Darren Tucker wrote:> On Tue, Feb 16, 2016 at 01:28:42AM -0500, Jeff Wieland wrote: >> The Solaris privilege code breaks building on Solaris 10. If >> you let configure just do its thing, you get the following error >> when compiling: >> >> "sandbox-solaris.c", line 22: #error: "--with-solaris-privs must be used >> with the Solaris sandbox" > Could you please try this patch? It adds priv_basicset to the test for > whether or not we have usable solaris privs support. Note that you will > need to run "autoreconf" after applying the patch to rebuild configure. > > Thanks. > > diff --git a/configure.ac b/configure.ac > index b4c0aaa..5b50b9e 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -897,8 +897,10 @@ mips-sony-bsd|mips-sony-newsos4) > AC_MSG_RESULT([no]) > fi > AC_CHECK_FUNC([setppriv], > - [ AC_CHECK_HEADERS([priv.h], [ > - SOLARIS_PRIVS="yes" > + AC_CHECK_FUNC([priv_basicset], > + [ AC_CHECK_HEADERS([priv.h], [ > + SOLARIS_PRIVS="yes" > + ]) > ]) > ]) > AC_ARG_WITH([solaris-contracts], >The patch appears to get getting mangled somewhere along the way -- would you please resend it as some sort of an attachment? Even uuencoding it would be fine :-). -- Jeff Wieland | Purdue University Network Systems Administrator | ITIS UNIX Platforms Voice: (765)496-8234 | 155 S. Grant Street FAX: (765)496-1380 | West Lafayette, IN 47907
On Tue, Feb 16, 2016 at 08:18:03PM -0500, Jeff Wieland wrote:> The patch appears to get getting mangled somewhere along the way -- would > you please resend it as some sort of an attachment? Even uuencoding it > would be fine :-).Attached. Also uploaded at https://www.dtucker.net/tmp/openssh-solaris-priv.patch if that doesn't work. -- Darren Tucker (dtucker at zip.com.au) GPG key 8FF4FA69 / D9A3 86E9 7EEE AF4B B2D4 37C9 C982 80C7 8FF4 FA69 Good judgement comes with experience. Unfortunately, the experience usually comes from bad judgement. -------------- next part -------------- diff --git a/configure.ac b/configure.ac index b4c0aaa..5b50b9e 100644 --- a/configure.ac +++ b/configure.ac @@ -897,8 +897,10 @@ mips-sony-bsd|mips-sony-newsos4) AC_MSG_RESULT([no]) fi AC_CHECK_FUNC([setppriv], - [ AC_CHECK_HEADERS([priv.h], [ - SOLARIS_PRIVS="yes" + AC_CHECK_FUNC([priv_basicset], + [ AC_CHECK_HEADERS([priv.h], [ + SOLARIS_PRIVS="yes" + ]) ]) ]) AC_ARG_WITH([solaris-contracts],