Details

Reviewers

jlduran
jrm
ashish

Summary

Add helper functions to update the target pkg SQLite database when NanoBSD modifies configuration files and filesystem layout:

Refresh config_files content and sha256 checksums when etc/defaults/rc.conf, etc/ttys, or etc/ssh/sshd_config are modified.
Update files table paths from /usr/local/etc to /etc/local.
Repoint /tmp rows to /var/tmp in pkg_directories after /tmp is symlinked.
Set file mtimes and package times to NANO_TIMESTAMP for reproducible builds.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 75788
Build 72671: arc lint + arc unit

Event Timeline

senguptaangshuman17_gmail.com created this revision.Wed, Aug 19, 10:17 AM

Herald added a subscriber: imp. · View Herald TranscriptWed, Aug 19, 10:17 AM

senguptaangshuman17_gmail.com requested review of this revision.Wed, Aug 19, 10:17 AM

Harbormaster completed remote builds in B75707: Diff 184319.Wed, Aug 19, 10:17 AM

senguptaangshuman17_gmail.com added a parent revision: D58953: nanobsd: Support -PU builds with COMPONENTS.Wed, Aug 19, 10:17 AM

senguptaangshuman17_gmail.com added a child revision: D58955: nanobsd: Update pkg database sha256 and config content.

senguptaangshuman17_gmail.com updated this revision to Diff 184454.Thu, Aug 20, 10:10 AM

Harbormaster completed remote builds in B75788: Diff 184454.Thu, Aug 20, 10:10 AM

senguptaangshuman17_gmail.com updated this revision to Diff 184495.Thu, Aug 20, 12:20 PM

Harbormaster completed remote builds in B75824: Diff 184495.Thu, Aug 20, 12:20 PM

jlduran added inline comments.Thu, Aug 20, 4:37 PM

tools/tools/nanobsd/defaults.sh
1255	I don't think you can have an empty if body. Try using `:` in the meantime.

Consolidate target pkg database synchronization (merges D58954, D58955, D58960)

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

Harbormaster completed remote builds in B75908: Diff 184594.Thu, Aug 20, 7:01 PM

senguptaangshuman17_gmail.com added a child revision: D59037: nanobsd: Add metalog_add_data helper.Thu, Aug 20, 7:04 PM

senguptaangshuman17_gmail.com mentioned this in D58955: nanobsd: Update pkg database sha256 and config content.

senguptaangshuman17_gmail.com mentioned this in D58960: nanobsd: Fix target pkg database /tmp symlink and timestamps.

senguptaangshuman17_gmail.com retitled this revision from nanobsd: Add tgt_pkg_update_config_files_content() to nanobsd: Synchronize target pkg database with image modifications.

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

senguptaangshuman17_gmail.com updated this revision to Diff 184823.Sun, Aug 23, 11:01 PM

Harbormaster completed remote builds in B76009: Diff 184823.Sun, Aug 23, 11:01 PM

senguptaangshuman17_gmail.com updated this revision to Diff 184976.Tue, Aug 25, 6:12 AM

Harbormaster completed remote builds in B76099: Diff 184976.Tue, Aug 25, 6:12 AM

jlduran mentioned this in D58956: nanobsd: Use local repository cache for precompiled package base builds.Wed, Aug 26, 2:49 PM

senguptaangshuman17_gmail.com edited the summary of this revision. (Show Details)Thu, Aug 27, 9:31 PM

Normalize comment line wrap in config files update helper

Harbormaster completed remote builds in B76243: Diff 185253.Thu, Aug 27, 9:34 PM