nanobsd: Add _xxx_pkg_db_dump_or_vacuum
Needs ReviewPublic
Actions

Authored by senguptaangshuman17_gmail.com on Thu, Aug 20, 12:22 PM.

Details

Reviewers

jlduran
jrm
ashish

Summary

Verify and compact target pkg database after manual edits, and dump it
to support deterministic pkgbase images.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 75914
Build 72797: arc lint + arc unit

Event Timeline

senguptaangshuman17_gmail.com created this revision.Thu, Aug 20, 12:22 PM

Herald added a subscriber: imp. · View Herald TranscriptThu, Aug 20, 12:22 PM

senguptaangshuman17_gmail.com requested review of this revision.Thu, Aug 20, 12:22 PM

Harbormaster completed remote builds in B75840: Diff 184511.Thu, Aug 20, 12:22 PM

senguptaangshuman17_gmail.com added a parent revision: D59045: nanobsd: Add _xxx_pkg_add_var_db_files_to_metalog.Thu, Aug 20, 12:33 PM

senguptaangshuman17_gmail.com added a child revision: D58961: nanobsd: Clean up target pkg database and metalog after install.

Address review comments: normalize comment to # XXX:

Herald added a reviewer: scheduler. · View Herald TranscriptThu, Aug 20, 7:03 PM

Harbormaster completed remote builds in B75914: Diff 184600.Thu, Aug 20, 7:03 PM

lwhsu removed a reviewer: scheduler.Thu, Aug 20, 8:17 PM

I think this should only do a vacuum (integrity check).
So far we have not been able to create deterministic pkgbase images.

In D59046#1354947, @jlduran wrote:

I think this should only do a vacuum (integrity check).
So far we have not been able to create deterministic pkgbase images.

As per my understanding, I'd like to keep the dump rather than dropping it, the function just dumps the db as a precondition for someday diffing/reproducing images. Once pkgbase builds are reproducible, this becomes the hook that lets us verify it, without another patch reintroducing it then.
Please let me know your view on this, then will change accordingly.

senguptaangshuman17_gmail.com updated this revision to Diff 184834.Sun, Aug 23, 11:02 PM

Harbormaster completed remote builds in B76020: Diff 184834.Sun, Aug 23, 11:02 PM