In D56381#1295504, @ziaee wrote:

note, i like a blank line at the end of scripts for cat reasons. if we don't want this, i can remove it, but it doesn't hurt anything and we do it all over the place.

In D56381#1295529, @ziaee wrote:

use logic inspired from @lwhsu to prevent reboot if nothing happened. compare the file size in bytes of the pkg database instead of using pkg inf and checksum to save water/coal. tested on my laptop

In D56381#1295532, @ivy wrote:

@lwhsu out of interest, how do you end up with a system that has FreeBSD-base packages installed but doesn't have pkg bootstrapped?

@lwhsu out of interest, how do you end up with a system that has FreeBSD-base packages installed but doesn't have pkg bootstrapped?

use logic inspired from @lwhsu to prevent reboot if nothing happened. compare the file size in bytes of the pkg database instead of using pkg inf and checksum to save water/coal. tested on my laptop

implement bootstrap pkg if it is not done so already, based on lwhsu feedback, thanks! note, i like a blank line at the end of scripts for cat reasons. if we don't want this, i can remove it, but it doesn't hurt anything and we do it all over the place.

btw, you can try to run it with rclint, but I don't usually follow "Do not quote values unless necessary" error.

Hi, I am also working on this and didn't know this is earlier than me. I think this one is more complete while mine is only focusing on FreeBSD-base repo. However I guess there are still something useful (or not, as I'm still testing):

• bootstrap/update pkg
• only reboot when there is pkg updated

In D55751#1276024, @cperciva wrote:

Thanks, I fixed the typo. Also I added periods at the end of sentences.

In D55751#1276101, @leres wrote:

Note that https://bugs.freebsd.org/293654 has been solved by D55713. (I'm not clear on how to insure it is included in 15.1)

Note that https://bugs.freebsd.org/293654 has been solved by D55713. (I'm not clear on how to insure it is included in 15.1)

Thanks, [.filename] highlights the text in green and avoids the need to have a space between the path and the terminating period.

Use [.filename].

Thanks, I fixed the typo. Also I added periods at the end of sentences.

Update with new text

Does that sound ok to you?

In D55751#1275976, @ziaee wrote:

not acceptable to me

The workaround is a part of an issue in the Open Issues section,

Is "use of a symlink to select the kernel tree to boot" not clear?

In D55751#1275970, @ziaee wrote:

Your version leaves out "regression" which I included because this use worked before.

I'm not attached to my version, we just need to have something that's clear. It's fine to include the word regression, however the sole point of this section is listing regressions.

Your version leaves out "regression" which I included because this use worked before.

Update issue number

In D55764#1275794, @ziaee wrote:

Approved with removing the anchors just before

Approved with removing the anchors just before

If you're going to take these out, probably should take out the anchors just before them too

Vladlen's suggested text would address my concerns. @leres can you do this before the release announcement goes out in ten hours? If I hear nothing and no one objects, I will commit the one I wrote already in a few hours.

Thanks for your hard work @vladlen!

rerun changed to reuse
dash (-) and the final + kept unchanged according to the style in 15.0R errata.adoc.

• Character Replacement Substitutions | Asciidoctor Docs
• Hard Line Breaks | Asciidoctor Docs.

I fixed punctuation and kept "then rerun man:freebsd-update[8]" , otherwise it looks like advise to run pkg instead of freebsd-update (too short description).

Make sure to use PR: and not Fixes: for the commit message trailer!

omg this is amazing!

I really like Graham's suggested workaround, but I personally think it's wrong to punctuate items in this type of list. It's bad for URLs. I approve for accepting any combination or none of the suggestions. Thanks @vladlen!