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

Flashrom non-reproducible #931

Closed
tlaurion opened this issue Dec 14, 2020 · 3 comments
Closed

Flashrom non-reproducible #931

tlaurion opened this issue Dec 14, 2020 · 3 comments

Comments

@tlaurion
Copy link
Collaborator

tlaurion commented Dec 14, 2020

@Tonux599 @Thrilleratplay @MrChromebox : comparing master build vs local build (both debian-10 based)

user@x230-master:~$ git clone https://github.com/osresearch/heads heads2
user@x230-master:~$ cd heads2
user@x230-master:~/heads2$ make BOARD=x230-hotp-maximized CPUS=2
user@x230-master:~/heads2$ wget https://277-64810881-gh.circle-artifacts.com/0/build/x230-hotp-maximized/hashes.txt -O remote_hashes.txt
user@x230-master:~/heads2$ egrep '^[0-9a-f]{64}' build/x230-hotp-maximized/hashes.txt | while read line; do HASH_REF=$(echo $line|awk -F " " {'print $1'}); FILE_REF=$(echo $line|awk -F "/" {'print $NF'}); if ! grep -q "$HASH_REF" ./remote_hashes.txt; then echo "$FILE_REF doesn't match";fi; done
tools.cpio doesn't match
flashrom doesn't match
gawk doesn't match

Originally posted by @tlaurion in #927 (comment)

@MrChromebox
Copy link
Contributor

able to extract the flashrom binary and vbindiff it with your locally built one? comparing one built within Heads to my local host shows that there are some embedded library paths that differ, so not hugely surprising to see it differ between a local Heads instance and a CI one

@osresearch
Copy link
Collaborator

osresearch commented Mar 2, 2021

Is flashrom still a problem? I just did a build on Fedora 33 and Ubuntu 20.04 and produced the same results. It looks like this might have been fixed in aba13a9 by pinning the module. I do not see any library paths embedded in the executable with strings -a.

@tlaurion
Copy link
Collaborator Author

Not in latest tests! Thanks @osresearch !

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

No branches or pull requests

3 participants