bee-san/RustScan

Add ARM support for GitHub actions

Open

#683 ouverte le 4 nov. 2024

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Rust (1 284 forks)batch import
good first issue

Métriques du dépôt

Stars
 (19 268 stars)
Métriques de merge PR
 (Merge moyen 44j 20h) (5 PRs mergées en 30 j)

Description

https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/accessing-contextual-information-about-workflow-runs#runner-context

We should add arm support to our github actions.

here's one comment about it:

 FYI, I did start looking at this:
  1. adding the appropriate build to the matrix and setting cross: true works perfectly!
  2. …but then the Python tests fail for that step only.
  3. it looks like actions-rs/toolchain is now deprecated and equally, it's not clear what the de facto standard Action might be.

So the build isn't too tricky but getting it working in a pipeline is proving a little more rabbit-hole-y than expected.

Originally posted by @PsypherPunk in https://github.com/RustScan/RustScan/issues/579#issuecomment-2107504881

Guide contributeur

Direction de recherche
Configurez une matrice GitHub Actions pour les cibles ARM, configurez la cross compilation avec un outil comme `cross` ou `setup rust` et assurez vous que les tests passent. Référez vous au commentaire sur la dépréciation de `actions rs/toolchain`.
Stack technique
rust
Domaine
ci cddevops
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitGitHub Actions
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45