Skip to content

Migrate libuv into a submodule #830

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

Closed
wants to merge 3 commits into from
Closed

Migrate libuv into a submodule #830

wants to merge 3 commits into from

Conversation

erickt
Copy link
Contributor

@erickt erickt commented Aug 17, 2011

This patch turns libuv into a submodule. This commit needs some work done ahead of time in order to work properly. Graydon needs to use github's fork on https://github.com/joyent/libuv and then needs to incorporate rust's build system customizations of libuv by doing:

git clone https://github.com/graydon/libuv.git
cd libuv
git remote add git://github.com/erickt/libuv.git
git fetch erickt
git checkout -b rust remotes/erickt/rust
git push origin rust

Once this is done this commit can be pulled in.

@brson
Copy link
Contributor

brson commented Sep 27, 2011

It would probably be very difficult to merge this by now, and I don't think we have the will to commit to this strategy at the moment, so I'm going to go ahead and close the pull request. We can revisit in the future.

@brson brson closed this Sep 27, 2011
arielb1 pushed a commit to arielb1/rust that referenced this pull request Apr 10, 2015
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this pull request Dec 12, 2017
Add constants used by getrandom linux syscall

getrandom syscall was added in kernel 3.17. Musl and android seems to use old kernel headers, but considering that they define SYS_getrandom, the constants GRND_NONBLOCK and GRND_RANDOM should also be defined.
ZuseZ4 pushed a commit to EnzymeAD/rust that referenced this pull request Mar 7, 2023
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.

2 participants