Skip to content
This repository was archived by the owner on Jul 10, 2026. It is now read-only.

update platforms/arch supported based on what modernc.org/libc is supporting#50

Merged
Nexucis merged 1 commit into
mainfrom
nexucis/supported-platform-list
Nov 10, 2022
Merged

update platforms/arch supported based on what modernc.org/libc is supporting#50
Nexucis merged 1 commit into
mainfrom
nexucis/supported-platform-list

Conversation

@Nexucis

@Nexucis Nexucis commented Nov 10, 2022

Copy link
Copy Markdown
Member

Since #42, the build is failing on the main branch because some platforms/arch are not supported by modernc.org/libc.

This PR aligned the same amount of platform/arch currently supported by this lib. If we want to fallback this PR https://gitlab.com/cznic/libc/-/tree/master/unistd need to be updated with the platform missing.
By looking at the commits, it looks like the code is generated (perhaps partially)

Note: dragonfly and illumos platform have been removed.

Signed-off-by: Augustin Husson husson.augustin@gmail.com

…ibc is supporting

Signed-off-by: Augustin Husson <husson.augustin@gmail.com>
@Nexucis Nexucis changed the title update supported platforms/arch supported based on what modernc.org/libc is supporting update platforms/arch supported based on what modernc.org/libc is supporting Nov 10, 2022
@juliusv

juliusv commented Nov 10, 2022

Copy link
Copy Markdown
Member

👍

FYI @SuperQ

@Nexucis

Nexucis commented Nov 10, 2022

Copy link
Copy Markdown
Member Author

For the records, here the code repo of the libc used: https://gitlab.com/cznic/libc
By looking at the different issues already opened to support others platforms, it doesn't look like it will be easy to come back to the initial list.

@Nexucis

Nexucis commented Nov 10, 2022

Copy link
Copy Markdown
Member Author

@SuperQ it would be cool to know if you are ok with this PR. Or Do you prefer to revert the PR #42.

On my side dropping CGO is more important than giving up on few platforms/archs, but that's just my opinion.

@Nexucis Nexucis requested a review from SuperQ November 10, 2022 13:59
@juliusv

juliusv commented Nov 10, 2022

Copy link
Copy Markdown
Member

I think we should just merge it for now so we can still do the release today. I think getting rid of CGO is nice enough to warrant dropping support for some of the more arcane platforms / archs.

@juliusv juliusv mentioned this pull request Nov 10, 2022
@SuperQ

SuperQ commented Nov 10, 2022

Copy link
Copy Markdown
Member

Yea, this is fine. I also would prefer to avoid needing CGO for our builds.

@Nexucis

Nexucis commented Nov 10, 2022

Copy link
Copy Markdown
Member Author

Nice ! Thanks @SuperQ !

@Nexucis Nexucis merged commit 6c65228 into main Nov 10, 2022
@Nexucis Nexucis deleted the nexucis/supported-platform-list branch November 10, 2022 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants