This uses afero to proxy all file system accesses and then injects an
in-memory file system in order to test the getFile function.

Johannes Marbach authored 3 years ago and Clayton Craft committed 3 years ago

This commit includes three changes:

1. Use getFile in getHookFiles to add the contents of file hooks
2. Within getFile, first attempted a glob expansion on the specified
   file expression. This allows specifying e.g.
   /lib/udev/rules.d/*.rules.
3. Within getFile, next if the path points to a directory, add all
   files including those from subdirectories. This allows specifying
   e.g. /usr/share/X11/xkb.

Relates to: pmaports#1309

This will replace the busybox dhcp server.

With this change, mkinitfs depends on the 'unudhcpd' packge in Alpine to
provide the binary at /usr/bin/unudhcpd.

The glob might result in a vmlinuz-* filename that causes mkinitfs to
copy a modified kernel file to the boot-deploy working directory. This
excludes files that boot-deploy has touched from being copied/used by
boot-deploy

Minecrell authored 3 years ago and Clayton Craft committed 3 years ago

This should do the same as far as I can tell :)

Minecrell authored 3 years ago and Clayton Craft committed 3 years ago

At the moment modules in modules.dep are matched even on a submatch
e.g. looking up "msm" ends up matching "snd-soc-msm8916-digital.ko"
instead of "msm.ko". To fix this, disallow submatches using ^ and $.

bo41 authored 3 years ago and Clayton Craft committed 3 years ago

part of https://gitlab.com/postmarketOS/postmarketos/-/issues/26

Fixes an issue where some modules might have both - and _ in the name...
Also refactored this to make testing easier.

An earlier revision of this patch had test for invalid lines, but I
decided to remove it because deviceinfo is sourced by so many things in
pmOS that it should be /very/ obvious that there are invalid lines or
other garbage in the file.

I also tried to make the parser fairly forgiving if it does encounter
things that don't meet the deviceinfo "spec", so that mkinitfs will try
as hard as it can to make an initfs.

This replaces an external toml library with a simple parser for
unmarshalling a deviceinfo.

This directory doesn't exist all the time, e.g. if the kernel was built
without modules or no modules were installed for some reason. Assume the
kernel package knows what it is doing and just print a message that
might be helpful if the kernel package ends up not knowing what it is
doing.

The getopt thing was an experiment, and I'd rather lose external
dependencies than use getopt-like parsing especially since the only
argument this app takes is largely for testing/development purposes
only.

See: #6

Specifically this fixes failures: ST1005, S1028, S1011

See #6

See #6

staticcheck isn't available in a stable release of Alpine Linux yet...

See: #6

See: #6

See: #6

See: #6

See: #6

Apparently the space after is important or Go will consider the
copyright comments as package documentation.

See: #6

See #6

It has been determined that this test isn't really necessary. busybox
gzip is probably not going to break (and would be discovered elsewhere
too if it does...)

fixes #4

I misread what strings.Trim does, the 2nd param is a _cut set_, so this
was removing any occurance any characters in the 2nd param. Since the
file extension is always a suffix, this uses TrimSuffix.

Fixes a bug where 'virtio_blk.ko' was being trimmed to 'virtio_bl'

the characters - and _ are indistinguishable for module names, to the
kernel. But obviously are different when it comes to filenames/paths...
so this fixes an issue where the module was previously not found if it
was specified in config with the opposite - or _ than what the real
filename for the module had.

boot-deploy will handle post-processing steps for initramfs/kernel, and
installation into /boot.
Also see: https://gitlab.com/postmarketOS/boot-deploy

post-processing/installation will be handled by an external app, so
use a temp work dir for the external app to work out of when it is
invoked

A custom extract test includes reading AND writing the extracted archive
contents to /tmp (which is probably a slow disk), so this uses busybox's
gzip -t ("test file integrity") and seems to take about half the time
(~10s vs ~6s measured loosely on pinephone)

It might be missing if the device package is installed after mkinitfs is
being called, so don't cause apk triggers to error out by returning
failure.

Sometimes this is fine, e.g. when mkinitfs is run before the device
package is installed in a chroot. So it shouldn't kill the app.

The free space check, checksumming, and moving to the final location
will be outsourced to "boot-deploy", so the verification done here is
just to extract the archive to make sure it is valid.

This file is no longer a template since pmOS is only supporting 1
installed kernel at a time

pmOS will only support 1 kernel:
https://gitlab.com/postmarketOS/pmaports/-/issues/660