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

Update liburing source URL to kernel.org #4422

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

shjala
Copy link
Member

@shjala shjala commented Nov 6, 2024

The liburing-0.7 is missing from the original source URL. This change updates the source URL to kernel.org. The kernel.org version is the same:

➜  shah-dev git clone https://git.kernel.dk/cgit/liburing/
➜  liburing git:(master) git checkout 45f0735219a615ae848033c47c7e2d85d101d43e
Note: switching to '45f0735219a615ae848033c47c7e2d85d101d43e'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 45f0735 .gitignore: add test/cq-overflow-peek
➜  liburing git:(liburing-0.7)
➜  liburing git:(liburing-0.7) cd ..
➜  shah-dev wget https://git.kernel.org/pub/scm/linux/kernel/git/axboe/liburing.git/snapshot//liburing-0.7.tar.gz
➜  shah-dev tar -xzf liburing-0.7.tar.gz
➜  shah-dev diff -r --exclude=".git" liburing liburing-0.7
➜  shah-dev

The liburing-0.7 is missing from the original source URL. This change
updates the source URL to kernel.org.

Signed-off-by: Shahriyar Jalayeri <[email protected]>
@shjala shjala added bug Something isn't working dependencies Pull requests that update a dependency file labels Nov 6, 2024
@OhmSpectator
Copy link
Member

I think we should also backport it.

@OhmSpectator OhmSpectator merged commit 2ba0fe1 into lf-edge:master Nov 6, 2024
33 of 46 checks passed
@shjala
Copy link
Member Author

shjala commented Nov 6, 2024

yup, I'll do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants