-
-
Notifications
You must be signed in to change notification settings - Fork 392
Nix support 2.0 #194
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
Nix support 2.0 #194
Conversation
|
Note about the note, the black box bug most likely has nothing to do with the nix package, but it was a thing of freerdp when you have the windows desktop open, it can't open apps. |
|
Thanks, hopefully this will work better for Nix folks. 🙏 |
|
@TibixDev PR didn't close the related issues |
|
@Simon-Weij thanks for the hard work. Seems like you removed the support of the module? |
|
This flake is pretty much from scratch again, I or someone else can re-add the module if you ask me. |
|
I was close to finishing packaging winboat from source for Nix, you may be interested: NixOS/nixpkgs#440321 |
* Fix nix flake * Added workflow * Added git add flake.nix * Added git merge * added branch to push to nix-support for testing purposes * Added some logging * Moved git credentials definition * switched back to main branch
I packaged winboat again, with a couple of changes.
Over the current flake:
Packages the tarball instead of the appImage, this solves a couple isolation issues (like not detecting the docker group)
Uses the tarball from the release, this will eventually make it easier to merge winboat into nixpkgs
Doesn't package freerdp, this should help with a bug where freerdp showed a black box and instantly crashes (see note)
It tries to keep stuff pretty minimal so there won't be many nix-specific bugs
Workflow to update the hash, which I've tested and should work fine. But obviously it hasn't been tested on the main repository
This pr should fix issue #171 and #164
NOTE:
The black box still happens sometimes, this package works 100% fine on my desktop, but the bug occurred on my laptop, which later resolved itself, I'm not entirely sure how to reproduce this, but this version should work for more people than the current version that packages the appImage.