Skip to content

Fix Nix builds#409

Merged
slotThe merged 1 commit intoxmonad:masterfrom
alternateved:fix-nix-build
Aug 2, 2022
Merged

Fix Nix builds#409
slotThe merged 1 commit intoxmonad:masterfrom
alternateved:fix-nix-build

Conversation

@alternateved
Copy link
Copy Markdown
Contributor

@alternateved alternateved commented Jul 23, 2022

Description

As advised in NixOS/nixpkgs@36d5761 this PR provides a way to set the path for xmessage and ghc binaries via environment variables XMONAD_XMESSAGE and XMONAD_GHC.

Checklist

  • I've read CONTRIBUTING.md

  • I've confirmed these changes don't belong in xmonad-contrib instead

  • I've considered how to best test these changes (property, unit,
    manually, ...) and concluded: XXX

  • I updated the CHANGES.md file

@alternateved alternateved force-pushed the fix-nix-build branch 2 times, most recently from 741e931 to 54778f7 Compare July 24, 2022 08:42
As advised in [1], provide a way to set the path for the xmessage and
ghc binaries via XMONAD_XMESSAGE and XMONAD_GHC environment variables.

[1]: NixOS/nixpkgs@36d5761
@slotThe slotThe merged commit 23f36d7 into xmonad:master Aug 2, 2022
@slotThe
Copy link
Copy Markdown
Member

slotThe commented Aug 2, 2022

Thank you! (I just added the PR description to the commit)

@sternenseemann
Copy link
Copy Markdown

I've opened a PR to reflect this change on the NixOS side: NixOS/nixpkgs#184784. After this, xmonad from master should work with the module shipped with NixOS directly.

@alternateved alternateved deleted the fix-nix-build branch August 4, 2022 11:16
sternenseemann added a commit to sternenseemann/nixpkgs that referenced this pull request Sep 14, 2022
We had a long standing downstream patch for XMonad allowing us to
override the ghc and xmessage binaries used. That has been upstreamed
and released in 0.17.1 and we can drop it!

* patch upstreamed in xmonad/xmonad#409
* downstream patch adjusted in NixOS#184784
sternenseemann added a commit to NixOS/nixpkgs that referenced this pull request Sep 14, 2022
We had a long standing downstream patch for XMonad allowing us to
override the ghc and xmessage binaries used. That has been upstreamed
and released in 0.17.1 and we can drop it!

* patch upstreamed in xmonad/xmonad#409
* downstream patch adjusted in #184784
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants