nordtheme / gnome-terminal

An arctic, north-bluish clean and elegant GNOME Terminal color theme.
https://www.nordtheme.com/ports/gnome-terminal
MIT License
489 stars 39 forks source link

No new profile created #20

Open liggettla opened 6 years ago

liggettla commented 6 years ago

I saw another user having problems with no new 'Nord' profile being created, and I am having the same problem. I installed all of your specified dependencies and the install appears to be working correctly. But I do not get a new profile being created.

I am using Gnome terminal 3.22.2 on Debian Stretch and below is the output I see. It looks to me like it is applying the settings to the '' profile rather than the 'Nord' profile.

[INFO] Validated required dependencies: dconf expr gsettings uuidgen
[INFO] Detected compatible GNOME Terminal version 3.22.2 (>= 3.8 dconf migrated)
GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.
[INFO] Available profile UUIDs: b1dcc9dd-5262-4d8d-a863-c897e6d979b9
GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.
GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.
GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.
[INFO] Cloned the default profile 'b1dcc9dd-5262-4d8d-a863-c897e6d979b9' with new UUID '48a30685-7008-46d9-beb0-1520e6e4b080'
[DEBUG] Applied Nord color palette
[DEBUG] Applied background- and foreground colors
[DEBUG] Applied bold color and configuration
[DEBUG] Applied system theme compability configuration
[DEBUG] Applied cursor colors and configuration
[DEBUG] Applied highlight colors and configuration
[DEBUG] Applied highlight colors and configuration
[DEBUG] Set Nord GNOME Terminal version key of the '' profile
[INFO] Applied theme colors and configurations
[SUCCESS] Nord GNOME Terminal version 0.1.0 has been successfully applied to the newly created 'Nord' profile
[DEBUG] Cleaning up script execution by unsetting declared functions and variables
arcticicestudio commented 6 years ago

Thanks for your contribution :+1: Looks like this is related to #19.

This line is not logged by the script but the underlying used library:

GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.

This implies that the script works fine, but the changes made to dconf are read-only and will be dropped as soon as the script finish. I'll evaluate in which version this change has been introduced to GLib/GIO and fix it via #19.

arcticicestudio commented 6 years ago

@liggettla Can you please try to reproduce the problem when running the following line before the install script:

export GIO_EXTRA_MODULES=/usr/lib/x86_64-linux-gnu/gio/modules
liggettla commented 6 years ago

When I first run the line you gave me, the 'Nord' profile is now created and I can successfully use it. So, looks like that fixed it.

arcticicestudio commented 6 years ago

@liggettla Thanks for testing. I'll add the constant to the script to fix the problem for affected distributions.

AtomScott commented 4 years ago

Is this still not added? I ran into the same problem today and export GIO_EXTRA_MODULES=/usr/lib/x86_64-linux-gnu/gio/modules fixed it.

arcticicestudio commented 4 years ago

@AtomScott The line above was just for testing purposes, it should be added to the install script itself without exporting it into the current session, otherwise it'll mess up the users environment, for example when the variables was already set and included other values or multiple paths. Feel free to submit a PR, every contribution is always welcome.