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

hiddify-next: init at 1.1.1 #303085

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

wenjinnn
Copy link

@wenjinnn wenjinnn commented Apr 10, 2024

init with appimage version
fix #285375, related #287770
also add wenjinnn to maintainer-list

Description of changes

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

Copy link
Member

@donovanglover donovanglover left a comment

Choose a reason for hiding this comment

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

Welcome to nixpkgs! Make sure to put new packages in pkgs/by-name. You can also format your code with nixfmt-rfc-style

pkgs/applications/networking/hiddify-next/default.nix Outdated Show resolved Hide resolved
pkgs/applications/networking/hiddify-next/default.nix Outdated Show resolved Hide resolved
pkgs/applications/networking/hiddify-next/default.nix Outdated Show resolved Hide resolved
pkgs/applications/networking/hiddify-next/default.nix Outdated Show resolved Hide resolved
@donovanglover
Copy link
Member

When you're done, make sure that you have 2 separate commits in this order:

  1. maintainers: add wenjinnn
  2. hiddify-next: init at 1.1.1

@wenjinnn
Copy link
Author

When you're done, make sure that you have 2 separate commits in this order:

  1. maintainers: add wenjinnn
  2. hiddify-next: init at 1.1.1

All done! thanks for the suggestion and guide.

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/3855

@wenjinnn
Copy link
Author

wenjinnn commented Jun 2, 2024

Seems like after 24.05 release, appimageTools.wrapType2 behavior changed. after 24.05, this command will fail:

    mv $out/bin/hiddify-next-1.1.1 $out/bin/hiddify

now the location $out/bin/hiddify-next-1.1.1 change to $out/bin/hiddify-next, so I make some change to adapt this behavior.

init with appimage version
@linsui
Copy link
Contributor

linsui commented Jul 11, 2024

Since hidiffy violates the license used by sing-box I thought we shouldn't package it.

@wenjinnn
Copy link
Author

Well, I'm ok with that reason. should we close this PR now or wait for confirmation from others?

@BeiyanYunyi BeiyanYunyi mentioned this pull request Jul 11, 2024
13 tasks
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.

Package request: hiddify-next
7 participants