-
Notifications
You must be signed in to change notification settings - Fork 4
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
chore: Put files in /etc/
instead in /usr/etc
#315
Conversation
This also modifies build-time part of modules to work with `/etc/` instead of `/usr/etc/`
/etc/
instead in `/etc/
instead in /usr/etc
/etc/
instead in /usr/etc
/etc/
instead in /usr/etc
/etc/
instead in /usr/etc
/etc/
instead in /usr/etc
I think that this needs to be tested for regressions, since I couldn't copy to The error I got was |
Is this OK to merge then or do we need to do something first? |
Have to test all modules for regression, but Since Ublue images don't copy |
There is a regression in I get this error:
When reverting to the current Everything else works. The workaround is to keep EDIT: Done the workaround |
This is ready to merge now (when other PRs get approved too) |
I think the signing module is busted until this gets merged in BTW |
Ready to merge, if CLI is ready @gmpinder Merge order: |
@xynydev CLI is ready, time to merge stuff |
* chore: Recommend the usage of `/etc/` instead of `/usr/etc/` Related PR: blue-build/modules#315 * docs: Remove mention of putting files to `/usr/etc/` in recipe
* chore: Recommend the usage of `/etc/` instead of `/usr/etc/` Related PR: blue-build/modules#315 * docs: Remove mention of putting files to `/usr/etc/` in recipe
* chore: Recommend the usage of `/etc/` instead of `/usr/etc/` Related PR: blue-build/modules#315 * docs: Remove mention of putting files to `/usr/etc/` in recipe (cherry picked from commit fa30038613a4b455c32a8115df2ca6ee4837097e)
This modifies build-time part of modules to work with
/etc/
instead of/usr/etc/
Related PR:
blue-build/template#38
blue-build/website#65