Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hurd: Fix st_dev name #3785

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

hurd: Fix st_dev name #3785

wants to merge 1 commit into from

Conversation

sthibaul
Copy link
Contributor

Currently struct stat and struct stat64 are not coherent: struct stat is using st_dev, and struct stat64 is using st_fsid.

st_dev is the more commonly-known name, already used by e.g. ~45 rust software in Debian, so better fix st_fsid into st_dev, rather than having to uselessly spend time hand-patching all these software.

@rustbot
Copy link
Collaborator

rustbot commented Jul 25, 2024

r? @JohnTitor

rustbot has assigned @JohnTitor.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@sthibaul
Copy link
Contributor Author

I am re-submitting here, since my comments on #3758 don't seem to be getting answers.

Currently struct stat and struct stat64 are not coherent: struct stat is
using st_dev, and struct stat64 is using st_fsid.

st_dev is the more commonly-known name, already used by e.g. ~45 rust
software in Debian, so better fix st_fsid into st_dev, rather than having to
uselessly spend time hand-patching all these software.
@tgross35
Copy link
Contributor

Brought this up for policy in #3839, that should at least give us better guidelines.

@rustbot blocked

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants