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
imhex: init at 1.6.0-git #106576
imhex: init at 1.6.0-git #106576
Conversation
nativeBuildInputs = [ cmake pkgconfig ]; | ||
buildInputs = [ nlohmann_json capstone file glfw3 glm jsoncpp llvm_9 openssl python38 ]; | ||
|
||
patches = [ ./default-db.patch ]; |
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.
please submit the patch upstream, and fetch the corresponding commit (in your fork) with fetchpatch.
This patch shouldn't be upstreamed. I've talked to the developer about this
and not using the default magic database was intended to make it easier to
package on Windows.
…On Thu, Dec 10, 2020, 3:42 PM Guillaume Girol ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In pkgs/applications/editors/imhex/default.nix
<#106576 (comment)>:
> + version = "1.6.0-git"; # adds install target
+
+ src = fetchFromGitHub {
+ owner = "WerWolv";
+ repo = "ImHex";
+ #rev = "v${version}";
+ rev = "e3b5a55eba51aaab1473abf4c2a40975ddd1e9fa";
+ sha256 = "1nfq2yy21chldkg79r9nnb8n7g9fma6c9fnfy7yqgsl6y7iffxqw";
+ };
+
+ hardeningDisable = [ "format" ];
+
+ nativeBuildInputs = [ cmake pkgconfig ];
+ buildInputs = [ nlohmann_json capstone file glfw3 glm jsoncpp llvm_9 openssl python38 ];
+
+ patches = [ ./default-db.patch ];
please submit the patch upstream, and fetch the corresponding commit (in
your fork) with fetchpatch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#106576 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB7X32LQADQKNOCZWF43VWDSUEXCPANCNFSM4UVKOVVA>
.
|
then it should be commented copiously. |
|
||
hardeningDisable = [ "format" ]; | ||
|
||
nativeBuildInputs = [ cmake pkgconfig ]; |
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.
nativeBuildInputs = [ cmake pkgconfig ]; | |
nativeBuildInputs = [ cmake pkg-config ]; |
{ stdenv, gcc10Stdenv, cmake, pkgconfig, nlohmann_json, capstone, file, glfw3, glm, jsoncpp, llvm_9, openssl, python38, fetchFromGitHub }: | ||
gcc10Stdenv.mkDerivation rec { |
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.
{ stdenv, gcc10Stdenv, cmake, pkgconfig, nlohmann_json, capstone, file, glfw3, glm, jsoncpp, llvm_9, openssl, python38, fetchFromGitHub }: | |
gcc10Stdenv.mkDerivation rec { | |
{ stdenv, cmake, pkg-config, nlohmann_json, capstone, file, glfw3, glm, jsoncpp, llvm_9, openssl, python3, fetchFromGitHub }: | |
stdenv.mkDerivation rec { |
substituteInPlace source/lang/preprocessor.cpp --replace 'include/' "${patterns}/includes/" | ||
''; | ||
|
||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
Also please change the version to 1.6.0 |
I marked this as stale due to inactivity. → More info |
Another attempt has been made at #184488 |
Motivation for this change
ImHex has been receiving a lot of attention the past few days, I know the author, and I wanted to try it.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)