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
axoloti: init at 1.0.12 #28328
axoloti: init at 1.0.12 #28328
Conversation
Why I am the author of this commit :) |
@@ -5,19 +5,19 @@ | |||
, gtk2, hunspell, icu, libevent, libjpeg, libnotify | |||
, libstartup_notification, libvpx, makeWrapper, mesa | |||
, nspr, nss, pango, perl, python, libpulseaudio, sqlite | |||
, unzip, xlibs, which, yasm, zip, zlib | |||
, unzip, xlibs, which, yasm, zip, zlib, gcc |
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 make this a separate commit.
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.
indeed, this does not belong there, must be a merge issue since I didn't modify this myself -- will check
@@ -60,7 +60,7 @@ in pythonPackages.buildPythonApplication rec { | |||
owner = "foosel"; | |||
repo = "OctoPrint"; | |||
rev = version; | |||
sha256 = "06l8khbq3waaaa4cqpv6056w1ziylkfgzlb28v30i1h234rlkknq"; | |||
sha256 = "1hci8cfmbzcghla1vmrcn6zicm8nj50drm7gp2hkr0drglq5fgr2"; |
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 make this a separate commit.
@@ -4,7 +4,7 @@ | |||
}: | |||
|
|||
let | |||
version = "4.3.0"; | |||
version = "4.4.0"; |
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 make this a separate commit.
cp -r doc firmware chibios platform_linux CMSIS *.txt $out/share/axoloti/ | ||
install -vD dist/Axoloti.jar $out/share/axoloti/ | ||
|
||
cat > $out/bin/axoloti <<EOF |
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.
makeWrapper is preferred here, see https://nixos.org/nixpkgs/manual/#sec-language-java as an example
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.
The package itself looks good to me.
@TealG are you still working on this? |
lib/maintainers.nix
Outdated
@@ -590,6 +590,7 @@ | |||
taku0 = "Takuo Yonezawa <mxxouy6x3m_github@tatapa.org>"; | |||
tari = "Peter Marheine <peter@taricorp.net>"; | |||
tavyc = "Octavian Cerna <octavian.cerna@gmail.com>"; | |||
TealG = "Teal Gaure <~@Teal.Gr>" |
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.
missing trailing semicolon.
@Mic92 fixed |
Did you close the pr by accident? |
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)