-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
obsidian: init at 0.7.3 #91454
obsidian: init at 0.7.3 #91454
Conversation
@@ -19,6 +19,9 @@ appimageTools.wrapType2 rec { | |||
export XDG_DATA_DIRS=${gsettings-desktop-schemas}/share/gsettings-schemas/${gsettings-desktop-schemas.name}:${gtk3}/share/gsettings-schemas/${gtk3.name}:$XDG_DATA_DIRS | |||
''; | |||
|
|||
# Strip version from binary name. | |||
extraInstallCommands = "mv $out/bin/${name} $out/bin/${pname}"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I get mixed results with this... When running nix-build -A obsidian
from the root of my cloned nixpkgs folder, the operation works correctly. However, in my /etc/nixos/config, if I import my custom branch and include obsidian, then it remains obsidian-0.7.3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found a good example in the notable package. Tested the fix on a seperate branch, got the expected name.
commit 2d26a738eefebc6fd4718ae019505668d3330084 Author: Elian Doran <contact@eliandoran.me> Date: Wed Dec 29 02:00:18 2021 +0200 keeweb: Use alternative solution for GSettings Apparently the last attempt to use `wrapGAppsHook` into `extraPkgs` failed to solve the issue. Found in NixOS#91454 a potential solution that might also work here. commit 9ea5fdeea16d810f23fd339aa1401aa7adba28c2 Author: Elian Doran <contact@eliandoran.me> Date: Wed Dec 29 01:52:19 2021 +0200 keeweb: Fix open/save dialogs by wrapping GSettings
Motivation for this change
Adds Obsidian editor version 0.7.3 (#89089)
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)