We don't include -g in the SDK CFLAGS, so don't waste time and space on
splitdebug. Other compilers like Rust still generate debug symbols, but
we don't need these either.
Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
Gentoo starts with this directory being empty, and Portage complains if
you populate it with merge-usr symlinks.
Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
A recent Portage fix should avoid any collisions by salting the debug
files with the package name and version. The fix inadvertently broke the
debuglink CRC, which gdb checks, but this went unnoticed in Gentoo
because gdb prefers looking up symbols with the build-id instead.
However, this was noticed in Flatcar because of the INSTALL_MASK! This
has now also been fixed in Gentoo. We can drop the INSTALL_MASK now to
rely on the build-id instead of the debuglink.
Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
I see a 25% improvement. `man make.conf` says that this can potentially
result in build failures, but it seems unlikely when merely installing
binary packages. I haven't had any issues after several local builds.
Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
This is used to read version.txt, which was written with single quotes
when Jenkins reverted a build. These quotes were then erroneously
written into /etc/os-release, which caused havoc. This has already been
addressed in jenkins-os, but let's make this code more robust anyway.
Fix this by grepping out the lines we need (in one go) and actually
parsing them with Bash. Less safe, but we already parse version.txt with
Bash in other places.
Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
Entering the SDK via `su` breaks stdout/stderr permissions because the
underlying /dev/pts/N device is still owned by root. `sudo` doesn't have
this problem. Neither does `runuser`, which might be preferable, but the
SDK doesn't have that.
This was previously `sudo su`, so changing this to `sudo sudo` might
seem logical, but I think the original `sudo` was only there to prevent
`su` from asking for a password. That shouldn't happen with a single
`sudo` call.
Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
Azure connection is not yet enabled back, so let's skip this for now to
allow the other garbage collection tasks to run.
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Loading config from the initrd with `oem://` was broken because Ignition
was still looking in /usr/share/oem, which is now moved to /oem by the
minimal initrd.
This also fixes mounting the OEM partition when /mnt does not already
exist. This fix is slightly academic, because this currently only
happens when PXE booting, where the OEM partition won't exist anyway,
but we should fail for the right reason.
Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
