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
insomnia-designer: init at 2020.4.1 #92093
Conversation
Hey, any particular reason this is built from a |
As I answer to @Sohalt : I took the same approach as the currently available insomnia. I imagine is feasible to build it from source but I didn't try it and this worked well enough for me. |
@Sohalt @puzzlewolf will building from source be a requirement for this package to be merged? I wonder how |
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.
No, building from source is not required, but it's usually preferred.
I don't have much experience with this kind of package, sry. But the the derivation is very similar to insomnia
, and that got merged. So I guess this is ok :)
Result of nixpkgs-review pr 92093
1
1 package built:
- insomnia-designer
004b19d
to
9a1c5b1
Compare
@puzzlewolf I added myself as maintainer and fixed metadata. |
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.
looks good to me now 👍
9a1c5b1
to
506492d
Compare
Update to 2020.4.1. I've been successfully running this package since several months. |
url = | ||
"https://github.com/Kong/insomnia/releases/download/designer@${version}/Insomnia.Designer-${version}.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/Kong/insomnia/releases/download/designer@${version}/Insomnia.Designer-${version}.deb"; | |
url = "https://github.com/Kong/insomnia/releases/download/designer@${version}/Insomnia.Designer-${version}.deb"; |
nativeBuildInputs = | ||
[ autoPatchelfHook dpkg makeWrapper gobject-introspection wrapGAppsHook ]; |
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 = | |
[ autoPatchelfHook dpkg makeWrapper gobject-introspection wrapGAppsHook ]; | |
nativeBuildInputs = [ autoPatchelfHook dpkg makeWrapper gobject-introspection wrapGAppsHook ]; |
}; | ||
|
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.
}; | |
}; |
description = "The most intuitive cross-platform REST API Client"; | ||
license = licenses.mit; | ||
platforms = [ "x86_64-linux" ]; | ||
maintainers = with maintainers; [ markus1189 babariviere ]; |
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 add yourself as maintainer and not other people.
Result of 1 package built:
|
@Pablo1107 Would you be willing to implement @SuperSandro2000's suggestions? I have not reviewed myself yet, but this discussion looks to me like when you implement them it'll become possible to land this PR, and hopefully the suggestions should be simple to implement :) (I'd just go ahead, click “Commit suggestion” and land myself right now, but I don't want to add you as a maintainer if you don't actually want to be :/) |
@Ekleog yes, I'll see to push those changes so you can merge it. |
I marked this as stale due to inactivity. → More info |
Motivation for this change
Add
insomnia-designer
.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)