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

Add support for "prepare" script in descriptor #123

Closed
wants to merge 1 commit into from
Closed

Add support for "prepare" script in descriptor #123

wants to merge 1 commit into from

Conversation

laanwj
Copy link
Contributor

@laanwj laanwj commented Jun 17, 2016

The prepare script consists of commands that are run as root on the target after installing the necessary packages, but before running the build script.

This is necessary to fix up an ARM build system issue on Ubuntu Trusty (see bitcoin/bitcoin#8212).

The prepare script consists of commands that are run as root
on the target after installing the necessary packages, but before
running the build script.

This is necessary to fix up an ARM build system issue on Ubuntu
Trusty (see bitcoin/bitcoin#8212).
@laanwj
Copy link
Contributor Author

laanwj commented Jun 17, 2016

I see an --allow-sudo flag was added recently in e4b4780. This would allow for the same result. I worked on this without realizing that.

However I slightly prefer this solution as it enforces privilege separation, by not allowing everything in the entire build script to use sudo, but restrict this to a few lines.

@theuni
Copy link
Contributor

theuni commented Jul 7, 2016

ACK 46c6539

@devrandom
Copy link
Owner

OK. I'll merge this, but I'm away from my key so it will have to wait until next week.

@laanwj
Copy link
Contributor Author

laanwj commented Jul 8, 2016

@devrandom Thanks for your reply.
We only need this functionality temporarily, and @theuni proposed a workable hack around this at the meeting - just have separate descriptors for x86 and ARM - until Ubuntu fixes it upstream.
So this may not be necessary after all - I'm going to close this for now.

@laanwj laanwj closed this Jul 8, 2016
@devrandom
Copy link
Owner

Thank you for the update.

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.

3 participants