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

draft: hunter: init at 1.3.5 #99898

Closed
wants to merge 1 commit into from
Closed

Conversation

magnetophon
Copy link
Member

@magnetophon magnetophon commented Oct 6, 2020

fixes #83554

Thanks to @callahad for writing the derivation!

Motivation for this change
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.

@magnetophon
Copy link
Member Author

Sorry for the force-push: I accidentally added some commits to this branch.

@magnetophon
Copy link
Member Author

I' having trouble interpreting the error grahamofborg-eval is giving me.
@callahad Do you know what is going on?

@callahad
Copy link
Member

Hi @magnetophon, I'm terribly sorry, but I don't fully understand that error either.

I do think this might be tricky to get merged into nixpkgs because it also pulls in the nixpkgs-mozilla repo to get a nightly version of the Rust compiler... not sure what the conventions are around accepting something like that here. This will be solved if rabite0/hunter#89 ever gets closed, but it seems like the Hunter author might have disappeared? They don't have any GitHub activity in the past ~5 months.

@magnetophon
Copy link
Member Author

@callahad Thanks.

Yeah, I also noticed the lack of progress on hunter.

, callPackage
, makeRustPlatform
, fetchFromGitHub
, IOKit ? null
Copy link
Member

Choose a reason for hiding this comment

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

Please use inherit in top-level to not default this to null cause this will always fail on darwin.

Comment on lines +11 to +12

assert stdenv.isDarwin -> IOKit != null;
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
assert stdenv.isDarwin -> IOKit != null;

description = "The fastest file manager in the galaxy!";
homepage = "https://github.com/rabite0/hunter";
license = licenses.wtfpl;
maintainers = [ maintainers.magnetophon ];
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
maintainers = [ maintainers.magnetophon ];
maintainers = with maintainers; [ magnetophon ];

assert stdenv.isDarwin -> IOKit != null;

let
date = "2020-05-22";
Copy link
Member

Choose a reason for hiding this comment

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

Is the current rust version in nix now sufficient?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nothing changed upstream since May 25.
Shall I mark this PR as WIP until upstream doesn't need nightly anymore?

Copy link
Member

Choose a reason for hiding this comment

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

Yes please. I don't think it is a good idea to include the overlay in nix.

@magnetophon magnetophon changed the title hunter: init at 1.3.5 draft: hunter: init at 1.3.5 Dec 28, 2020
@SuperSandro2000 SuperSandro2000 marked this pull request as draft January 18, 2021 00:28
@fufexan fufexan mentioned this pull request Apr 21, 2021
10 tasks
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.

Add hunter (file manager) package
4 participants