On Tue, 19 Oct 2021 at 20:11, Khem Raj <raj.k...@gmail.com> wrote: > On Tue, Oct 19, 2021 at 10:44 AM Alexander Kanavin > <alex.kana...@gmail.com> wrote: > > > > I disagree. It carries a support burden, can be easily provided by the > host, and it’s purpose is source code formatting. It does not belong in > core. > > rustfmt is used by default in a lot of rust projects so its better to > include the minimal set of tools in core even though when there is no > sample in core using it. perhaps adding such a sample to oe-core > example could be better patch here. We have Joe as maintainer for this > recipe so its not a orphaned package. >
I can't agree with the 'it might be needed somehow, somewhere' reasoning to keep it (or anything else really). It only got added to core because someone copied recipes-examples/ wholesale from meta-rust. If it is actually needed somewhere, we can easily bring it back, but until then I would take it out. Especially considering it's a generated recipe (via cargo-bitbake) and it's not at all clear how to keep it current. Joe who? The listed maintainer is Randy. Alex > > > > > Alex > > > > On Tue 19. Oct 2021 at 18.34, Khem Raj <raj.k...@gmail.com> wrote: > >> > >> > >> > >> On 10/19/21 8:33 AM, Alexander Kanavin wrote: > >> > The recipes does not have any consumers, is host-only, > >> > is rather out of date, and is still present in meta-rust. > >> > > >> > Rust toolchain is now tested through other items, so I don't see > >> > a use case for it. For a target example there's rust-hello-world. > >> > > >> > >> its a tool. Think of strace, so there is value in keeping it in oe-core > >> so lets not drop it. > >> > >> > Signed-off-by: Alexander Kanavin <a...@linutronix.de> > >> > --- > >> > meta/conf/distro/include/maintainers.inc | 1 - > >> > meta/recipes-example/rustfmt/rustfmt_1.4.2.bb | 171 > ------------------ > >> > 2 files changed, 172 deletions(-) > >> > delete mode 100644 meta/recipes-example/rustfmt/rustfmt_1.4.2.bb > >> > > >> > diff --git a/meta/conf/distro/include/maintainers.inc > b/meta/conf/distro/include/maintainers.inc > >> > index baec2bef4d..d26d99a43d 100644 > >> > --- a/meta/conf/distro/include/maintainers.inc > >> > +++ b/meta/conf/distro/include/maintainers.inc > >> > @@ -662,7 +662,6 @@ RECIPE_MAINTAINER:pn-rt-tests = "Alexander > Kanavin <alex.kana...@gmail.com>" > >> > RECIPE_MAINTAINER:pn-ruby = "Ross Burton <ross.bur...@arm.com>" > >> > RECIPE_MAINTAINER:pn-run-postinsts = "Ross Burton < > ross.bur...@arm.com>" > >> > RECIPE_MAINTAINER:pn-rust = "Randy MacLeod < > randy.macl...@windriver.com>" > >> > -RECIPE_MAINTAINER:pn-rustfmt = "Randy MacLeod < > randy.macl...@windriver.com>" > >> > RECIPE_MAINTAINER:pn-rust-cross-${TARGET_ARCH} = "Randy MacLeod < > randy.macl...@windriver.com>" > >> > RECIPE_MAINTAINER:pn-rust-cross-canadian-${TRANSLATED_TARGET_ARCH} > = "Randy MacLeod <randy.macl...@windriver.com>" > >> > RECIPE_MAINTAINER:pn-rust-hello-world = "Randy MacLeod < > randy.macl...@windriver.com>" > >> > diff --git a/meta/recipes-example/rustfmt/rustfmt_1.4.2.bb > b/meta/recipes-example/rustfmt/rustfmt_1.4.2.bb > >> > deleted file mode 100644 > >> > index 2e4a47eb00..0000000000 > >> > --- a/meta/recipes-example/rustfmt/rustfmt_1.4.2.bb > >> > +++ /dev/null > >> > @@ -1,171 +0,0 @@ > >> > -# Auto-Generated by cargo-bitbake 0.3.13 > >> > -# > >> > -inherit cargo > >> > - > >> > -# If this is git based prefer versioned ones if they exist > >> > -# DEFAULT_PREFERENCE = "-1" > >> > - > >> > -# how to get rustfmt-nightly could be as easy as but default to a > git checkout: > >> > -# SRC_URI += "crate://crates.io/rustfmt-nightly/1.4.2" > >> > -SRC_URI += "git:// > github.com/rust-lang/rustfmt.git;protocol=https;nobranch=1" > >> > -SRCREV = "aeb3496f31a0dfa90fc511520d2023634e885260" > >> > -S = "${WORKDIR}/git" > >> > -CARGO_SRC_DIR = "" > >> > -EXCLUDE_FROM_WORLD = "1" > >> > - > >> > -# please note if you have entries that do not begin with crate:// > >> > -# you must change them to how that package can be fetched > >> > -SRC_URI += " \ > >> > - crate://crates.io/aho-corasick/0.7.4 \ > >> > - crate://crates.io/annotate-snippets/0.6.1 \ > >> > - crate://crates.io/ansi_term/0.11.0 \ > >> > - crate://crates.io/argon2rs/0.2.5 \ > >> > - crate://crates.io/arrayvec/0.4.11 \ > >> > - crate://crates.io/atty/0.2.13 \ > >> > - crate://crates.io/autocfg/0.1.5 \ > >> > - crate://crates.io/backtrace-sys/0.1.31 \ > >> > - crate://crates.io/backtrace/0.3.33 \ > >> > - crate://crates.io/bitflags/1.1.0 \ > >> > - crate://crates.io/blake2-rfc/0.2.18 \ > >> > - crate://crates.io/bstr/0.2.6 \ > >> > - crate://crates.io/bytecount/0.5.1 \ > >> > - crate://crates.io/byteorder/1.3.2 \ > >> > - crate://crates.io/c2-chacha/0.2.2 \ > >> > - crate://crates.io/cargo_metadata/0.8.1 \ > >> > - crate://crates.io/cc/1.0.38 \ > >> > - crate://crates.io/cfg-if/0.1.9 \ > >> > - crate://crates.io/clap/2.33.0 \ > >> > - crate://crates.io/cloudabi/0.0.3 \ > >> > - crate://crates.io/constant_time_eq/0.1.3 \ > >> > - crate://crates.io/crossbeam-channel/0.3.9 \ > >> > - crate://crates.io/crossbeam-deque/0.2.0 \ > >> > - crate://crates.io/crossbeam-epoch/0.3.1 \ > >> > - crate://crates.io/crossbeam-utils/0.2.2 \ > >> > - crate://crates.io/crossbeam-utils/0.6.6 \ > >> > - crate://crates.io/derive-new/0.5.7 \ > >> > - crate://crates.io/diff/0.1.11 \ > >> > - crate://crates.io/dirs-sys/0.3.3 \ > >> > - crate://crates.io/dirs/2.0.1 \ > >> > - crate://crates.io/either/1.5.2 \ > >> > - crate://crates.io/ena/0.13.0 \ > >> > - crate://crates.io/env_logger/0.6.2 \ > >> > - crate://crates.io/failure/0.1.5 \ > >> > - crate://crates.io/failure_derive/0.1.5 \ > >> > - crate://crates.io/fnv/1.0.6 \ > >> > - crate://crates.io/fuchsia-cprng/0.1.1 \ > >> > - crate://crates.io/getopts/0.2.19 \ > >> > - crate://crates.io/getrandom/0.1.6 \ > >> > - crate://crates.io/globset/0.4.4 \ > >> > - crate://crates.io/heck/0.3.1 \ > >> > - crate://crates.io/humantime/1.2.0 \ > >> > - crate://crates.io/ignore/0.4.8 \ > >> > - crate://crates.io/indexmap/1.0.2 \ > >> > - crate://crates.io/itertools/0.8.0 \ > >> > - crate://crates.io/itoa/0.4.4 \ > >> > - crate://crates.io/jobserver/0.1.16 \ > >> > - crate://crates.io/lazy_static/1.3.0 \ > >> > - crate://crates.io/libc/0.2.60 \ > >> > - crate://crates.io/lock_api/0.1.5 \ > >> > - crate://crates.io/log/0.4.7 \ > >> > - crate://crates.io/memchr/2.2.1 \ > >> > - crate://crates.io/memoffset/0.2.1 \ > >> > - crate://crates.io/nodrop/0.1.13 \ > >> > - crate://crates.io/num_cpus/1.10.1 \ > >> > - crate://crates.io/owning_ref/0.4.0 \ > >> > - crate://crates.io/packed_simd/0.3.3 \ > >> > - crate://crates.io/parking_lot/0.7.1 \ > >> > - crate://crates.io/parking_lot_core/0.4.0 \ > >> > - crate://crates.io/ppv-lite86/0.2.5 \ > >> > - crate://crates.io/proc-macro2/0.4.30 \ > >> > - crate://crates.io/quick-error/1.2.2 \ > >> > - crate://crates.io/quote/0.6.13 \ > >> > - crate://crates.io/rand/0.6.5 \ > >> > - crate://crates.io/rand/0.7.0 \ > >> > - crate://crates.io/rand_chacha/0.1.1 \ > >> > - crate://crates.io/rand_chacha/0.2.1 \ > >> > - crate://crates.io/rand_core/0.3.1 \ > >> > - crate://crates.io/rand_core/0.4.0 \ > >> > - crate://crates.io/rand_core/0.5.0 \ > >> > - crate://crates.io/rand_hc/0.1.0 \ > >> > - crate://crates.io/rand_hc/0.2.0 \ > >> > - crate://crates.io/rand_isaac/0.1.1 \ > >> > - crate://crates.io/rand_jitter/0.1.4 \ > >> > - crate://crates.io/rand_os/0.1.3 \ > >> > - crate://crates.io/rand_pcg/0.1.2 \ > >> > - crate://crates.io/rand_xorshift/0.1.1 \ > >> > - crate://crates.io/rdrand/0.4.0 \ > >> > - crate://crates.io/redox_syscall/0.1.56 \ > >> > - crate://crates.io/redox_users/0.3.0 \ > >> > - crate://crates.io/regex-syntax/0.6.10 \ > >> > - crate://crates.io/regex/1.2.0 \ > >> > - crate://crates.io/rustc-ap-arena/542.0.0 \ > >> > - crate://crates.io/rustc-ap-graphviz/542.0.0 \ > >> > - crate://crates.io/rustc-ap-rustc_data_structures/542.0.0 \ > >> > - crate://crates.io/rustc-ap-rustc_errors/542.0.0 \ > >> > - crate://crates.io/rustc-ap-rustc_lexer/542.0.0 \ > >> > - crate://crates.io/rustc-ap-rustc_macros/542.0.0 \ > >> > - crate://crates.io/rustc-ap-rustc_target/542.0.0 \ > >> > - crate://crates.io/rustc-ap-serialize/542.0.0 \ > >> > - crate://crates.io/rustc-ap-syntax/542.0.0 \ > >> > - crate://crates.io/rustc-ap-syntax_pos/542.0.0 \ > >> > - crate://crates.io/rustc-demangle/0.1.15 \ > >> > - crate://crates.io/rustc-hash/1.0.1 \ > >> > - crate://crates.io/rustc-rayon-core/0.2.0 \ > >> > - crate://crates.io/rustc-rayon/0.2.0 \ > >> > - crate://crates.io/rustc-workspace-hack/1.0.0 \ > >> > - crate://crates.io/rustc_version/0.2.3 \ > >> > - crate://crates.io/ryu/1.0.0 \ > >> > - crate://crates.io/same-file/1.0.5 \ > >> > - crate://crates.io/scoped-tls/1.0.0 \ > >> > - crate://crates.io/scoped_threadpool/0.1.9 \ > >> > - crate://crates.io/scopeguard/0.3.3 \ > >> > - crate://crates.io/semver-parser/0.7.0 \ > >> > - crate://crates.io/semver/0.9.0 \ > >> > - crate://crates.io/serde/1.0.97 \ > >> > - crate://crates.io/serde_derive/1.0.97 \ > >> > - crate://crates.io/serde_json/1.0.40 \ > >> > - crate://crates.io/smallvec/0.6.10 \ > >> > - crate://crates.io/spin/0.5.0 \ > >> > - crate://crates.io/stable_deref_trait/1.1.1 \ > >> > - crate://crates.io/strsim/0.8.0 \ > >> > - crate://crates.io/structopt-derive/0.2.18 \ > >> > - crate://crates.io/structopt/0.2.18 \ > >> > - crate://crates.io/syn/0.15.42 \ > >> > - crate://crates.io/synstructure/0.10.2 \ > >> > - crate://crates.io/term/0.6.0 \ > >> > - crate://crates.io/termcolor/1.0.5 \ > >> > - crate://crates.io/textwrap/0.11.0 \ > >> > - crate://crates.io/thread_local/0.3.6 \ > >> > - crate://crates.io/toml/0.5.1 \ > >> > - crate://crates.io/ucd-util/0.1.5 \ > >> > - crate://crates.io/unicode-segmentation/1.3.0 \ > >> > - crate://crates.io/unicode-width/0.1.5 \ > >> > - crate://crates.io/unicode-xid/0.1.0 \ > >> > - crate://crates.io/unicode_categories/0.1.1 \ > >> > - crate://crates.io/utf8-ranges/1.0.3 \ > >> > - crate://crates.io/vec_map/0.8.1 \ > >> > - crate://crates.io/walkdir/2.2.9 \ > >> > - crate://crates.io/winapi-i686-pc-windows-gnu/0.4.0 \ > >> > - crate://crates.io/winapi-util/0.1.2 \ > >> > - crate://crates.io/winapi-x86_64-pc-windows-gnu/0.4.0 \ > >> > - crate://crates.io/winapi/0.3.7 \ > >> > - crate://crates.io/wincolor/1.0.1 \ > >> > -" > >> > - > >> > - > >> > - > >> > -# FIXME: update generateme with the real MD5 of the license file > >> > -LIC_FILES_CHKSUM = " \ > >> > - file://LICENSE-APACHE;md5=1836efb2eb779966696f473ee8540542 \ > >> > - file://LICENSE-MIT;md5=0b29d505d9225d1f0815cbdcf602b901 \ > >> > -" > >> > - > >> > -SUMMARY = "Tool to find and fix Rust formatting issues" > >> > -HOMEPAGE = "https://github.com/rust-lang/rustfmt" > >> > -LICENSE = "Apache-2.0 | MIT" > >> > - > >> > -# includes this file if it exists but does not fail > >> > -# this is useful for anything you may want to override from > >> > -# what cargo-bitbake generates. > >> > -include rustfmt-nightly-${PV}.inc > >> > -include rustfmt-nightly.inc > >> > > >> > > >> > > >> > > >> > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#157197): https://lists.openembedded.org/g/openembedded-core/message/157197 Mute This Topic: https://lists.openembedded.org/mt/86442640/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-