Skip to content

build, doc: Fix configure script output indentation and typos #21292

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 24, 2021
Merged

build, doc: Fix configure script output indentation and typos #21292

merged 1 commit into from
Feb 24, 2021

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Feb 24, 2021

This PR is follow up of #16546, that breaks the configure script output indentation for gui/qt/qr lines:

Options used to compile and link:
  external signer = no
  multiprocess    = no
  with libs       = yes
  with wallet     = yes
    with sqlite   = yes
    with bdb      = yes
    with gui / qt = yes
  with qr         = yes
  with zmq        = yes
  with test       = yes
...

With this PR:

Options used to compile and link:
  external signer = no
  multiprocess    = no
  with libs       = yes
  with wallet     = yes
    with sqlite   = yes
    with bdb      = yes
  with gui / qt   = yes
    with qr       = yes
  with zmq        = yes
  with test       = yes
...

Also typos are fixed.

@hebasto
Copy link
Member Author

hebasto commented Feb 24, 2021

cc @Sjors

vasild
vasild approved these changes Feb 24, 2021
View reviewed changes
Copy link
Contributor

@vasild vasild left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 8b08d0f

@Sjors
Copy link
Member

Sjors commented Feb 24, 2021

Oops, I remember adjusting that 100 times during rebases, must have broken it at some point.

utACK 8b08d0f

@maflcko maflcko merged commit b54a10e into bitcoin:master Feb 24, 2021
@hebasto hebasto deleted the 210224-layout branch February 24, 2021 14:38
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Feb 24, 2021
Merge bitcoin#21292: build, doc: Fix configure script output indentat…
25d29ed 
…ion and…
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@vasild vasild vasild approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
Build system Docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@hebasto @Sjors @vasild @maflcko @DrahtBot