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

fix longstanding error in pthread_attr_t definition #25614

Closed
cfriedt opened this issue May 26, 2020 · 1 comment
Closed

fix longstanding error in pthread_attr_t definition #25614

cfriedt opened this issue May 26, 2020 · 1 comment
Labels
bug The issue is a bug, or the PR is fixing a bug

Comments

@cfriedt
Copy link
Member

cfriedt commented May 26, 2020

Describe the bug
It looks like there was a copy-paste error made several years ago when defining pthread_attr_t.

c8aa657

typedef struct pthread_attr_t {
...
} pthread_attr_t;

To Reproduce
Discovered during CI build of #25527

Expected behavior
Expected build to succeed

Impact
Blocks #25527

Screenshots or console output
https://app.shippable.com/github/zephyrproject-rtos/zephyr/runs/73710/1/console

pthread_attr_t defined as wrong kind of tag
...
conflicting types for pthread_attr_t

Environment (please complete the following information):
CI (see shippable link)

Additional context

@cfriedt cfriedt added the bug The issue is a bug, or the PR is fixing a bug label May 26, 2020
@cfriedt cfriedt changed the title Fix longstanding error in ptheead_attr_t definition fix longstanding error in pthread_attr_t definition May 26, 2020
@cfriedt
Copy link
Member Author

cfriedt commented May 26, 2020

Even though it's a legitimate typo, it looks like it might be specific to the nrf52_bsim.
This kind of gives it away:

-include /home/buildslave/src/github.com/zephyrproject-rtos/zephyr/arch/posix/include/posix_cheats.h

I should adjust my #ifdefs accordingly in #25527

@cfriedt cfriedt closed this as completed May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

No branches or pull requests

1 participant