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
hyperspace: init at 1.1.4 #73564
hyperspace: init at 1.1.4 #73564
Conversation
e5b2cd5
to
cce9337
Compare
Thank you for your contributions. This has been automatically marked as stale because it has had no activity for 180 days. If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity. Here are suggestions that might help resolve this more quickly:
|
The package works, would be nice to have it in nixpkgs... |
I marked this as stale due to inactivity. → More info |
cce9337
to
bcc67dc
Compare
ef6472f
to
eddb645
Compare
eddb645
to
b155ba0
Compare
Updates/notes for anyone reading this in the future:
Addressed review feedback, thank you! Sorry for delay! :) Tested the result and seems to work as expected so far. The icon is... X11 logo? Which suggests something strange happening but I don't think is critical. -- Well anyway, here's the latest hyperspace :). |
version = "1.1.4"; | ||
|
||
src = fetchurl { | ||
url = "https://github.com/hyperspacedev/${pname}/releases/download/v${version}/${pname}_${version}_amd64.deb"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
url = "https://github.com/hyperspacedev/${pname}/releases/download/v${version}/${pname}_${version}_amd64.deb"; | |
url = "https://github.com/hyperspacedev/hyperspace/releases/download/v${version}/hyperspace_${version}_amd64.deb"; |
easy to visit links are important if you want to research with one click if this can be build from source.
dontStrip = true; | ||
dontPatchELF = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those break the binary?
description = "A beautiful, fluffy client for the fediverse"; | ||
homepage = "https://hyperspace.marquiskurt.net"; | ||
license = licenses.npl4; | ||
maintainers = with maintainers; [ dtzWill ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Presumable platforms.linux, right?
Please rebase |
Closing because of lack of activity from the author. Please re-open if you plan to work on this. |
Motivation for this change
Mastodon client :).
(electron-based)
FWIW, the license is new to us and not widely used.
Added it to the list, but am unsure how it should be handled.
Hopefully it's alright to include software under this license,
but I mention it because I would appreciate a second opinion O:).
Fixes #57357.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)