Raise MSRV to 1.64 and delegate `ctypes` type aliases to `core::ffi`

Ouverte Adaptée aux débutants
#188 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
72/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
Calme
Stack technique
rust
Domaine
operating-systems

Piste de recherche

Commencez par le paramètre rust-version dans Cargo.toml et le module d’alias de types ctypes dans src/lib.rs. Vérifiez les types core::ffi de Rust 1.64 et les conditionnelles de cible existantes, puis exécutez la suite de tests existante. C’est terminé lorsque le MSRV est 1.64, que les alias délèguent à core::ffi et que les cibles prises en charge conservent leur comportement actuel.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Just sharing this observation despite the benefit likely being minimal. Delegating the C FFI type aliases to those from core::ffi by bumping rust-version to 1.64 seems desirable?

It'd have avoided the need for PRs like this in March 2026 for example (which is merged but still unreleased).

Details

Presently linux-raw-sys supports Rust 1.63 (Aug 2022) as a minimum (in May 2023 rust-version was bumped from 1.48 to benefit from improved const fn support):

https://github.com/sunfishcode/linux-raw-sys/blob/3b15c17f2688445cd3e9ef2907113829837773dd/Cargo.toml#L13

In Nov 2021 linux-raw-sys switched from a cty crate dep to vendoring it's own ctypes type alias module, which presently has required a bit of maintenance support over the past year for Tier 3 targets with ensuring the correct c_char (m68k, hexagon (as a follow-up to restoring support), MIPS R6).

Since Rust 1.64 (Sep 2022) the equivalent FFI C types have been available via the core crate (the target list is managed by a cfg_select! macro, and prior to that the cfg_if! crate macro).

Although this comment says core was not available for a m68k target? (unless they used rustc_codegen_gcc) but as core is already used elsewhere in this crate I assume that wouldn't equate to a regression 😅

Langage dominant
Rust
Étoiles
71
Forks
63
Aucune PR mergée en 30 j

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de sunfishcode/linux-raw-sys

Toutes les issues de sunfishcode/linux-raw-sys

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.