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

Created flake for project #40

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

ink-pot-monkey
Copy link

Pretty simple project to package as everything is already contained within the repository.

There are some apps that load the browser version and some addition checks that make use of the general tests in the project

@ink-pot-monkey ink-pot-monkey deleted the ngi-nix branch August 18, 2021 16:46
@ink-pot-monkey
Copy link
Author

Sorry this was too eager, need to get this check by the ngi review team

@ink-pot-monkey ink-pot-monkey restored the ngi-nix branch August 18, 2021 16:54
@Ayms
Copy link
Owner

Ayms commented Aug 18, 2021

Yes super simple to package, the only external thing is the third party check for the code loaded https://github.com/Ayms/bitcoin-transactions/blob/master/hash.txt (not so external right now because it is myself)

ink-pot-monkey and others added 15 commits August 19, 2021 08:45
1. src now equals lib.cleanSource ./.
2. supportedSystems expanded to any system that can run a browser
3. combined apps into single script with flags
4. created nixosModule which creates a .desktop file
5. removed unused version var
6. enclose bash vars in ""
Co-authored-by: Aaron Janse <aaron@ajanse.me>
@ink-pot-monkey ink-pot-monkey reopened this Oct 8, 2021
@ink-pot-monkey
Copy link
Author

Hi,

This flake has been completed and integrates this project into the Nix ecosystem :0

It builds a package and exposes a shell command
nix run github:Ayms/bitcoin-transactions which opens the project with xdg-open

it also exposes a flag (clear || c) which runs the walletclear.html

and makes it installable as a desktop item on NixOS through a nixosModule (which is why the png has been added to the repo as well)

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.

None yet

2 participants