Bug 216830 - RPI2 configuration script for nanobsd is missing partition change requirements
Summary: RPI2 configuration script for nanobsd is missing partition change requirements
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: bin (show other bugs)
Version: 11.0-STABLE
Hardware: arm Any
: --- Affects Some People
Assignee: freebsd-bugs (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-02-05 17:06 UTC by karl
Modified: 2017-07-16 02:54 UTC (History)
1 user (show)

See Also:

Attachments
Define the partitions so the built system can mount root on startup (646 bytes, patch)
2017-02-05 17:06 UTC, karl 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description karl 2017-02-05 17:06:42 UTC 
Created attachment 179651 [details]
Define the partitions so the built system can mount root on startup

NanoBSD's RPI2 configuration has to override the partition settings or the root and cfg filesystems are wrong, and thus the built system will not boot.

Patch attached.
Comment 1 Warner Losh freebsd_committer freebsd_triage 2017-02-06 04:32:52 UTC 
It should already get all these settings from the std-embedded setting for NANO_LAYOUT. That's the bug that needs to get fixed. It used to do this just fine.
Comment 2 commit-hook freebsd_committer freebsd_triage 2017-02-06 06:16:44 UTC 
A commit references this bug:

Author: imp
Date: Mon Feb  6 06:15:39 UTC 2017
New revision: 313326
URL: https://svnweb.freebsd.org/changeset/base/313326

Log:
  o Add mkimg to the cross tools, and use the TMPPATH as PATH to pick up
    mkimg for building on systems like FreeBSD 11.0 that don't have my
    -a changes.
  o Set NANO_ROOT and NANO_ALTROOT for std-* since their values don't
    change when we set NANO_SLICE*.

  PR: 216829
  PR: 216830

Changes:
  head/tools/tools/nanobsd/embedded/common
Comment 3 Warner Losh freebsd_committer freebsd_triage 2017-07-16 02:54:34 UTC 
Committed, close it.