Differential D48535

gpart: Stop documenting freebsd-vinum partition types
ClosedPublic
Actions

Authored by jhb on Jan 20 2025, 7:46 PM.

Tags

None

Referenced Files

	F132371421: D48535.id150841.diff
	Thu, Oct 16, 7:50 AM

	Unknown Object (File)
	Sat, Oct 4, 1:34 AM

	Unknown Object (File)
	Sep 12 2025, 6:27 PM

	Unknown Object (File)
	Sep 10 2025, 10:04 AM

	Unknown Object (File)
	Sep 9 2025, 11:16 PM

	Unknown Object (File)
	Sep 8 2025, 4:59 AM

	Unknown Object (File)
	Sep 7 2025, 10:06 AM

	Unknown Object (File)
	Sep 7 2025, 8:30 AM

View All 33 Files

Subscribers

Details

Reviewers

emaste
imp

Commits

rGd35b039af944: gpart: Stop documenting freebsd-vinum partition types

Summary

Support for the partition types is still retained so that tools can
display existing partitions.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Jan 20 2025, 7:46 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 20 2025, 7:46 PM

jhb requested review of this revision.Jan 20 2025, 7:46 PM

Harbormaster completed remote builds in B61808: Diff 149578.Jan 20 2025, 7:46 PM

jhb added a child revision: D48536: ccdconfig.8: Refer to graid(8) and zfs(8) instead of gvinum(8).Jan 20 2025, 7:46 PM

imp accepted this revision.Jan 20 2025, 7:49 PM

This revision is now accepted and ready to land.Jan 20 2025, 7:49 PM

I'm fine with this, but wonder if we should leave this temporarily with a note that it's deprecated? I.e. imagine a user runs gpart and sees Vinum reported and looks for more information in the man page?

Yeah, I was a bit borderline on this change. I think we already don't document the dragonfly-vinum entry in this manpage though.

Closed by commit rGd35b039af944: gpart: Stop documenting freebsd-vinum partition types (authored by jhb). · Explain WhyFeb 11 2025, 2:14 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGd35b039af944: gpart: Stop documenting freebsd-vinum partition types.

Herald added a subscriber: ziaee. · View Herald TranscriptFeb 11 2025, 2:14 PM

Revision Contents
Changeset List

Path

Size

lib/

geom/

part/

11 lines

Diff 150841

lib/geom/part/gpart.8

Loading...