-
Notifications
You must be signed in to change notification settings - Fork 30
Feature/ledger signer #1944
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
base: master
Are you sure you want to change the base?
Feature/ledger signer #1944
Changes from 7 commits
a7af4f3
2235ee0
7c9c6e7
2b82eaf
1ae1ecc
4c711b6
b66c972
df98f63
1bcd08f
132141f
50d5293
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,7 +8,7 @@ on: | |
| branches: | ||
| - master | ||
| schedule: | ||
| - cron: '15 0 * * *' # every day at 00:15 UTC | ||
| - cron: "15 0 * * *" # every day at 00:15 UTC | ||
|
|
||
| env: | ||
| CARGO_TERM_COLOR: always | ||
|
|
@@ -59,7 +59,7 @@ jobs: | |
| - name: Update local dependency repositories | ||
| run: sudo apt-get update | ||
| - name: Install build dependencies | ||
| run: sudo apt-get install -yqq --no-install-recommends build-essential python3 python3-toml podman pkg-config libssl-dev | ||
| run: sudo apt-get install -yqq --no-install-recommends build-essential python3 python3-toml podman pkg-config libssl-dev libdbus-1-dev libusb-1.0-0-dev | ||
| - name: Install rust | ||
| run: curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain $(python3 ./build-tools/cargo-info-extractor/extract.py --rust-version) | ||
| - name: Build | ||
|
|
@@ -117,9 +117,9 @@ jobs: | |
| run_tests_on_trezor_preparation: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| # Note: we need to mimic the directory structure of the run_tests_on_trezor job, otherwise nextest | ||
| # will fail to execute archived tests. So we checkout the source code to "./mintlayer-core". | ||
| # (Also note that because of this the resulting path of the source dir will be "/.../mintlayer-core/mintlayer-core/mintlayer-core") | ||
| # Note: we need to mimic the directory structure of the run_tests_on_trezor job, otherwise nextest | ||
| # will fail to execute archived tests. So we checkout the source code to "./mintlayer-core". | ||
| # (Also note that because of this the resulting path of the source dir will be "/.../mintlayer-core/mintlayer-core/mintlayer-core") | ||
| - name: Checkout the core repository | ||
| uses: actions/checkout@v4 | ||
| with: | ||
|
|
@@ -130,7 +130,7 @@ jobs: | |
| run: sudo apt-get update | ||
|
|
||
| - name: Install build dependencies | ||
| run: sudo apt-get install -yqq --no-install-recommends build-essential python3 python3-toml pkg-config libssl-dev | ||
| run: sudo apt-get install -yqq --no-install-recommends build-essential python3 python3-toml pkg-config libssl-dev libdbus-1-dev libusb-1.0-0-dev | ||
|
|
||
|
Comment on lines
+133
to
134
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why? We don't even run any tests in this job.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just for compilation purposes |
||
| - name: Extract required info from Cargo.toml | ||
| id: extract_cargo_info | ||
|
|
@@ -227,13 +227,12 @@ jobs: | |
| # Note: since we haven't installed Cargo in this job, we have to execute "cargo-nextest nextest" | ||
| # instead of "cargo nextest". | ||
| - name: Run tests in the emulator | ||
| run: | ||
| nix-shell --run " | ||
| poetry run core/emu.py | ||
| --headless --quiet --temporary-profile | ||
| --mnemonic \"abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon about\" | ||
| --command env --chdir ../mintlayer-core | ||
| cargo-nextest nextest run --archive-file tests.tar.zst -j1 trezor_signer | ||
| run: nix-shell --run " | ||
| poetry run core/emu.py | ||
| --headless --quiet --temporary-profile | ||
| --mnemonic \"abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon about\" | ||
| --command env --chdir ../mintlayer-core | ||
| cargo-nextest nextest run --archive-file tests.tar.zst -j1 trezor_signer | ||
| " | ||
| working-directory: ./mintlayer-trezor-firmware | ||
| timeout-minutes: 10 | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -25,7 +25,7 @@ jobs: | |
| - name: Update local dependency repositories | ||
| run: sudo apt-get update | ||
| - name: Install dependencies | ||
| run: sudo apt-get install -yqq --no-install-recommends build-essential python3 python3-toml | ||
| run: sudo apt-get install -yqq --no-install-recommends build-essential python3 python3-toml libdbus-1-dev libusb-1.0-0-dev | ||
|
|
||
|
Comment on lines
+28
to
29
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why is this needed in
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just for compilation |
||
| - name: Install rust | ||
| run: curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain $(python3 ./build-tools/cargo-info-extractor/extract.py --rust-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.
Is this needed for ledger tests? I don't think you run them yet though.
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.
Just for compilation