BitBoxSwiss / bitbox02-firmware

Firmware code of the BitBox02 hardware wallet
https://bitbox.swiss/bitbox02
Apache License 2.0
217 stars 81 forks source link

verifying firmware for Multi v9.15.0 and adding assertation #1134

Closed guggerf closed 9 months ago

guggerf commented 9 months ago

My GPG pub key is already in PR #1133

benma commented 9 months ago

@guggerf I think something went wrong with the rebase. There are now 5 commits instead of one. You can just apply your original commit on top of origin/master, or rebase it on origin/master., and force push it onto the same branch in your fork. Let me know if you need help.

guggerf commented 9 months ago

@guggerf I think something went wrong with the rebase. There are now 5 commits instead of one. You can just apply your original commit on top of origin/master, or rebase it on origin/master., and force push it onto the same branch in your fork. Let me know if you need help.

Hmm I thought I did this. Well it seams that I need help ;-)

thisconnect commented 9 months ago

@guggerf I usually have the original as my fork and the original repo as "upstream" remote. so my git remote -v shows:

origin  git@github.com:thisconnect/bitbox02-firmware.git (fetch)
origin  git@github.com:thisconnect/bitbox02-firmware.git (push)
upstream    https://github.com/digitalbitbox/bitbox02-firmware.git (fetch)
upstream    https://github.com/digitalbitbox/bitbox02-firmware.git (push)

In this case you could just do git fetch --all and then git rebase upstream/master assuming you are on the branch you want to rebase. You can also rebase a local branch without checking it out first with git rebase upstream/master verifyFirmwareMulti.

BTW. you don't have to use upstream, if you use origin as the original digitalbitbox/bitbox02-firmware repo then of corse you would use git rebase origin/master verifyFirmwareMulti, just whatever works for you.

Taking a look at the "network graph", it seems like there is merge commit, but I assume this would go away after you rebased. see:

Screen Shot 2023-10-02 at 11 15 12

I'd expect one branch off of digitalbitbox master and then your commit.

https://github.com/digitalbitbox/bitbox02-firmware/network

btw. you can check this also locally with git log --graph --oneline --all.

or add a shortcut to your gitconfig do this commend with git tree

# ~/.gitconfig
[alias]
tree = log --graph --oneline --all
# usage on command line:
git tree

I hope this helps.

guggerf commented 9 months ago

@thisconnect thank you so much. I think I made the mistake to rebase the main and not the branch. But now it should be correct with only one commit.

Hoppefully next time I get it correct the first time ;-)

thisconnect commented 9 months ago

LGTM 👍