Skip to content

Conversation

@JeremyRand
Copy link
Contributor

2cecb24 changed "precise" to "trusty" in the shell command, but didn't change the text above it accordingly. This commit fixes that.

bitcoin@2cecb24 changed "precise" to "trusty" in the shell command, but didn't change the text above it accordingly.  This commit fixes that.
@fanquake
Copy link
Member

fanquake commented Apr 3, 2016

ACK 190c1e2

@jonasschnelli
Copy link
Contributor

ACK 190c1e2

@laanwj
Copy link
Member

laanwj commented Apr 3, 2016

There are various references to precise in doc/gitian-building.md as well, may be good to change them at the same time.

@maflcko
Copy link
Member

maflcko commented Apr 3, 2016

good to change them at the same time.

Or remove them (replace with something general) so they never need to be changed again.

@laanwj
Copy link
Member

laanwj commented Apr 3, 2016

Or remove them (replace with something general) so they never need to be changed again.

As parts of those are part of commands that have to be executed literally that's much more work, I don't think we can ask that of the OP.

@maflcko
Copy link
Member

maflcko commented Apr 3, 2016

Oh, all of them are part of a command, so what I said does not apply to this particular pull. ☁️

@paveljanik
Copy link
Contributor

ACK 190c1e2

@btcdrak
Copy link
Contributor

btcdrak commented Apr 5, 2016

ACK 190c1e2

@laanwj laanwj merged commit 190c1e2 into bitcoin:master Apr 5, 2016
laanwj added a commit that referenced this pull request Apr 5, 2016
190c1e2 Doc: change Precise to Trusty in gitian-building.md (JeremyRand)
@JeremyRand JeremyRand deleted the patch-1 branch April 11, 2016 02:25
codablock pushed a commit to codablock/dash that referenced this pull request Dec 19, 2017
190c1e2 Doc: change Precise to Trusty in gitian-building.md (JeremyRand)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants