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

tree-sitter: update to 0.20.7 #40158

Merged
merged 1 commit into from Oct 26, 2022

Conversation

mdkcore0
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, (x86_64-libc)
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • armv7l
    • i686-libc

@mdkcore0
Copy link
Contributor Author

don't get it; xlint tree-sitter didn't have reported any issues locally

Error: "cargo update" commands should include the specific version we are updating from in the --package SPEC
Error: Process completed with exit code 1.

Explicitly set autocfg version on `cargo update` to 1.0.1 [1].

[1] https://github.com/tree-sitter/tree-sitter/blob/v0.20.7/Cargo.lock#L69
@mdkcore0
Copy link
Contributor Author

Looking in the repository it seems I have to set autocfg version on cargo update

updated the PR to set it to version 1.0.1, as seems on tree-sitter Cargo.lock file.

@classabbyamp
Copy link
Member

don't get it; xlint tree-sitter didn't have reported any issues locally

xtools doesn't get a version bump for every xlint change, and CI uses the HEAD of xtools' master branch

@mdkcore0
Copy link
Contributor Author

mdkcore0 commented Oct 25, 2022

don't get it; xlint tree-sitter didn't have reported any issues locally

xtools doesn't get a version bump for every xlint change, and CI uses the HEAD of xtools' master branch

oh, cool, thanks for clarifying that for me!

@classabbyamp classabbyamp merged commit 4713de1 into void-linux:master Oct 26, 2022
@mdkcore0 mdkcore0 deleted the update-tree-sitter branch October 26, 2022 13:25
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.

None yet

2 participants