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

gnome-shell-extension-pixel-saver: init at 1.10 #34185

Merged
merged 1 commit into from Jan 24, 2018

Conversation

jonafato
Copy link
Contributor

Motivation for this change

Add the pixel saver GNOME Shell extension

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

description = "Pixel Saver is designed to save pixel by fusing activity bar and title bar in a natural way";
license = licenses.mit;
maintainers = with maintainers; [ jonafato ];
homepage = https://github.com/daedalnix/pixel-saver;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The URL is broken (ea → ae)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jtojnar Thanks, fixed.

@jonafato jonafato force-pushed the gnome-shell-extension-pixel-saver branch 2 times, most recently from 9e28cf5 to 6f8460a Compare January 24, 2018 02:05
@jtojnar
Copy link
Contributor

jtojnar commented Jan 24, 2018

Thanks.

meta = with stdenv.lib; {
description = "Pixel Saver is designed to save pixel by fusing activity bar and title bar in a natural way";
license = licenses.mit;
maintainers = with maintainers; [ jonafato ];
Copy link
Contributor

@jtojnar jtojnar Jan 24, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add the platforms attribute? https://nixos.org/nixpkgs/manual/#idm140737317010160

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to include platforms = platforms.linux;.

@jonafato jonafato force-pushed the gnome-shell-extension-pixel-saver branch from 6f8460a to d48e704 Compare January 24, 2018 02:17
@jonafato
Copy link
Contributor Author

@jtojnar Thanks for the feedback here. I plan to package more extensions, so getting a couple into a good state should make the following ones easier.

@jtojnar jtojnar merged commit 9b245b5 into NixOS:master Jan 24, 2018
@jtojnar
Copy link
Contributor

jtojnar commented Jan 24, 2018

Thanks.

@jonafato jonafato deleted the gnome-shell-extension-pixel-saver branch January 24, 2018 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants