Skip to content

Conversation

@fanquake
Copy link
Member

Otherwise we'll just override other flags passed in (i.e msan).

Should fix https://cirrus-ci.com/task/6598922274078720?logs=ci#L3661.

Otherwise we'll just override other flags passed in (i.e msan).
@DrahtBot
Copy link
Contributor

DrahtBot commented Jan 30, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK TheCharlatan, achow101

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Copy link
Contributor

@sedited sedited left a comment

Choose a reason for hiding this comment

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

ACK 56a03f1

Also checked that this is done correctly for the other $(package)_*flags.

@achow101
Copy link
Member

ACK 56a03f1

@achow101 achow101 merged commit ceb74b8 into bitcoin:master Jan 30, 2023
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jan 30, 2023
56a03f1 depends: ensure we are appending to sqlite cflags (fanquake)

Pull request description:

  Otherwise we'll just override other flags passed in (i.e msan).

  Should fix https://cirrus-ci.com/task/6598922274078720?logs=ci#L3661.

ACKs for top commit:
  achow101:
    ACK 56a03f1
  TheCharlatan:
    ACK 56a03f1

Tree-SHA512: 5890018cfc5deaef18b0f01a3a0396f803e97f9a9785bf6873ef48bc13b74b644315f0f29cf11d3522964a6396f74e1f080bb4e412bc302956a651fed28b27df
@fanquake fanquake deleted the msan_flags_missing branch January 31, 2023 09:29
@bitcoin bitcoin locked and limited conversation to collaborators Jan 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants