Skip to content

Unable to compile syntex_syntax using Rust 1.41 #68729

Description

@yaa110

Unfortunately, syntex_syntax is not maintained and is not compiled using rust 1.41:

   Compiling syntex_syntax v0.58.1
error[E0423]: expected function, tuple struct or tuple variant, found struct `ast::Name`
   --> /root/.cargo/registry/src/github.com-1ecc6299db9ec823/syntex_syntax-0.58.1/src/symbol.rs:146:27
    |
146 |                       name: ast::Name($index),
    |                             ^^^^^^^^^
...
165 | / declare_keywords! {
166 | |     // Invalid identifier
167 | |     (0,  Invalid,        "")
168 | |
...   |
231 | |     (56, CrateRoot, "{{root}}")
232 | | }
    | |_- in this macro invocation

   Compiling clap v2.33.0
   Compiling nom v3.2.1
   Compiling parking_lot v0.10.0
error: aborting due to previous error

For more information about this error, try `rustc --explain E0423`.
error: could not compile `syntex_syntax`.
warning: build failed, waiting for other jobs to finish...
error: build failed
make[1]: *** [objs/Makefile:1314: cargo] Error 101
# rustc --version
rustc 1.41.0 (5e1a79984 2020-01-27)

# uname -a
Linux 6b52ecd4014b 5.3.18-1-MANJARO #1 SMP PREEMPT Wed Dec 18 18:34:35 UTC 2019 x86_64 GNU/Linux

Activity

  1. added
    regression-from-stable-to-stablePerformance or correctness regression from one stable version to another.
    T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.
    on Feb 1, 2020
  2. jonas-schievink commented on Feb 1, 2020

    @jonas-schievink
    Contributor

    It builds fine on 1.40.

    FWIW this is not the most up to date version, which is 0.59.1 and builds fine on 1.41.

  3. pnkfelix commented on Feb 6, 2020

    @pnkfelix
    Contributor

    Hmm I hadn't noticed that syntex_syntax had been updated in that manner; I just saw that it has not been maintained in three years (see their readme)

    This problem is expected fallout from PR #65785

  4. pnkfelix commented on Feb 6, 2020

    @pnkfelix
    Contributor

    triage: P-low, with intent to close. Leaving nominated because I want to at least discuss 1. whether we need/want to collect data on instances of such breakage, and 2. whether we were too aggressive in deciding that going straight to an error without some mitigating mechanism was too aggressive.

  5. yaa110 commented on Feb 6, 2020

    @yaa110
    Author

    syntex_syntax was used in older versions of bindgen crate and is not maintained. So, this issue could be closed. However, this kind of known breaking changes should be declared in blog posts of rustlang update.

  6. pnkfelix commented on Mar 4, 2020

    @pnkfelix
    Contributor

    Removing nomination label and closing.

    As I wrote on zulip:

    this was not such a big deal in the end. The main question I would want resolved is whether we actually documented this breaking change somewhere visible (release notes from release blog post or something similar). But that does not need to be a discussion during the synchronous T-compiler meeting.

  7. added a commit that references this issue on Apr 11, 2020
  8. Mawfyy commented on Jan 23, 2024

    @Mawfyy

    I can reproduce this error in rust 1.73.0, there's any solution to this?

  9. ShipmasterKyle commented on Jun 19, 2024

    @ShipmasterKyle

    This can repoduced on rust 1.76.0 as well. Is there a solution?

    We could attempt syntex_sytntax to an older version but I'm not sure that it'll solve the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P-lowLow priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.regression-from-stable-to-stablePerformance or correctness regression from one stable version to another.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions