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

No new profile created #20

Open
liggettla opened this issue Mar 23, 2018 · 6 comments
Open

No new profile created #20

liggettla opened this issue Mar 23, 2018 · 6 comments

Comments

@liggettla
Copy link

liggettla commented Mar 23, 2018

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
Copy link
Contributor

Thanks for your contribution 👍
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
Copy link
Contributor

@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
Copy link
Author

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
Copy link
Contributor

arcticicestudio commented Mar 26, 2018

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

@arcticicestudio arcticicestudio added the Hacktoberfest This repository participates in the Hacktoberfest label Oct 1, 2018
@arcticicestudio arcticicestudio removed their assignment Dec 6, 2018
@AtomScott
Copy link

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
Copy link
Contributor

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants