[meta-security] Remove outdated meta-rust dependency text in README

Message ID 20220604063124.279064-1-fede@evolware.org
State Accepted, archived
Delegated to: Armin Kuster
Headers show
Series [meta-security] Remove outdated meta-rust dependency text in README | expand

Commit Message

Federico Pellegrin June 4, 2022, 6:31 a.m. UTC
Since ~16 months the base of meta-rust was merged into meta-oe,
down to honister as far as I can see, and therefore the indications
to use meta-rust are really not needed and may just create extra
work and possibly problems to users.
---
 README | 12 ------------
 1 file changed, 12 deletions(-)

Comments

akuster808 June 4, 2022, 10:55 p.m. UTC | #1
On 6/3/22 23:31, Federico Pellegrin wrote:
> Since ~16 months the base of meta-rust was merged into meta-oe,
> down to honister as far as I can see, and therefore the indications
> to use meta-rust are really not needed and may just create extra
> work and possibly problems to users.
> ---
>   README | 12 ------------
>   1 file changed, 12 deletions(-)
>
> diff --git a/README b/README
> index 081669f..a9b3600 100644
> --- a/README
> +++ b/README
> @@ -62,18 +62,6 @@ other layers needed. e.g.:
>       /path/to/meta-openembedded/meta-networking \
>       /path/to/layer/meta-security "

thanks for the patch.

did I miss some rust changes in 
https://git.yoctoproject.org/meta-security/commit/?h=master-next&id=edf8b109e77f6d3f2740e09bdc89f0aaf2885dc4 
?

- armin
>   
> -Optional Rust dependancy
> -======================================
> -If you want to use the latest Suricata that needs rust, you will need to clone
> -
> -  URI: https://github.com/meta-rust/meta-rust.git
> -  branch: master
> -
> -  BBLAYERS += "/path/to/layer/meta-rust"
> -
> -This will activate the dynamic-layer mechanism and pull in the newer suricata
> -
> -
>   
>   Maintenance
>   ======================================
Federico Pellegrin June 5, 2022, 3:01 a.m. UTC | #2
Hi Armin,
Sorry if it was unclear or if I misunderstood something.

As far as I could see and tried as I had to build it yesterday, I can now
build suricata without adding meta-rust as the rust support (compilers etc)
is now already in oe-core, ie:
https://git.openembedded.org/openembedded-core/tree/meta/recipes-devtools/rust
and
https://git.openembedded.org/openembedded-core/tree/meta/recipes-devtools/cargo
Some relevant discussion: https://github.com/meta-rust/meta-rust/pull/364

The note in README may still just apply to versions < honister I believe
(ie. dunfell is the only not EOL-ed). For more recent versions it may just
potentially create confusion IMO.

Cheers,
Federico

Il giorno dom 5 giu 2022 alle ore 00:58 akuster808 <akuster808@gmail.com>
ha scritto:

>
>
> On 6/3/22 23:31, Federico Pellegrin wrote:
> > Since ~16 months the base of meta-rust was merged into meta-oe,
> > down to honister as far as I can see, and therefore the indications
> > to use meta-rust are really not needed and may just create extra
> > work and possibly problems to users.
> > ---
> >   README | 12 ------------
> >   1 file changed, 12 deletions(-)
> >
> > diff --git a/README b/README
> > index 081669f..a9b3600 100644
> > --- a/README
> > +++ b/README
> > @@ -62,18 +62,6 @@ other layers needed. e.g.:
> >       /path/to/meta-openembedded/meta-networking \
> >       /path/to/layer/meta-security "
>
> thanks for the patch.
>
> did I miss some rust changes in
>
> https://git.yoctoproject.org/meta-security/commit/?h=master-next&id=edf8b109e77f6d3f2740e09bdc89f0aaf2885dc4
> ?
>
> - armin
> >
> > -Optional Rust dependancy
> > -======================================
> > -If you want to use the latest Suricata that needs rust, you will need
> to clone
> > -
> > -  URI: https://github.com/meta-rust/meta-rust.git
> > -  branch: master
> > -
> > -  BBLAYERS += "/path/to/layer/meta-rust"
> > -
> > -This will activate the dynamic-layer mechanism and pull in the newer
> suricata
> > -
> > -
> >
> >   Maintenance
> >   ======================================
>
>

Patch

diff --git a/README b/README
index 081669f..a9b3600 100644
--- a/README
+++ b/README
@@ -62,18 +62,6 @@  other layers needed. e.g.:
     /path/to/meta-openembedded/meta-networking \
     /path/to/layer/meta-security "
 
-Optional Rust dependancy
-======================================
-If you want to use the latest Suricata that needs rust, you will need to clone
-
-  URI: https://github.com/meta-rust/meta-rust.git
-  branch: master
-
-  BBLAYERS += "/path/to/layer/meta-rust"
-
-This will activate the dynamic-layer mechanism and pull in the newer suricata
-
-
 
 Maintenance
 ======================================