-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Finish merged-usr process for generic images, move SDK over to it too #2068
Merged
+84
−203
Commits on Sep 5, 2024
-
overlay sys-apps/baselayout: Refactor and complete merged-usr process
The initial goals of this commit were: - drop symlink-usr USE flag and keep the code paths where symlink-usr was evaluated to true, - make sbin a symlink to its bin counterpart, effectively doing the merged-sbin process too - unify filesystem layouts of the SDK and generic images. But over the course, more changes have accumulated: - use EAPI 8, - drop the check_sym function - it never worked due to typos (real_path and real_value versus read_path and read_value), - do the SDK-specific or generic-image-specific customizations in the src_prepare phase, - follow the changes made in the baselayout repository: - remove unnecessary tmpfiles.d conf files instead of fiddling with sed to edit them: - in the baselayout repo, the conf files were split to make it possible, - use tmpfiles.d to create core home directory: - used to be done differently for generic images and for SDKs, - use dumb-tmpfiles-proc.sh instead of systemd's tmpfile processor: - this removes the need to install valid passwd and group files into /etc before, - also it seems to be fixing some issues with installing files for users and groups that weren't there anyway, - drop generating of baselayout-usr in src_compile, and creating debug directories in pkg_preinst, these are handled by the Makefile now - this made inheriting systemd and tmpfiles eclasses unnecessary - install files in the src_install phase and install the directory structure in the pkg_preinst phase: - empty directories created in src_install are not guaranteed to be preserved, and indeed at some point /usr/local/bin was not installed on the final rootfs, - currently installed /etc/passwd and /etc/group are now empty - drop DEPEND variable entirely - systemd isn't really needed, I don't know what was the point of depending on libidn2, and the rest were conflicts with some old versions of packages.
Configuration menu - View commit details
-
Copy full SHA for 35325ae - Browse repository at this point
Copy the full SHA 35325aeView commit details -
overlay coreos-base/update_engine: Drop symlink-usr USE flag
It's gone, assuming its enabled.
Configuration menu - View commit details
-
Copy full SHA for 83341df - Browse repository at this point
Copy the full SHA 83341dfView commit details -
Configuration menu - View commit details
-
Copy full SHA for b0ab201 - Browse repository at this point
Copy the full SHA b0ab201View commit details -
Configuration menu - View commit details
-
Copy full SHA for c1c162e - Browse repository at this point
Copy the full SHA c1c162eView commit details -
overlay profiles: Some cleanups
Mask split-usr globally, not only for generic images. Move some SDK only USE flags to SDK target profile (cros_host, expat). Drop duplicated disabling of cups USE flag.
Configuration menu - View commit details
-
Copy full SHA for c850079 - Browse repository at this point
Copy the full SHA c850079View commit details -
overlay app-admin/sudo: Keep /etc/sudoers.d directory
It seems to be randomly kept or removed during installation, and we need the directory when creating the SDK container.
Configuration menu - View commit details
-
Copy full SHA for c134bac - Browse repository at this point
Copy the full SHA c134bacView commit details -
Configuration menu - View commit details
-
Copy full SHA for e893cc8 - Browse repository at this point
Copy the full SHA e893cc8View commit details -
Configuration menu - View commit details
-
Copy full SHA for f3f51ed - Browse repository at this point
Copy the full SHA f3f51edView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.