Skip to content

Remove -Zno-landing-pads flag#428

Merged
marco-c merged 4 commits intomozilla:masterfrom
wusyong:nolandingpads
May 6, 2020
Merged

Remove -Zno-landing-pads flag#428
marco-c merged 4 commits intomozilla:masterfrom
wusyong:nolandingpads

Conversation

@wusyong
Copy link
Contributor

@wusyong wusyong commented May 2, 2020

Fix #427
Remove -Zno-landing-pads in README instructions and CI files since this flag is no longer exist.

@calixteman
Copy link
Collaborator

@wusyong, could you update your patch to take into account: #427 (comment) ?
Thanks.

Copy link
Collaborator

@marco-c marco-c left a comment

Choose a reason for hiding this comment

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

Even if we don't have Rust doc tests, please add RUSTDOCFLAGS too so users of grcov know that it's needed.

@marco-c marco-c merged commit 083b395 into mozilla:master May 6, 2020
@wusyong wusyong deleted the nolandingpads branch May 6, 2020 15:20
Minoru added a commit to newsboat/newsboat that referenced this pull request May 8, 2020
`-Zno-landing-pads` got deprecated. This commit replaces it with options
suggested by grcov project: mozilla/grcov#428

Fixes #916.
Minoru added a commit to newsboat/newsboat that referenced this pull request May 11, 2020
`-Zno-landing-pads` got deprecated. This commit replaces it with options
suggested by grcov project: mozilla/grcov#428

Fixes #916.
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.

README.md: Command line example does not work with rustc nightly 2020-04-30

3 participants