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(haiku): execve not allowed if other pthreads have already been created #10953

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Sylvain78
Copy link
Contributor

Compiling for Haiku

…eated

Co-authored-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
Signed-off-by: Anarchos <sylvain_kerjean@hotmail.com>
Signed-off-by: Sylvain78 <sylvain_kerjean@hotmail.com>
@anmonteiro anmonteiro changed the title Haiku fix(haiku): execve not allowed if other pthreads have already been created Sep 29, 2024
@anmonteiro
Copy link
Collaborator

@rgrinberg do you think this needs a changelog entry?

@Alizter
Copy link
Collaborator

Alizter commented Sep 30, 2024

I think it would be worth a changelog entry. I'm still unsure about the issue this is resolving since I've never seen it, so I wouldn't mind a link to some Haiku documentation explaining the issue here. I also don't have a Haiku VM avilable at the moment, so if somebody could double check they can build and run dune on Haiku that would be great.

@rgrinberg
Copy link
Member

I don't see why not. If this change is useful to anyone using Haiku, the change log entry would also be useful

@Sylvain78
Copy link
Contributor Author

I think it would be worth a changelog entry. I'm still unsure about the issue this is resolving since I've never seen it, so I wouldn't mind a link to some Haiku documentation explaining the issue here. I also don't have a Haiku VM avilable at the moment, so if somebody could double check they can build and run dune on Haiku that would be great.

I encounter this when compiling lwt : "dune exec..." leads to "Operation not supported" because of this line kernel/team.cpp#1981

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.

4 participants