Minor addition to release man page

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Minor addition to release man page

Hrishikesh Murukkathampoondi
Added small info to the release man page. Without this step building the
system gives the error

---
test -z "doas" || doas rm -f /var/run/testdata_root /var/run/keycommand_root
doas: Operation not permitted
*** Error 1 in regress/usr.bin/ssh (Makefile:207 'clean')
*** Error 1 in regress/usr.bin (<bsd.subdir.mk>:48 'cleandir')
*** Error 1 in regress (<bsd.subdir.mk>:48 'cleandir')
*** Error 1 in . (<bsd.subdir.mk>:48 'cleandir')
*** Error 1 in /usr/src (Makefile:77 'build')

---

Thanks
Hrishi


Index: share/man/man8/release.8
===================================================================
RCS file: /cvs/src/share/man/man8/release.8,v
retrieving revision 1.74
diff -u -p -r1.74 release.8
--- share/man/man8/release.8    23 Jul 2016 00:10:15 -0000      1.74
+++ share/man/man8/release.8    19 Mar 2017 09:16:23 -0000
@@ -176,6 +176,11 @@ It's safer (but slower) to remove your o
 them before the build.
 The steps are:
 .Pp
+Edit /etc/doas.conf as root and add the following line:
+.Bd -literal -offset indent
+permit keepenv nopass root as root
+.Ed
+.Pp
 Move all your existing object files out of the way and then remove
 them in the background:
 .Bd -literal -offset indent
Reply | Threaded
Open this post in threaded view
|

Re: Minor addition to release man page

Theo Buehler
On Sun, Mar 19, 2017 at 03:05:37PM +0530, Hrishikesh Muruk wrote:
> Added small info to the release man page. Without this step building the
> system gives the error
>
> ---
> test -z "doas" || doas rm -f /var/run/testdata_root /var/run/keycommand_root
> doas: Operation not permitted

Thanks. I agree that the old release manual didn't do a great job at
documenting how to set up doas.

However, in this release cycle, we overhauled the entire build process:
you're now supposed to start the build as root and the SUDO variable is
no longer used for base and xenocara. Thus, this addition is not needed.

Compare the two manuals (in particular sections 2-4)
http://man.openbsd.org/release (for -current)
http://man.openbsd.org/OpenBSD-6.0/release (for -release)
and see
https://www.openbsd.org/faq/current.html#r20161006
https://www.openbsd.org/faq/current.html#r20161014
https://www.openbsd.org/faq/current.html#r20161119
for the most important changes.

Reply | Threaded
Open this post in threaded view
|

Re: Minor addition to release man page

Theo de Raadt-2
In reply to this post by Hrishikesh Murukkathampoondi
No way.  Building a release does not need doas.  You've done something
wrong.

>Added small info to the release man page. Without this step building the
>system gives the error
>
>---
>test -z "doas" || doas rm -f /var/run/testdata_root /var/run/keycommand_root
>doas: Operation not permitted
>*** Error 1 in regress/usr.bin/ssh (Makefile:207 'clean')
>*** Error 1 in regress/usr.bin (<bsd.subdir.mk>:48 'cleandir')
>*** Error 1 in regress (<bsd.subdir.mk>:48 'cleandir')
>*** Error 1 in . (<bsd.subdir.mk>:48 'cleandir')
>*** Error 1 in /usr/src (Makefile:77 'build')
>
>---
>
>Thanks
>Hrishi
>
>
>Index: share/man/man8/release.8
>===================================================================
>RCS file: /cvs/src/share/man/man8/release.8,v
>retrieving revision 1.74
>diff -u -p -r1.74 release.8
>--- share/man/man8/release.8    23 Jul 2016 00:10:15 -0000      1.74
>+++ share/man/man8/release.8    19 Mar 2017 09:16:23 -0000
>@@ -176,6 +176,11 @@ It's safer (but slower) to remove your o
> them before the build.
> The steps are:
> .Pp
>+Edit /etc/doas.conf as root and add the following line:
>+.Bd -literal -offset indent
>+permit keepenv nopass root as root
>+.Ed
>+.Pp
> Move all your existing object files out of the way and then remove
> them in the background:
> .Bd -literal -offset indent
>