-
Notifications
You must be signed in to change notification settings - Fork 37
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
Consider using flakes #6
Comments
I've been trying to write a pure Nix Flake for this but have not succeeded. This is what I have thus far: {
outputs = { self, nixpkgs, flake-utils, ... } @ inputs:
(flake-utils.lib.eachSystem [ "x86_64-linux" ] (system:
let
pkgs = import inputs.nixpkgs {
inherit system;
overlays = [
(self: super: {
spotify-unwrapped = super.spotify-unwrapped.overrideAttrs (drv: {
buildInputs = (drv.buildInputs or [ ]) ++ (with super; [
spicetify-cli
]);
postInstall = (drv.postInstall or "") + ''
export SPICETIFY_CONFIG=$out
touch $out/prefs
# Generate config file
spicetify-cli config || true
sed -i "s|.*spotify_path.*|spotify_path = $out/share/spotify|g" $out/config-xpui.ini
sed -i "s|.*prefs_path.*|prefs_path = $out/prefs|g" $out/config-xpui.ini
spicetify-cli backup enable-devtool apply
'';
});
})
];
config = {
allowUnfreePredicate = pkg: builtins.elem (lib.getName pkg) [
"spotify"
"spotify-unwrapped"
];
};
};
lib = pkgs.lib;
in
{
packages.default = pkgs.spotify;
defaultPackage = self.packages.${system}.default;
}
));
}
It says it installs |
@pietdevries94 do you have any ideas about why this Flake doesn't work, I tried to the minimum needed from Lines 63 to 121 in c386ede
|
@PAI5REECHO @Ashe See #7 if you are still trying to get this working. |
@the-argus thank you, I'll look into it! |
* flake: rename outputs to match schema/standard * flake: reintroduce old output names as deprecated * flake: remove unnecessary uses of rec
Hey, massive fan of what you're doing, but since I've moved my config over to using flakes I can't seem to make use of your work since it doesn't allow me to pass nixpkgs into the package and therefore the package is impure :(
Would love to see some updates and even getting this into homemanager eventually!
The text was updated successfully, but these errors were encountered: