Skip to content

/rust

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.

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

Regression in test suite of probor-0.2.1 in Rust 1.15 #38715

Closed
brson opened this Issue Dec 30, 2016 · 1 comment

Comments

Assignees

@jseyfried jseyfried

Labels
A-macros T-compiler regression-from-stable-to-beta
Projects
None yet
Milestone
No milestone
2 participants
@brson @jseyfried
@brson
Copy link
Contributor

brson commented Dec 30, 2016

https://github.com/tailhook/probor at d0be60ec37ded2eee0f851887e2d7c06fe2e7021

101 brian@ip-10-145-43-250:~/dev/probor/rust⟫ rustc +beta -Vv
rustc 1.15.0-beta.1 (d9a0f0df7 2016-12-19)
binary: rustc
commit-hash: d9a0f0df7051c603011d6b60fbdd155318fc47f3
commit-date: 2016-12-19
host: x86_64-unknown-linux-gnu
release: 1.15.0-beta.1
LLVM version: 3.9

brian@ip-10-145-43-250:~/dev/probor/rust⟫ cargo +beta test
   Compiling byteorder v1.0.0
   Compiling libc v0.2.18
   Compiling byteorder v0.5.3
   Compiling cbor-codec v0.5.0
   Compiling probor v0.2.1 (file:///mnt2/dev/probor/rust)
error[E0428]: a macro named `_probor_pattern` has already been defined in this module

error: aborting due to previous error

Build failed, waiting for other jobs to finish...
error: Could not compile `probor`.

cc @jseyfried I haven't seen one quite like this.

cc @tailhook

@brson brson added A-macros regression-from-stable-to-beta T-compiler labels Dec 30, 2016

@jseyfried jseyfried self-assigned this Dec 30, 2016

@jseyfried jseyfried referenced this issue Jan 3, 2017

Merged

Fix regression with duplicate `#[macro_export] macro_rules!` #38793

@jseyfried

This comment has been minimized.

Sign in to view
Copy link
Contributor

jseyfried commented Jan 3, 2017

Fixed in #38793.

bors added a commit that referenced this issue Jan 4, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
485e2a4

bors added a commit that referenced this issue Jan 5, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
f7f5950

bors added a commit that referenced this issue Jan 5, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
c711172

bors added a commit that referenced this issue Jan 5, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
4b0378d

bors added a commit that referenced this issue Jan 5, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
131d5ce

bors added a commit that referenced this issue Jan 6, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
0fc0e17

bors added a commit that referenced this issue Jan 6, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
c4065eb

bors added a commit that referenced this issue Jan 6, 2017

@bors
Auto merge of #38793 - jseyfried:fix_macro_export_duplicates, r=nrc 
Fix regression with duplicate `#[macro_export] macro_rules!`

Fixes #38715.
r? @nrc
Loading status checks…
e7907a9

@bors bors closed this in #38793 Jan 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.