Improve build script

• Cosmetic improvements
• Do not install rustup whether Rust is already installed by the distribution package manager
• Use command -v to know if an executable is present in the PATH

changed milestone to %1.9

@moul pense a marquer la MR avec le label S-request review la prochaine fois ou/et a me tagger pour que je sache que tu attend une review.

Merci pour les améliorations, mais par contre le elif rustc ne peut pas convenir. Il n'est pas possible de builder Duniter avec rustc directement. Cargo est absolument nécessaire pour aller chercher les dépendances Rust. Merci de remplacer le elif par un else.

added S-waiting author label

assigned to @moul and unassigned @librelois

added 5 commits

• 9ddd29a4...53af8359 - 3 commits from branch dev
• a9b099f7 - [build] Change neon/build.sh script
• 4e7d8b8b - [build] Do not install rustup whether Rust is already installed

Compare with previous version

I wasn't aware that this methodology was used for Duniter development. I will use it, then.

I switch from rustc to cargo, which is a more used command.

I need the elif to check the case when rustup and cargo are not installed, to install rustup. In case rustup is not installed, but cargo is (which means rust tools are installed from the package manager of the distribution), I do not want rustup to be installed.

For which reason do you want the elif to be replaced by else? Without the elif, it will install rustup, which is unnecessary under Fedora with dnf install rust. This is the main point of this MR.

added S-request review label

removed S-waiting author label

assigned to @librelois and unassigned @moul

merged

Mouais rustup et un peu l'équivalent de nvm pour les dev node. Je pense que vouloir dev en Rust sans rustup n'a pas de sens.

Cependant pour la compilation manuelle ça fait sens, donc ok pour pouvoir builder sans rustup. Mais du coup il faut virer la commande rustup show, de toute façon elle était redondante avec les commandes qui affichent las version de cargo et rustc.

Merci c'est fusionnée :)