Skip to content

ARMv6K Horizon OS support#2480

Merged
bors merged 6 commits into
rust-lang:masterfrom
Meziu:master
Jan 28, 2022
Merged

ARMv6K Horizon OS support#2480
bors merged 6 commits into
rust-lang:masterfrom
Meziu:master

Conversation

@Meziu

@Meziu Meziu commented Oct 24, 2021

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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

Meziu commented Oct 24, 2021

Copy link
Copy Markdown
Contributor Author

@rustbot label: +O-arm

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

Amanieu commented Oct 26, 2021

Copy link
Copy Markdown
Member

r? @JohnTitor

@Meziu

Meziu commented Oct 31, 2021

Copy link
Copy Markdown
Contributor Author

@JohnTitor Why did it get blocked?

@JohnTitor

Copy link
Copy Markdown
Member

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

@Meziu

Meziu commented Nov 2, 2021

Copy link
Copy Markdown
Contributor Author

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

Meziu commented Dec 8, 2021

Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown
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

Meziu commented Dec 26, 2021

Copy link
Copy Markdown
Contributor Author

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

AzureMarker commented Jan 6, 2022

Copy link
Copy Markdown
Contributor

@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

AzureMarker commented Jan 27, 2022

Copy link
Copy Markdown
Contributor

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

Amanieu commented Jan 28, 2022

Copy link
Copy Markdown
Member

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

bors commented Jan 28, 2022

Copy link
Copy Markdown
Contributor

📌 Commit 98ef7e7 has been approved by Amanieu

@bors

bors commented Jan 28, 2022

Copy link
Copy Markdown
Contributor

⌛ 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

Meziu commented Jan 28, 2022

Copy link
Copy Markdown
Contributor Author

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

bors commented Jan 28, 2022

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-actions

@Amanieu

Amanieu commented Jan 28, 2022

Copy link
Copy Markdown
Member

@bors retry

@bors

bors commented Jan 28, 2022

Copy link
Copy Markdown
Contributor

⌛ Testing commit 98ef7e7 with merge ba81130...

@bors

bors commented Jan 28, 2022

Copy link
Copy Markdown
Contributor

☀️ 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