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
rendersvg: init at 0.7.0 #52433
rendersvg: init at 0.7.0 #52433
Conversation
@flosse are you able to address the comments above and continue with this PR? |
80a8770
to
4af947b
Compare
... also updated to |
@flosse looks like issue with |
To be honest I have not idea what's the difference between |
@flosse Yeah the documentation for
So build system tools like |
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.
I've left some comments I hope you will find helpful. I'm not overly familiar with rust
but seeing as @fpletz had no complaints I'll assume everything I have not commented on is in good order.
fetchFromGitHub, | ||
rustPlatform, | ||
pango, | ||
gdk_pixbuf, |
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.
gdk_pixbuf
-> gdk-pixbuf
}: | ||
|
||
rustPlatform.buildRustPackage rec { | ||
name = "rendersvg-${version}"; |
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 use pname = "rendersvg";
as name = "${pname}-${version}";
is already implied.
|
||
buildInputs = [ | ||
pango | ||
gdk_pixbuf |
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.
gdk_pixbuf
-> gdk-pixbuf
|
||
meta = with stdenv.lib; { | ||
description = "A SVG rendering application based on resvg"; | ||
homepage = https://github.com/RazrFalcon/resvg/tree/master/tools/rendersvg; |
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 quote.
homepage = https://github.com/RazrFalcon/resvg/tree/master/tools/rendersvg; | ||
license = with licenses; [ mpl20 ]; | ||
maintainers = with maintainers; [ flosse ]; | ||
platforms = [ "x86_64-linux" ]; |
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.
After a quick glance at the site I would have thought this to be platforms.all
.
@flosse do you currently have any time/motivation/ability to continue with this PR? |
Thank you for your contributions.
|
I marked this as stale due to inactivity. → More info |
Motivation for this change
rendersvg
enables me to render SVG-Files without Inkscape :)Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)