Page MenuHomeFreeBSD

freebsd-update(8): Add -j flag to support jails
ClosedPublic

Authored by kfv_kfv.io on Jul 18 2020, 12:24 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 17 2024, 12:33 PM
Unknown Object (File)
Feb 10 2024, 6:20 PM
Unknown Object (File)
Jan 10 2024, 1:13 PM
Unknown Object (File)
Dec 20 2023, 8:45 AM
Unknown Object (File)
Dec 12 2023, 1:26 PM
Unknown Object (File)
Dec 7 2023, 4:37 PM
Unknown Object (File)
Nov 11 2023, 6:53 AM
Unknown Object (File)
Nov 10 2023, 12:49 PM
Subscribers

Details

Summary

Make freebsd-update(8) support jails by adding the -j flag which takes jail jid or name as an argument. This requires and depends on D25705 since it uses freebsd-version -j jail to get the version of the installed userland.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 35353
Build 32279: arc lint + arc unit

Event Timeline

bcr requested changes to this revision.Jul 18 2020, 9:38 AM
bcr added a subscriber: bcr.
bcr added inline comments.
usr.sbin/freebsd-update/freebsd-update.8
99 ↗(On Diff #74606)

s/the/The/

101 ↗(On Diff #74606)

The sentence stop is missing here.

This revision now requires changes to proceed.Jul 18 2020, 9:38 AM

The man page looks good now, thanks!

Any updates on the src part of the change?

I think we won't progress here if we don't get someone else to review the src parts.
@cperciva: Can you take a look at it?

dteske requested changes to this revision.Dec 11 2020, 4:43 PM

Just a simple bogon in the src bits, after which I will approve

usr.sbin/freebsd-update/freebsd-update.sh
332

I would use NR instead of FNR here simply because FNR has the additional complexity that it will reset to 1 when working on the next file, but when used to process a pipe stream it is in-essence working on a single file (/dev/stdin) and thus NR and FNR are always aligned making the use of FNR a bit of a bogon

This revision now requires changes to proceed.Dec 11 2020, 4:43 PM

Fixed. Thanks for the neat point, I'll keep it in mind.

Fixing my previous mistake - my apologies.

Sorry again, it must be good this time.

@bcr: I think we could proceed now. Please note, that a very little change was made in D25705 where this diff relies on, and if you or anyone approve differentials, I think we can have them landed in 13.0-RELEASE, right? Please let me know if there is anything else I need to improve and update.

Just a simple bogon in the src bits, after which I will approve

However, not essential to change. In thinking further about this, FNR prevents a future change if FNR is indeed necessary.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 1 2021, 6:51 PM
This revision was automatically updated to reflect the committed changes.