7908 add loader manpage to pkg://system/boot/loader

Reviews
Diff

- Download Diff

Information
Submitter:	Toomas Soome
Repository:	illumos-gate
Branch:
Bugs:	7908
Depends On:
Commit:	a66f8a4...
Reviewers
Groups:	general
People:

Description

7908 add loader manpage to pkg://system/boot/loader

Note the same review is available as http://cr.illumos.org/~webrev/tsoome/7908/

Testing Done

Issues

0
1
0
All Issues: 1

Description	From	Last Updated
built in -> builtin	Adam Števko	April 26, 2017, 9 a.m.

usr/src/man/man4/loader.conf.4 (Diff revision 1)

Show all issues

built in -> builtin

Description:

		7908 add loader manpage to pkg://system/boot/loader
	+
	+	Note the same review is available as http://cr.illumos.org/~webrev/tsoome/7908/

Commit:

-	cda62ee4ad9bc495316a3b216ebbc641559f67ad
+	2ddee01c3f1880511e71553a66027007057b3c0d

Diff:

Revision 2 (+529 -1131)

Show changes

	usr/src/man/man3/Makefile
	usr/src/boot/lib/libstand/libstand.3
	usr/src/man/man4/Makefile
	usr/src/boot/sys/boot/forth/loader.conf.5
	usr/src/man/man5/Makefile
	usr/src/boot/sys/boot/forth/beastie.4th.8
	usr/src/boot/sys/boot/forth/brand.4th.8
	usr/src/boot/sys/boot/forth/check-password.4th.8
	14 more

Section indentation seems wrong in zfsloader

altho should be spelt although

should the pages be in man5 or man4? (particularly .4th pages)

should we ship oi/dilos/omnios branding with illumos by default?

pxeboot.5 - I thought tftp/nfs had been reworked to *not* use option
150?

Toomas Soome May 17, 2017, 7:05 p.m.

The 4th scripts are more about scripting and less about specific configuration, thats why I did use section 5. I do not know if the section 5 is the best place -- perhaps we just should introduce section 8, but this can

The branding bits is a bit more complicated issue, in any case, those can always be added later if people feel so.

the tftp option 150 is not yet commited, the versioning does sort of serialize things;) but if we get the manuals in first, I'll update that change to include update for the manual.

Change Summary:

Suggestions from Peter.

Commit:

-	2ddee01c3f1880511e71553a66027007057b3c0d
+	24611abd8ca83cf9e86147278cbdc9381e6ead9e

Diff:

Revision 3 (+534 -1136)

Show changes

	usr/src/man/man3/Makefile
	usr/src/boot/lib/libstand/libstand.3
	usr/src/man/man4/Makefile
	usr/src/boot/sys/boot/forth/loader.conf.5
	usr/src/man/man5/Makefile
	usr/src/boot/sys/boot/forth/beastie.4th.8
	usr/src/boot/sys/boot/forth/brand.4th.8
	usr/src/boot/sys/boot/forth/check-password.4th.8
	14 more

Ship it!

Ship It!

Change Summary:

remove libstand.3

Commit:

-	24611abd8ca83cf9e86147278cbdc9381e6ead9e
+	a66f8a4273db6ff144bad8d4882f762c82fdb0f7

Diff:

Revision 4 (+527 -1807)

Show changes

	usr/src/boot/lib/libstand/libstand.3
	usr/src/man/man4/Makefile
	usr/src/boot/sys/boot/forth/loader.conf.5
	usr/src/man/man5/Makefile
	usr/src/boot/sys/boot/forth/beastie.4th.8
	usr/src/boot/sys/boot/forth/brand.4th.8
	usr/src/boot/sys/boot/forth/check-password.4th.8
	usr/src/boot/sys/boot/forth/color.4th.8
	13 more

Ship it!

Ship It!

Ship it!

Ship It!

You have a pending review.

Review Board 2.5.16

Screenshots

Files

Issues

Description:

Commit:

Diff:

Change Summary:

Commit:

Diff:

Change Summary:

Commit:

Diff:

Status: Closed (submitted)