prefix
ClosedPublic
Actions

Authored by rew on Dec 10 2020, 7:31 PM.

Details

Reviewers

trasz
kevans
imp

Commits

rG0a09fd56501c: geom(8): list geoms with /dev/ prefix
rS368576: geom(8): list geoms with /dev/ prefix

Summary

gpart show accepts geom names with the '/dev/' prefix, but gpart list does
not.

This patch modifies find_geom() in sbin/geom/core/geom.c to be consistent with
the behavior of find_geom() in lib/geom/part/geom_part.c

PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=188213

Test Plan

gpart list /dev/ada0

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rew created this revision.Dec 10 2020, 7:31 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 10 2020, 7:31 PM

rew requested review of this revision.Dec 10 2020, 7:31 PM

Harbormaster completed remote builds in B35320: Diff 80549.Dec 10 2020, 7:31 PM

Dozens of ways to skin this cat... this is one of the less objectionable ones :).

This revision is now accepted and ready to land.Dec 10 2020, 7:41 PM

LGTM

Approved by: kevans (mentor)

Closed by commit rS368576: geom(8): list geoms with /dev/ prefix (authored by rew). · Explain WhyDec 12 2020, 7:22 AM

This revision was automatically updated to reflect the committed changes.

rew added a commit: rS368576: geom(8): list geoms with /dev/ prefix.

rew added a commit: rG0a09fd56501c: geom(8): list geoms with /dev/ prefix.Jan 11 2021, 9:15 PM

Revision Contents
Changeset List

Path

Size

head/

sbin/

geom/

core/

geom.c

4 lines

Diff 80631

View Options

head/sbin/geom/core/geom.c

geom(8): list geoms with /dev/ prefixClosedPublicActions