Skip to content

several const ioctl() BPF aren't defined on OpenBSD #1482

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

Merged
merged 1 commit into from
Aug 18, 2019

Conversation

semarie
Copy link
Contributor

@semarie semarie commented Aug 18, 2019

unbreak OpenBSD after #1448

@rust-highfive
Copy link

r? @gnzlbg

(rust_highfive has picked a reviewer for you, use r? to override)

@gnzlbg
Copy link
Contributor

gnzlbg commented Aug 18, 2019 via email

@bors
Copy link
Contributor

bors commented Aug 18, 2019

📌 Commit 3264945 has been approved by gnzlbg

@bors
Copy link
Contributor

bors commented Aug 18, 2019

⌛ Testing commit 3264945 with merge 4f4fa08...

bors added a commit that referenced this pull request Aug 18, 2019
several const ioctl() BPF aren't defined on OpenBSD

unbreak OpenBSD after #1448
@semarie
Copy link
Contributor Author

semarie commented Aug 18, 2019

I could do better if I include net/bpf.h in libc-test

@semarie
Copy link
Contributor Author

semarie commented Aug 18, 2019

@gnzlbg I looked deeper to include net/bpf.h in openbsd ci, and redefine what's need.

ioctl not defined on OpenBSD:

  • BIOCGSEESENT
  • BIOCSSEESENT

ioctl with different value on OpenBSD:

  • BIOCGRSIG
  • BIOCSRSIG
  • BIOCSDLT

@equal-l2
Copy link
Contributor

@semarie Thank you for the follow-up!

@gnzlbg
Copy link
Contributor

gnzlbg commented Aug 18, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented Aug 18, 2019

📌 Commit 3b7be3c has been approved by gnzlbg

@bors
Copy link
Contributor

bors commented Aug 18, 2019

⌛ Testing commit 3b7be3c with merge 50fbf0c...

bors added a commit that referenced this pull request Aug 18, 2019
several const ioctl() BPF aren't defined on OpenBSD

unbreak OpenBSD after #1448
@bors
Copy link
Contributor

bors commented Aug 18, 2019

☀️ Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, status-azure
Approved by: gnzlbg
Pushing 50fbf0c to master...

@bors bors merged commit 3b7be3c into rust-lang:master Aug 18, 2019
@semarie semarie deleted the bpf-openbsd branch August 22, 2021 04:49
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.

5 participants