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

Linux POSIX support #4118

Merged
merged 35 commits into from
Oct 13, 2024
Merged

Linux POSIX support #4118

merged 35 commits into from
Oct 13, 2024

Conversation

andradei
Copy link
Contributor

@andradei andradei commented Aug 21, 2024

Add the missing Linux-specific support for many POSIX headers.

core/sys/posix/dirent.odin Outdated Show resolved Hide resolved
core/sys/posix/errno.odin Outdated Show resolved Hide resolved
@andradei andradei marked this pull request as ready for review August 24, 2024 02:24
core/os/os_linux.odin Outdated Show resolved Hide resolved
core/sys/posix/dirent.odin Outdated Show resolved Hide resolved
core/sys/posix/fcntl.odin Outdated Show resolved Hide resolved
core/sys/posix/sys_stat.odin Outdated Show resolved Hide resolved
core/sys/posix/dirent.odin Outdated Show resolved Hide resolved
core/sys/posix/dirent.odin Outdated Show resolved Hide resolved
core/sys/posix/dirent.odin Outdated Show resolved Hide resolved
@laytan
Copy link
Collaborator

laytan commented Aug 29, 2024

Also, are you planning on keeping this PR going or should we merge this and do other PRs for the rest?

@andradei
Copy link
Contributor Author

@laytan Now that I have a better idea of the work to be done, 'll keep this PR going (unless I find a change that's so massive it would be saner if it had its own PR).

@andradei
Copy link
Contributor Author

andradei commented Sep 22, 2024

@laytan As I get experience writing these POSIX implementations, I realize there is a lot of work still to be done. This PR became very, very big.

I'd like to consider it ready for review for what it implements. I'll create smaller PRs with another chunk of POSIX implementations after this is fully reviewed, approved, and merged. Those future PRs be smaller, with 1-3 files in core/sys/posix edited to add Linux support.

I think this way can keep these PRs saner for everyone, but I'm open to the opinion of reviewers.

@laytan laytan self-assigned this Sep 22, 2024
@laytan
Copy link
Collaborator

laytan commented Sep 30, 2024

@andradei instead of putting review comments everywhere and taking the time to explain every little thing and having you then make the changes I opted to just commit my reviews/corrections/cleanups here directly. Let me know about any questions regarding the changes. I will merge this once those questions have been answered/if you got no questions.

Still need to do a couple headers I see and then we can also enable the tests etc. I see from your last comment that you are picking those up in separate PRs which is fine by me.

@laytan laytan assigned andradei and unassigned laytan Sep 30, 2024
@andradei
Copy link
Contributor Author

@laytan I left questions on your commit. Thank you for reviewing this.

@andradei andradei requested a review from laytan October 11, 2024 03:43
core/sys/posix/signal.odin Outdated Show resolved Hide resolved
core/sys/posix/sys_sem.odin Outdated Show resolved Hide resolved
core/sys/posix/fcntl.odin Show resolved Hide resolved
core/sys/posix/limits.odin Show resolved Hide resolved
core/sys/posix/netdb.odin Show resolved Hide resolved
core/sys/posix/netinet_in.odin Show resolved Hide resolved
core/sys/posix/pthread.odin Show resolved Hide resolved
@laytan laytan merged commit 0157ff1 into odin-lang:master Oct 13, 2024
7 checks passed
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.

5 participants