Page MenuHomeFreeBSD

bectl: push space-in-name check down into libbe
ClosedPublic

Authored by kevans on Mar 18 2022, 4:43 PM.
Tags
None
Referenced Files
F102941651: D34605.diff
Tue, Nov 19, 12:56 AM
Unknown Object (File)
Wed, Nov 6, 4:44 PM
Unknown Object (File)
Wed, Oct 30, 11:55 PM
Unknown Object (File)
Oct 1 2024, 3:48 PM
Unknown Object (File)
Oct 1 2024, 3:47 PM
Unknown Object (File)
Oct 1 2024, 3:47 PM
Unknown Object (File)
Oct 1 2024, 3:46 PM
Unknown Object (File)
Oct 1 2024, 3:36 PM
Subscribers

Details

Summary
bectl: push space-in-name check down into libbe

This check was previously in `create` only, not applying to renames.  It
should really be applied at the libbe level, so that we can avoid
writing about this restriction over and over again.

While we're here: `bectl rename` always succeeds, even when it doesn't.
Start returning the error.

Reported By:    Christian McDonald <cmcdonald netgate com>
Differential Revision:  https://reviews.freebsd.org/D34605

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

This revision is now accepted and ready to land.Mar 18 2022, 6:18 PM
kevans retitled this revision from bectl: disallow names with spaces in `rename` command to bectl: push space-in-name check down into libbe.
kevans edited the summary of this revision. (Show Details)

Reverse course, push the naming restriction into libbe as well for others' benefit (suggested by Christian).

This revision now requires review to proceed.Mar 31 2022, 10:56 PM
This revision is now accepted and ready to land.Apr 1 2022, 4:05 AM