Skip to content
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

Can't compile with recent nightly #69

Closed
killercup opened this issue May 25, 2015 · 5 comments
Closed

Can't compile with recent nightly #69

killercup opened this issue May 25, 2015 · 5 comments

Comments

@killercup
Copy link
Member

Maybe you already know this, but it couldn't find an issue. (I'm compiling this project but it shouldn't matter.)

$ cargo run --features "dev" --release
   Compiling clippy v0.0.4 (https://github.com/Manishearth/rust-clippy.git#62f0fe0c)
/Users/pascal/.multirust/toolchains/nightly/cargo/git/checkouts/rust-clippy-2d80226e8d82d2cc/master/src/ptr_arg.rs:30:11: 30:39 error: this pattern has 5 fields, but the corresponding variant has 6 fields [E0023]
/Users/pascal/.multirust/toolchains/nightly/cargo/git/checkouts/rust-clippy-2d80226e8d82d2cc/master/src/ptr_arg.rs:30       if let &ItemFn(ref decl, _, _, _, _) = &item.node {
                                                                                                                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
note: in expansion of if let expansion
/Users/pascal/.multirust/toolchains/nightly/cargo/git/checkouts/rust-clippy-2d80226e8d82d2cc/master/src/ptr_arg.rs:30:3: 32:4 note: expansion site
/Users/pascal/.multirust/toolchains/nightly/cargo/git/checkouts/rust-clippy-2d80226e8d82d2cc/master/src/ptr_arg.rs:30:11: 30:39 help: run `rustc --explain E0023` to see a detailed explanation
error: aborting due to previous error
Could not compile `clippy`.

To learn more, run the command again with --verbose.
$ rustc --version --verbose
rustc 1.2.0-nightly (7c4eedc21 2015-05-25) (built 2015-05-24)
binary: rustc
commit-hash: 7c4eedc21e609446671cc838d08473421917c353
commit-date: 2015-05-25
build-date: 2015-05-24
host: x86_64-apple-darwin
release: 1.2.0-nightly
@llogiq
Copy link
Contributor

llogiq commented May 25, 2015

Yep, that's the problem with using unstable APIs. All breakage that results is ours to fix.

@killercup
Copy link
Member Author

the problem

The challenge! :)

The signature of ItemFn was changed in rust-lang/rust@af37957. Looks like this can be fixed with an additional , _ (the third parameter now defines Constness).

@llogiq
Copy link
Contributor

llogiq commented May 25, 2015

Yes, but I think we should use conditional compilation to avoid breaking previous builds.

@Manishearth
Copy link
Member

Nah; lockfiles handle that already. We should stick to nightlies and use travis to find out when we need to fix it.

@llogiq
Copy link
Contributor

llogiq commented May 25, 2015

Ok then.

llogiq added a commit that referenced this issue May 25, 2015
llogiq added a commit that referenced this issue May 25, 2015
@llogiq llogiq closed this as completed May 26, 2015
yaahc pushed a commit to yaahc/rust-clippy that referenced this issue Mar 14, 2019
…lexcrichton

Closes rust-lang#69

Adds a new syntax for the `build = ` command:

```
build = [
    ["./configure"],
    ["make", "lib/libovr.a"],
    ["/bin/mkdir", "target"],
    ["/bin/cp", "lib/libovr.a", "target/"]
]
```

The current syntax `build = "make"` still works.

`build = [ "a", "b" ]` is forbidden because it is ambiguous.
toidiu pushed a commit to toidiu/rust-clippy that referenced this issue Jan 13, 2023
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

No branches or pull requests

3 participants