Skip to content

๐Ÿš‘๏ธ zb: Fix sendmsg for iOS and other apple OS#1741

Open
dtolnay wants to merge 1 commit intoz-galaxy:mainfrom
dtolnay-contrib:ios
Open

๐Ÿš‘๏ธ zb: Fix sendmsg for iOS and other apple OS#1741
dtolnay wants to merge 1 commit intoz-galaxy:mainfrom
dtolnay-contrib:ios

Conversation

@dtolnay
Copy link
Copy Markdown

@dtolnay dtolnay commented Mar 9, 2026

Since #1660 zbus is unable to build for iOS.

$ cargo check --target=aarch64-apple-ios
error[E0599]: no associated item named `NOSIGNAL` found for struct `SendFlags` in the current scope
   --> zbus/src/connection/socket/unix.rs:330:28
    |
330 |     let flags = SendFlags::NOSIGNAL;
    |                            ^^^^^^^^ associated item not found in `SendFlags`

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Mar 9, 2026

Merging this PR will not alter performance

โœ… 22 untouched benchmarks


Comparing dtolnay-contrib:ios (40d8995) with main (aa34b2d)

Open in CodSpeed

@zeenix
Copy link
Copy Markdown
Contributor

zeenix commented Mar 9, 2026

@dtolnay Thanks! I will take care of modifying the commit message for you, don't worry about it.

While I have your attention: serde-rs/serde#2045 has been a real PITA for me in the zlink project. @RaitoBezarius has graciously volunteered to try and provide a PR for this so I'd really appreciate it if you could review that (and merge if it's good) when they've provided it. ๐Ÿ™ Of course, me merging this good PR is not at all conditioned on that. ๐Ÿ˜‰

@zeenix
Copy link
Copy Markdown
Contributor

zeenix commented Mar 9, 2026

oh and btw, I'm very surprised to hear that people use zbus on iOS. ๐Ÿ˜ฎ

@zeenix
Copy link
Copy Markdown
Contributor

zeenix commented Mar 9, 2026

@dtolnay Thanks! I will take care of modifying the commit message for you, don't worry about it.

I guess you didn't opt for the "Allow maintainers to apply changes to the PR":

โฏ git push dtolnay ios -f
ERROR: Permission to dtolnay-contrib/zbus.git denied to zeenix.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Please either allow me to do that or just modify the commit and add the ๐Ÿš‘๏ธ to the commit title.

@zeenix zeenix changed the title zb: Fix sendmsg for iOS and other apple OS ๐Ÿš‘๏ธ zb: Fix sendmsg for iOS and other apple OS Mar 9, 2026
    error[E0599]: no associated item named `NOSIGNAL` found for struct `SendFlags` in the current scope
       --> zbus/src/connection/socket/unix.rs:330:28
        |
    330 |     let flags = SendFlags::NOSIGNAL;
        |                            ^^^^^^^^ associated item not found in `SendFlags`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants