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

Added missing constants from elf.h #2897

Merged
merged 1 commit into from Sep 6, 2022

Conversation

paandahl
Copy link
Contributor

@paandahl paandahl commented Sep 3, 2022

I have added a bunch of constants from elf.h, useful for parsing ELF files.

Closes #1876

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.

Please see the contribution instructions for more information.

@JohnTitor
Copy link
Member

JohnTitor commented Sep 5, 2022

Could you squash commits into one? The commits would be a bit noisy when someone does git-blaming.

@paandahl
Copy link
Contributor Author

paandahl commented Sep 5, 2022

@JohnTitor Sure - it's fixed now. 👍

@JohnTitor
Copy link
Member

Thanks! @bors r+

@bors
Copy link
Contributor

bors commented Sep 5, 2022

📌 Commit 83da60c has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 5, 2022

⌛ Testing commit 83da60c with merge 698fbdb...

bors added a commit that referenced this pull request Sep 5, 2022
Added missing constants from elf.h

I have added a bunch of constants from elf.h, useful for parsing ELF files.

Closes #1876
@bors
Copy link
Contributor

bors commented Sep 5, 2022

💔 Test failed - checks-actions

@paandahl
Copy link
Contributor Author

paandahl commented Sep 6, 2022

I moved ELFOSABI_ARM_AEABI to linux/gnu since it's not defined in musl libc.

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 6, 2022

📌 Commit 1d11c02 has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 6, 2022

⌛ Testing commit 1d11c02 with merge 6b36b18...

@bors
Copy link
Contributor

bors commented Sep 6, 2022

☀️ Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 6b36b18 to master...

@bors bors merged commit 6b36b18 into rust-lang:master Sep 6, 2022
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.

libc should expose program header flag constants
5 participants