Skip to content

Conversation

mbyx
Copy link
Contributor

@mbyx mbyx commented Jul 24, 2025

Removes all uses of enums, replacing them with c_enums. Since c_enums creates numeric types and constants, cfg_attrs were also removed for deriving Copy and Clone. As per #4419

Additionally allow c_enum to take more than one enum as input, similar to the other macros.

@mbyx mbyx force-pushed the libc-remove-enum branch 4 times, most recently from 99cd358 to 2f1e1da Compare July 24, 2025 08:13
@mbyx mbyx marked this pull request as ready for review July 24, 2025 08:34
@rustbot
Copy link
Collaborator

rustbot commented Jul 24, 2025

Some changes occurred in solarish module

cc @jclulow, @pfmooney

src/macros.rs Outdated
(
($(
$(#[repr($repr:ty)])?
enum $ty_name:ident {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this require the pub keyword since we make the types pub? Similar to the other macros

@@ -798,7 +798,6 @@ s! {
}

// linux/openat2.h
#[non_exhaustive]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are these changes here?

@tgross35 tgross35 added the stable-declined This change is breaking, difficult to backport, low priority, or otherwise not relevant for 0.2 label Jul 26, 2025
@mbyx mbyx force-pushed the libc-remove-enum branch from 2f1e1da to 86305e5 Compare July 26, 2025 13:54
@mbyx mbyx force-pushed the libc-remove-enum branch from 86305e5 to a541bf4 Compare July 26, 2025 13:56
@mbyx mbyx requested a review from tgross35 July 26, 2025 14:51
@tgross35 tgross35 added this pull request to the merge queue Jul 28, 2025
Merged via the queue into rust-lang:main with commit 83280ed Jul 28, 2025
48 of 52 checks passed
@GrigorenkoPV
Copy link

This caused rust-lang/rust#145952

@tgross35
Copy link
Contributor

Not this PR, it's f0a2c3e specifically (which has a bit more context in the message)

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 29, 2025
std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes rust-lang#145952, which was caused by rust-lang/libc#4575

`@rustbot` label T-libs O-haiku
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 29, 2025
std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes rust-lang#145952, which was caused by rust-lang/libc#4575

```@rustbot``` label T-libs O-haiku
tgross35 added a commit to tgross35/rust that referenced this pull request Aug 29, 2025
std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes rust-lang#145952, which was caused by rust-lang/libc#4575

````@rustbot```` label T-libs O-haiku
tgross35 added a commit to tgross35/rust that referenced this pull request Aug 29, 2025
std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes rust-lang#145952, which was caused by rust-lang/libc#4575

`````@rustbot````` label T-libs O-haiku
tgross35 added a commit to tgross35/rust that referenced this pull request Aug 30, 2025
std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes rust-lang#145952, which was caused by rust-lang/libc#4575

``````@rustbot`````` label T-libs O-haiku
tgross35 added a commit to tgross35/rust that referenced this pull request Aug 30, 2025
std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes rust-lang#145952, which was caused by rust-lang/libc#4575

```````@rustbot``````` label T-libs O-haiku
rust-timer added a commit to rust-lang/rust that referenced this pull request Aug 30, 2025
Rollup merge of #145991 - GrigorenkoPV:haiku, r=tgross35

std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes #145952, which was caused by rust-lang/libc#4575

```````@rustbot``````` label T-libs O-haiku
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Aug 30, 2025
std: haiku: fix `B_FIND_PATH_IMAGE_PATH`

Fixes rust-lang/rust#145952, which was caused by rust-lang/libc#4575

```````@rustbot``````` label T-libs O-haiku
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-dragonfly O-linux O-macos O-solarish O-unix stable-declined This change is breaking, difficult to backport, low priority, or otherwise not relevant for 0.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants