Skip to content

Commit

Permalink
Merge pull request #324307 from NixOS/backport-324265-to-release-24.05
Browse files Browse the repository at this point in the history
[Backport release-24.05] catppuccin-gtk: fix inconsistent theme name
  • Loading branch information
fpletz authored Jul 9, 2024
2 parents 50d70e1 + c5897e9 commit 3bea90b
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 0 deletions.
32 changes: 32 additions & 0 deletions pkgs/by-name/ca/catppuccin-gtk/fix-inconsistent-theme-name.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
The theme name uses `default` as fallback for tweaks when they aren't
set, which not only is not a valid tweak name, but can lead to confusion
and inconsistencies (See: https://github.com/catppuccin/nix/pull/261).
---
sources/build/context.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/sources/build/context.py b/sources/build/context.py
index 3d43c01..6167f14 100644
--- a/sources/build/context.py
+++ b/sources/build/context.py
@@ -45,7 +45,8 @@ class BuildContext:
return f"{self.output_root}/{self.build_id()}"

def build_id(self) -> str:
- return f"{self.theme_name}-{self.flavor.identifier}-{self.accent.identifier}-{self.size}+{self.tweaks.id() or 'default'}"
+ tweaks = f"+{self.tweaks.id()}" if self.tweaks.id() != "" else ""
+ return f"{self.theme_name}-{self.flavor.identifier}-{self.accent.identifier}-{self.size}" + tweaks

def apply_suffix(self, suffix: Suffix) -> str:
if suffix.test(self):
@@ -59,6 +60,7 @@ class BuildContext:
Subsitution(find=f"\\${key}: {default}", replace=f"${key}: {value}"),
)

+
IS_DARK = Suffix(true_value="-Dark", test=lambda ctx: ctx.flavor.dark)
IS_LIGHT = Suffix(true_value="-Light", test=lambda ctx: not ctx.flavor.dark)
IS_WINDOW_NORMAL = Suffix(
--
2.45.1

2 changes: 2 additions & 0 deletions pkgs/by-name/ca/catppuccin-gtk/package.nix
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,8 @@ stdenvNoCC.mkDerivation {
hash = "sha256-q5/VcFsm3vNEw55zq/vcM11eo456SYE5TQA3g2VQjGc=";
};

patches = [ ./fix-inconsistent-theme-name.patch ];

nativeBuildInputs = [
gtk3
sassc
Expand Down

0 comments on commit 3bea90b

Please sign in to comment.