babashka: 0.0.71 -> 0.0.78 #83776
babashka: 0.0.71 -> 0.0.78 #83776
Conversation
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/trouble-with-creating-first-nix-package/6479/4 |
|
Why is it failing the Borg? |
Co-Authored-By: Martin Milata <martin@martinmilata.cz>
Co-Authored-By: Anderson Torres <torres.anderson.85@protonmail.com>
Co-Authored-By: Anderson Torres <torres.anderson.85@protonmail.com>
ofborg
bot
added
the
2.status: merge conflict
|
Maybe the location should be Anyway: looking forward to this package! |
ofborg
bot
removed
the
2.status: merge conflict
| license = licenses.epl10; | ||
| platforms = graalvm8.meta.platforms; | ||
| maintainers = with maintainers; [ DerGuteMoritz ]; | ||
| maintainers = with maintainers; [ bhougland jlesquembre DerGruteMoritz ]; |
There was a problem hiding this comment.
the spelling of this maintainer is DerGuteMoritz
|
Not sure why eval fails by borg, but for me this PR builds fine and works as expected. |
I am currently using the software as well, works fine. |
|
I would really like to see the latest babashka in NixOS, I would like to propose the following: I resubmit a new PR, basically re-doing #83886 with the, I think, proper location of This PR has all its commits removed sans adding @bhougland18 to the maintainer list which can get merged then. Can we agree on this course of action @AndersonTorres and @bhougland18? |
|
I agree with this course of action. This is my first commit and I really
don't know where to go from here as I don't understand why it is failing.
…On Fri, Apr 3, 2020 at 6:14 AM Benjamin Andresen ***@***.***> wrote:
I would really like to see the latest babashka in NixOS, I would like to
propose the following:
I resubmit a new PR, basically re-doing #83886
<#83886> with the, I think, proper
location of pkgs/development/interpreters/clojure/babashka.nix, based on
master. I would use this PR's meta.description and meta.longDescription.
This PR has all its commits removed sans adding @bhougland18
<https://github.com/bhougland18> to the maintainer list which can get
merged then.
Then I will add @DerGuteMoritz <https://github.com/DerGuteMoritz>,
@bhougland18 <https://github.com/bhougland18> & myself to the maintainers
of this package in my new PR, so we can all keep an eye on new releases,
graalvm updates affecting the package, etc.
Can we agree on this course of action @AndersonTorres
<https://github.com/AndersonTorres> and @bhougland18
<https://github.com/bhougland18>?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#83776 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AGZANCAT6HZWOGLJ2V7JW33RKWZJJANCNFSM4LWS3FTA>
.
|
|
OK for me. In fact I was uploading a new diff for it now. P.S.: I am a bit puristic, and I would like to ast two things:
|
|
"Superceded" by #84284 |
Janik-Haag
added
the
12. first-time contribution
Motivation for this change
Add new package to repository. This is a small command line utility that allows one to use a subset of the clojure programming language in their shell commands.
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)