-
Notifications
You must be signed in to change notification settings - Fork 83
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
Production-like build on dev PC #894
Conversation
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.
A little change proposition, looking awesome otherwise!!
tools/prod-like-build.sh
Outdated
docker run \ | ||
--mount type=bind,source="$(pwd)"/artifacts,target=/hostdir \ | ||
lib-ledger-core-build-env:latest \ | ||
/scripts/prod-like-build-inner.sh Debug |
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.
Maybe we want to duplicate this file to have dev-like-build or something?
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.
Moved this to command arguments. By default it builds Release
. Debug
if explicitly specified.
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.
Can you add a description on this script n the README please ? :)
Done |
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.
noice !
LedgerHQ/lib-ledger-core-build-env
ledger-lib-core.jar
Attn: default paths to jre libraries are different in prod and on typical dev machine.
LD_LIBRARY_PATH
must be modified accordingly to run WD:This PR depends on https://github.com/LedgerHQ/lib-ledger-core-build-env/pull/9