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

hyperspace: init at 1.1.4 #73564

Closed
wants to merge 2 commits into from
Closed

Conversation

dtzWill
Copy link
Member

@dtzWill dtzWill commented Nov 17, 2019

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
  • 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 nix-review --run "nix-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.

lib/licenses.nix Outdated Show resolved Hide resolved
@stale
Copy link

stale bot commented Jun 13, 2020

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:

  1. Search for maintainers and people that previously touched the related code and @ mention them in a comment.
  2. Ask on the NixOS Discourse.
  3. Ask on the #nixos channel on irc.freenode.net.

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 13, 2020
@matthiasbeyer
Copy link
Contributor

matthiasbeyer commented Oct 16, 2020

The package works, would be nice to have it in nixpkgs...

@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Oct 16, 2020
pkgs/applications/misc/hyperspace/default.nix Show resolved Hide resolved
pkgs/applications/misc/hyperspace/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/hyperspace/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/hyperspace/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/hyperspace/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/hyperspace/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/hyperspace/default.nix Outdated Show resolved Hide resolved
pkgs/top-level/all-packages.nix Outdated Show resolved Hide resolved
@stale
Copy link

stale bot commented Jun 3, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 3, 2021
@stale stale bot removed 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md labels Nov 18, 2021
@dtzWill dtzWill changed the title hyperspace: init at 1.0.3 hyperspace: init at 1.1.4 Nov 18, 2021
@dtzWill
Copy link
Member Author

dtzWill commented Nov 18, 2021

Updates/notes for anyone reading this in the future:

  • NPL is a bit of a mess, it seems? Not free (I think we discussed this somewhere previously too), according to hyperspace discussion on changing its license (not linking to avoiding bombing them with mention, LMK if curious/can't find), bumped to v4 from the v1 included with this PR originally. Authoritative URL for this version is tricky, it seems NPL author wants folks using v7+ and deprecated the v4 repository ... anyway this seems correct for hyperspace, which explicitly uses "NPLv4".
  • As I recall, the runtime deps are needed because they're looked for but not linked against explicitly, now that we have to pull in mesa for libgbm I'm unsure if that makes sense re:libGL. Leaving as I think libGL is preferred unless mesa itself is explicitly needed.
  • There's an appimage alternative they provide which might be preferred (if simpler to package), I don't know if there's a reason I went for the .deb. Seems less opaque :). In theory we could maybe bundle this ourselves but I haven't explored this, as the tooling isn't very Nix-compat (node). Staying the course for now.

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";
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
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.

Comment on lines +52 to +53
dontStrip = true;
dontPatchELF = true;
Copy link
Member

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 ];
Copy link
Member

Choose a reason for hiding this comment

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

Presumable platforms.linux, right?

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 19, 2022
@onny
Copy link
Contributor

onny commented Sep 7, 2022

Please rebase

@onny onny marked this pull request as draft September 7, 2022 06:32
@Mic92
Copy link
Member

Mic92 commented Dec 18, 2022

Closing because of lack of activity from the author. Please re-open if you plan to work on this.

@Mic92 Mic92 closed this Dec 18, 2022
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.

[request] hyperspace (mastodon client)
9 participants