Skip to content
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

rotki-bin: init at 1.32.1 #498

Merged
merged 1 commit into from
Mar 9, 2024
Merged

Conversation

mitchmindtree
Copy link
Contributor

I had a quick go at building it from source following the guide, then decided on just wrapping the appimage after running into some python build issues related to the ruff linter that was being invoked in a build phase for some reason. I'm sure it's doable with some patching if someone's keen enough, but this at least makes the package available for now.

I had a quick go at building it from source following [the guide]
[1], then decided on just wrapping the appimage after running into
some python build issues related to the `ruff` linter that was being
invoked in a build phase for some reason. I'm sure it's doable with some
patching if someone's keen enough, but this at least makes the package
available for now.

[1]: https://rotki.readthedocs.io/en/latest/installation_guide.html#build-from-source
@mitchmindtree mitchmindtree marked this pull request as ready for review March 9, 2024 07:16
@aldoborrero
Copy link
Collaborator

I'll build it from sources, but this first version is good enough :)

Copy link
Collaborator

@aldoborrero aldoborrero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aldoborrero aldoborrero merged commit a6b05fc into nix-community:main Mar 9, 2024
75 checks passed
@jhvst
Copy link
Contributor

jhvst commented Mar 10, 2024

The open PR on nixpkgs has some extra packages added to path: https://github.com/NixOS/nixpkgs/pull/281781/files#diff-43d2233c3f4a035c0efd6110ee4189d28f2260ae102c8a3a2ca12ace792b73a4R44-R47

Any idea if these are necessary?

@mitchmindtree
Copy link
Contributor Author

Ahh I didn't see that PR @jhvst - I just checked that the application GUI ran and it was looking good. I'm unsure whether or not there are any deeper runtime issues that might be caused without those on linux?

Either way, that nixpkgs PR appears to also support darwin, so would be great to see that extra support landed either in nixpkgs or here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants