Download it, build it, try to break it

Sorry @chriso , pedantry again, but should

willie@mir:~/safeup$ sudo target/release/safeup node

not be more correct as

safeup safenode ?

I’m not completely sure about this one. I think client and node are easy to understand, and safeup client and safeup node read a bit better than safeup safe and safeup safenode, but I’m willing to have my mind changed.

1 Like

I fully agree they read better, I am simply being pedantic and looking for consistency.

I too am willing to have my mind changed where ā€œcommon senseā€ trumps consistency :slight_smile:

Well, I think it’s consistent to use client and node. If node was going to be changed to safenode, then for consistency, client would be changed to safe.

BTW safeup passes cargo-audit with no complaints

Congratulations :clap: :confetti_ball: :slight_smile:

willie@gagarin:~/projects/maidsafe/jacderida/safeup$ cargo-audit audit
    Fetching advisory database from `https://github.com/RustSec/advisory-db.git`
      Loaded 537 security advisories (from /home/willie/.cargo/advisory-db)
    Updating crates.io index
    Scanning Cargo.lock for vulnerabilities (255 crate dependencies)
2 Likes

I think this links to the previous release?
Or I’m getting some cache issue.<\s>

Eh, disregard from that :upside_down_face::blush:

1 Like

oops /me checks

they are coming so thick and fast these past few days…

confirmed Release Safe Network v0.2.0/v0.15.5/v0.22.0/v0.8.0/v0.84.0/v0.82.0/v0.82.0/v0.76.0 Ā· maidsafe/safe_network Ā· GitHub
is the correct link

1 Like

IDK what happened there but you seem to have linked to the latest, but it doesn’t contain what I would expect :sweat_smile:
I’m in the bath now though so my lookup-abilities are somewhat limited :stuck_out_tongue_winking_eye:

1 Like

TMI +chars :slight_smile:

1 Like

Actually, I think something went wrong with the last release. Investigating.

4 Likes

Another release!!!

Has @dirvine brought in Ange Postecoglu as dev team manager

We never stop

2 Likes

And Scotland beating Spain. Something mighty strange going down.

3 Likes

We went one better, much better :smiley: :smiley:

1 Like

Btw, there’s a release for every PR that gets merged, so most of the time there probably won’t be huge differences from release to release.

4 Likes

Just as well you got safeup running then if we are going to get a release for every PR merge now.

Why the change? We were going weeks without a release a while back…

Not that I’m complaining…
You just determined to see that safeup tool getting well used :slight_smile:

You reincarnated Ally McLeod?!?!?!!

What’s the odds for a test net tomorrow?

1 Like

Bear in mind, the update command isn’t implemented in safeup just yet!

We changed to doing a release for every PR a while back, but sometimes we get a problem with publishing a crate, which can take days to fix, and no releases go out in the mean time.

2 Likes