Skip to content

wayfire: init at 0.4.0#86569

Closed
wucke13 wants to merge 1 commit intoNixOS:masterfrom
wucke13:wayfire
Closed

wayfire: init at 0.4.0#86569
wucke13 wants to merge 1 commit intoNixOS:masterfrom
wucke13:wayfire

Conversation

@wucke13
Copy link
Contributor

@wucke13 wucke13 commented May 2, 2020

Motivation for this change

I would like to try this wm!

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • 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 nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

This is a re-worked version of #76759
@Thra11 @primeos

@wucke13
Copy link
Contributor Author

wucke13 commented May 2, 2020

@GrahamcOfBorg build wayfire wcm wf-shell

@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label May 2, 2020
@ofborg ofborg bot requested a review from Thra11 May 2, 2020 11:03
@ofborg ofborg bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels May 2, 2020
@alyssais alyssais mentioned this pull request May 2, 2020
10 tasks
@alyssais
Copy link
Member

alyssais commented May 2, 2020

Hello! I’ve been working on this for the past few days as well. I opened a PR with my implementation as #86606. My version also has support for installing Wayfire plugins, which by the looks of things this one lacks.

How do you want to go forward from here? Should we maybe look at each other’s PRs and see if we can combine efforts? I think it would make sense to use my PR as a base because of the plugin support.

Does wf-shell work in this PR? If so, how did you get around the hard-coded Wayfire prefix for plugins?

@wucke13
Copy link
Contributor Author

wucke13 commented May 3, 2020

@alyssais I'd love to join forces! As I'm not super confident of my skills I'm totally supporting the idea to go on with your PR. Please feel free to look at my PR.
wf-shell works, in terms of starting and displaying something. What I did was to point each components metadata directory to $out of the regarding nixpkg, and use a buildEnv of all these paths for the wcm utility to pick them up. AFAIK that does work, though I'm not aware of any functionality that wf-shell could miss by this attempt.

@Thra11 Thra11 mentioned this pull request May 4, 2020
10 tasks
@wucke13
Copy link
Contributor Author

wucke13 commented May 23, 2020

closing in favor of #86606.

@wucke13 wucke13 closed this May 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: package (new) This PR adds a new package 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants