Uh oh!
There was an error while loading. Please reload this page.
Run cfg-stripping on generic parameters before invoking derive macros - #76010
Conversation
rust-highfive
commented
Aug 28, 2020
(rust_highfive has picked a reviewer for you, use r? to override) |
1f0b101 to
2f36045Compareecstatic-morse
commented
Aug 29, 2020
petrochenkov
commented
Aug 30, 2020
@bors r+ |
bors
commented
Aug 30, 2020
📌 Commit 2f360451a2c0fee7c8e38dc1691702764ab623f0 has been approved by |
Fixesrust-lang#75930 This changes the tokens seen by a proc-macro. However, ising a `#[cfg]` attribute on a generic paramter is unusual, and combining it with a proc-macro derive is probably even more unusual. I don't expect this to cause any breakage.
2f36045 to
a97dcfaCompareAaron1011
commented
Aug 30, 2020
@bors r=petrochenkov |
bors
commented
Aug 30, 2020
📌 Commit a97dcfa has been approved by |
bors
commented
Aug 30, 2020
⌛ Testing commit a97dcfa with merge 4aa7f66777680318c913502adec36ce81363cd7d... |
bors
commented
Aug 30, 2020
💔 Test failed - checks-azure |
It's a debuginfo failure on macOS: This seems completely unrelated to my change. |
Aaron1011
commented
Aug 30, 2020
@bors retry |
…petrochenkov Run cfg-stripping on generic parameters before invoking derive macros Fixesrust-lang#75930 This changes the tokens seen by a proc-macro. However, ising a `#[cfg]` attribute on a generic paramter is unusual, and combining it with a proc-macro derive is probably even more unusual. I don't expect this to cause any breakage.
…petrochenkov Run cfg-stripping on generic parameters before invoking derive macros Fixesrust-lang#75930 This changes the tokens seen by a proc-macro. However, ising a `#[cfg]` attribute on a generic paramter is unusual, and combining it with a proc-macro derive is probably even more unusual. I don't expect this to cause any breakage.
joshtriplett
commented
Aug 31, 2020
Should this go through a crater run to confirm that? |
petrochenkov
commented
Aug 31, 2020
@joshtriplett |
petrochenkov
commented
Aug 31, 2020
@bors retry |
bors
commented
Sep 1, 2020
⌛ Testing commit a97dcfa with merge 9ecbc8c67feb853804555f8b7ed7cccddbebd270... |
rust-log-analyzer
commented
Sep 1, 2020
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
bors
commented
Sep 1, 2020
💔 Test failed - checks-actions |
petrochenkov
commented
Sep 1, 2020
@bors retry |
bors
commented
Sep 1, 2020
⌛ Testing commit a97dcfa with merge 022b4699a4384b68da645d21ea9de66c43c24d36... |
bors
commented
Sep 1, 2020
💔 Test failed - checks-azure |
Aaron1011
commented
Sep 1, 2020
@bors retry |
bors
commented
Sep 1, 2020
bors
commented
Sep 1, 2020
☀️ Test successful - checks-actions, checks-azure |
Fixes#75930
This changes the tokens seen by a proc-macro. However, ising a
#[cfg]attributeon a generic paramter is unusual, and combining it with a proc-macro
derive is probably even more unusual. I don't expect this to cause any
breakage.