Differential D44356

freebsd-update: mark "cron" as fetched as "fetch" itself
ClosedPublic
Actions

Authored by michaelo on Mar 14 2024, 4:49 PM.

Tags

None

Referenced Files

	F116009625: D44356.diff
	Thu, May 1, 1:22 PM

	Unknown Object (File)
	Mon, Apr 28, 4:51 AM

	Unknown Object (File)
	Thu, Apr 24, 12:56 AM

	Unknown Object (File)
	Wed, Apr 23, 9:35 AM

	Unknown Object (File)
	Mar 25 2025, 2:16 AM

	Unknown Object (File)
	Mar 24 2025, 11:45 AM

	Unknown Object (File)
	Mar 24 2025, 10:38 AM

	Unknown Object (File)
	Feb 5 2025, 5:01 PM

View All 50 Files

Subscribers

Details

Reviewers

jrm
otis
cperciva
emaste

Commits

rGfc5208028a78: freebsd-update: mark "cron" as fetched as "fetch" itself
rGe54b9d17f82d: freebsd-update: mark "cron" as fetched as "fetch" itself
rG85c3ef77cc9f: freebsd-update: mark "cron" as fetched as "fetch" itself

Summary

The change in 33bd05c3187d7b49c80cf1b0132b405c105d0833 was incomplete
because it did not mark "cron" as ISFETCHED=1 although it performs the
same operations as "install", but less output and does not perform a
hard exit. Mark result as such and make "install" know that updates have
been fetched.

PR: 277699
Approved by: jrm (mentor), otis (mentor)
MFC after: 2 weeks

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 56602
Build 53490: arc lint + arc unit

Event Timeline

michaelo created this revision.Mar 14 2024, 4:49 PM

Herald added a subscriber: imp. · View Herald TranscriptMar 14 2024, 4:49 PM

michaelo requested review of this revision.Mar 14 2024, 4:49 PM

Harbormaster completed remote builds in B56602: Diff 135753.Mar 14 2024, 4:49 PM

@cperciva, it would be good to get your approval here as well.

This revision is now accepted and ready to land.Mar 15 2024, 3:47 PM

In D44356#1011926, @jrm wrote:

@cperciva, it would be good to get your approval here as well.

Unless someone says that one should never do "cron install" in one go in a cronjob...

Fine with me. Installing updates from cron isn't particularly recommended, but there may be cases where it is appropriate.

emaste accepted this revision.Mar 17 2024, 3:04 PM

Closed by commit rG85c3ef77cc9f: freebsd-update: mark "cron" as fetched as "fetch" itself (authored by michaelo). · Explain WhyMar 18 2024, 10:03 AM

This revision was automatically updated to reflect the committed changes.

michaelo added a commit: rG85c3ef77cc9f: freebsd-update: mark "cron" as fetched as "fetch" itself.

michaelo added a commit: rGe54b9d17f82d: freebsd-update: mark "cron" as fetched as "fetch" itself.Apr 1 2024, 7:19 PM

michaelo added a commit: rGfc5208028a78: freebsd-update: mark "cron" as fetched as "fetch" itself.

Revision Contents
Changeset List

Path

Size

usr.sbin/

freebsd-update/

freebsd-update.sh

1 line

Diff 135753

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

Loading...