I'd like to figure out what generator code is causing clippy errors. Is all that's required to repro this an update to 1.61, Edition 2021, and run `clippy`?
Thanks, Allen On Fri, Jun 24, 2022 at 10:49 AM Jiayu Liu <ji...@hey.com.invalid> wrote: > Sorry, wrong link. It should be > https://github.com/apache/thrift/pull/2527 > > On June 24, 2022, jiayu liu <ji...@hey.com.invalid> wrote: > > I've long been wishing to upgrade the rust lib and gen code to latest > > version and have a pull request up at > > https://github.com/apache/thrift/pull/2623/files however i do need to > > fix some generator code that spills out clippy-failing code first... > > > > About the quick fix, i think dropping `--all` flag should be easiest > > and > > minimally disruptive to how clippy works? > > > > On June 24, 2022, Allen George <allen.geo...@gmail.com> wrote: > > > Sorry Jens - > > > > > > I know I’ve a backlog of Rust stuff to look at. I can take a look. > > > Just scanning the error text above it sounds like a dependency may > > > have updated somewhere causing clippy to fail. The immediate > > > workaround is to remove the clippy call; the more complete one for > > me > > > is to take a look and figure out what changed, and why (I’d rather > > not > > > disable clippy). > > > > > > Allen > > > ________________________________ > > > From: Jens Geyer <je...@apache.org> > > > Sent: Friday, June 24, 2022 3:08:25 AM > > > To: Thrift-Dev <dev@thrift.apache.org> > > > Subject: Rust - attributes are not yet allowed on `if` expressions > > > > > > Hi, > > > > > > fresh day, fresh problems. > > > > > > > > > /root/.cargo/bin/cargo fmt --all -- --check > > > /root/.cargo/bin/cargo clippy --all -- -D warnings > > > Compiling libc v0.2.126 > > > Compiling autocfg v1.1.0 > > > Compiling log v0.4.17 > > > Checking cfg-if v1.0.0 > > > Checking integer-encoding v3.0.4 > > > error: attributes are not yet allowed on `if` expressions > > > --> > > > /root/.cargo/registry/src/github.com-1ecc6299db9ec823/integer- > > > encoding-3.0.4/src/fixed.rs:57:17 > > > | > > > 57 | #[cfg(target_endian = "big")] > > > | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > > ... > > > 90 | impl_fixedint!(usize); > > > | ---------------------- in this macro invocation > > > > > > > > > and a whole bunch more like that. > > > > > > Anything I could do? Anyone else? > > > > > > Full log > > > https://app.travis-ci.com/github/apache/thrift/jobs/574574155 > > > > > > Have fun, > > > JensG >