Skip to content
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

refactor(bluefin): clean up bluefin-branding spec (and remove distrobox config) #290

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tulilirockz
Copy link
Collaborator

@tulilirockz tulilirockz commented Mar 17, 2025

This pretty much just changes how the spec is made and moves some files to other folders instead of the weird structuring it had before.

Here is the diff, I just removed the default distrobox config because I remember having an issue where someone didnt want it

@@ -10,6 +10,9 @@
 ./etc/dconf/db/distro.d/05-bluefin-searchlight-extension
 ./etc/dconf/db/distro.d/locks
 ./etc/dconf/db/distro.d/locks/01-bluefin-locked-settings
-./etc/distrobox
-./etc/distrobox/distrobox.conf
-./etc/distrobox/distrobox.example
 ./etc/geoclue
 ./etc/geoclue/conf.d
 ./etc/geoclue/conf.d/99-beacondb.conf
@@ -75,6 +78,7 @@
 ./usr/share/backgrounds/bluefin/12-bluefin-night.jxl
 ./usr/share/backgrounds/bluefin/12-bluefin.xml
 ./usr/share/backgrounds/bluefin/chicken.jxl
-./usr/share/backgrounds/bluefin/gnome-background-properties
 ./usr/share/backgrounds/bluefin/xe_clouds.jxl
 ./usr/share/backgrounds/bluefin/xe_foothills.jxl
 ./usr/share/backgrounds/bluefin/xe_space_needle.jxl

This also allows us to individually version components

Fixes: ublue-os/bluefin#2156

@tulilirockz tulilirockz requested a review from castrojo as a code owner March 17, 2025 21:43
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 17, 2025
@tulilirockz tulilirockz added the safe-to-run PR is just fine to run, no RCE vulnerabilities found by manual review label Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe-to-run PR is just fine to run, no RCE vulnerabilities found by manual review size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove default distrobox config - NOT A BUG - design choice
1 participant