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

ARMv6K Horizon OS support #2480

Merged
merged 6 commits into from
Jan 28, 2022
Merged

ARMv6K Horizon OS support #2480

merged 6 commits into from
Jan 28, 2022

Conversation

Meziu
Copy link
Contributor

@Meziu Meziu commented Oct 24, 2021

Added support for the Horizon OS Nintendo 3DS system Newlib declarations.

Edit: after some changes, networking is now stable with standardized API.

@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.

@Meziu
Copy link
Contributor Author

Meziu commented Oct 24, 2021

@rustbot label: +O-arm

@rustbot rustbot added the O-arm label Oct 24, 2021
@Amanieu
Copy link
Member

Amanieu commented Oct 26, 2021

r? @JohnTitor

@Meziu
Copy link
Contributor Author

Meziu commented Oct 31, 2021

@JohnTitor Why did it get blocked?

@JohnTitor
Copy link
Member

We need an internal decision for this, sorry about that.

@Meziu
Copy link
Contributor Author

Meziu commented Nov 2, 2021

We need an internal decision for this, sorry about that.

Oh yeah don’t worry, I was just puzzled by the block without a message. Take all the time needed.

@Meziu
Copy link
Contributor Author

Meziu commented Dec 8, 2021

@JohnTitor I don't want to come off as insistent, but it's been over a month and I received no info. Is there any chance this will get merged? It's not a problem if it won't, I just want to know if I should continue to work seriously on this project so that I may set my priorities correctly.

@JohnTitor
Copy link
Member

This isn't blocked by me, but a team and they are busy with other work, so I cannot tell you when decision is made. And I don't have enough time to do open source work right now so cannot push the process, sorry about that.

@Meziu
Copy link
Contributor Author

Meziu commented Dec 26, 2021

Fixed the standards for the networking, matching the latest version of the compiler.

@Meziu Meziu force-pushed the master branch 4 times, most recently from 6761f3c to b3363cd Compare December 28, 2021 23:34
@AzureMarker
Copy link
Member

AzureMarker commented Jan 6, 2022

@JohnTitor another month has passed, any news? What team needs to make a decision, and what is it about? There isn't a lot of transparency going on right now. You mentioned you're busy right now, is there anyone else who can review this?

@AzureMarker
Copy link
Member

AzureMarker commented Jan 27, 2022

r? @rust-lang/libc
Since @JohnTitor said they don't have time to work on this. Can someone clarify what team needs to make what decision?

@rust-highfive rust-highfive assigned Amanieu and unassigned JohnTitor Jan 27, 2022
@Amanieu
Copy link
Member

Amanieu commented Jan 28, 2022

I chased down the various threads related to this issue. It seems that this is blocked on this comment by @skade which expresses concerns about bindings to vendor-specific APIs. However this PR only creates bindings to newlib standard C APIs so I think it is fine to merge.

@bors r+

@bors
Copy link
Contributor

bors commented Jan 28, 2022

📌 Commit 98ef7e7 has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Jan 28, 2022

⌛ Testing commit 98ef7e7 with merge f1ae4e4...

bors added a commit that referenced this pull request Jan 28, 2022
ARMv6K Horizon OS support

Added support for the Horizon OS Nintendo 3DS system `Newlib` declarations.

Edit: after some changes, networking is now stable with standardized API.
@Meziu
Copy link
Contributor Author

Meziu commented Jan 28, 2022

I chased down the various threads related to this issue. It seems that this is blocked on this comment by @skade which expresses concerns about bindings to vendor-specific APIs. However this PR only creates bindings to newlib standard C APIs so I think it is fine to merge.

@bors r+

Yes, that is indeed the reason why this PR is made to be this way. Glad to see it solved. 😃

@bors
Copy link
Contributor

bors commented Jan 28, 2022

💔 Test failed - checks-actions

@Amanieu
Copy link
Member

Amanieu commented Jan 28, 2022

@bors retry

@bors
Copy link
Contributor

bors commented Jan 28, 2022

⌛ Testing commit 98ef7e7 with merge ba81130...

@bors
Copy link
Contributor

bors commented Jan 28, 2022

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing ba81130 to master...

@bors bors merged commit ba81130 into rust-lang:master Jan 28, 2022
bors added a commit that referenced this pull request Apr 26, 2023
armv7 PSVita OS support via newlib

Added support for ps vita via newlib.
Similar to #2480, but for a different device.

I'm working on adding support of std for vita, and this is essentially a prerequisite.
bors added a commit that referenced this pull request Apr 26, 2023
armv7 PSVita OS support via newlib

Added support for ps vita via newlib.
Similar to #2480, but for a different device.

I'm working on adding support of std for vita, and this is essentially a prerequisite.
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.

8 participants