Skip to content

tree-sitter: 0.25.4 -> 0.25.6, update grammars#414233

Merged
GaetanLepage merged 5 commits intoNixOS:stagingfrom
dtomvan:dtomvan/push-rpxtlysmrrpn
Jun 29, 2025
Merged

tree-sitter: 0.25.4 -> 0.25.6, update grammars#414233
GaetanLepage merged 5 commits intoNixOS:stagingfrom
dtomvan:dtomvan/push-rpxtlysmrrpn

Conversation

@dtomvan
Copy link
Contributor

@dtomvan dtomvan commented Jun 5, 2025

Built tree-sitter{,.grammars}.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
  • NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 501-1000 This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 501-1000 This PR causes many rebuilds on Linux and should normally target the staging branches. labels Jun 5, 2025
@nix-owners nix-owners bot requested review from Profpatsch and uncenter June 5, 2025 13:09
@dtomvan dtomvan changed the base branch from master to staging June 5, 2025 14:24
@nixpkgs-ci nixpkgs-ci bot closed this Jun 5, 2025
@nixpkgs-ci nixpkgs-ci bot reopened this Jun 5, 2025
@github-actions github-actions bot added 6.topic: python Python is a high-level, general-purpose programming language. 6.topic: vscode A free and versatile code editor that supports almost every major programming language. labels Jun 5, 2025
@ofborg ofborg bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Jun 5, 2025
@nix-owners
Copy link

nix-owners bot commented Jun 5, 2025

The PR's base branch is set to staging, but 5 commits from the master branch are included. Make sure you know the right base branch for your changes, then:

  • If the changes should go to the master branch, change the base branch to master
  • If the changes should go to the staging branch, rebase your PR onto the merge base with the staging branch:
    # git rebase --onto $(git merge-base upstream/staging HEAD) $(git merge-base upstream/master HEAD)
    git rebase --onto 6c76fd5f512e3225ecff578ded7cbe74347253cc 2c8c54ac2176d8201d8c97b23d8c0d06b303e704
    git push --force-with-lease

@github-actions github-actions bot added 10.rebuild-linux: 1001-2500 This PR causes many rebuilds on Linux and should target the staging branches. and removed 10.rebuild-linux: 501-1000 This PR causes many rebuilds on Linux and should normally target the staging branches. labels Jun 5, 2025
@nix-owners nix-owners bot requested review from datafoo and mweinelt June 5, 2025 14:29
@dtomvan dtomvan force-pushed the dtomvan/push-rpxtlysmrrpn branch from 1496272 to 97edf60 Compare June 5, 2025 14:30
@github-actions github-actions bot removed 6.topic: python Python is a high-level, general-purpose programming language. 6.topic: vscode A free and versatile code editor that supports almost every major programming language. labels Jun 5, 2025
@dtomvan
Copy link
Contributor Author

dtomvan commented Jun 5, 2025

Github messed up the rebasing on staging. I hope I did it correctly manually. Sorry for the extra pings.

NOTE: I checked the "built" box above. Haven't built these on op of staging though. They did work on master.

@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Jun 5, 2025
@dtomvan dtomvan changed the title tree-sitter: 0.25.3 -> 0.25.6, update grammars tree-sitter: 0.25.4 -> 0.25.6, update grammars Jun 5, 2025
@dtomvan dtomvan force-pushed the dtomvan/push-rpxtlysmrrpn branch from 97edf60 to 2c0bcc5 Compare June 5, 2025 14:33
@dtomvan dtomvan removed request for datafoo and mweinelt June 5, 2025 14:38
@github-actions github-actions bot added 10.rebuild-linux: 501-1000 This PR causes many rebuilds on Linux and should normally target the staging branches. and removed 10.rebuild-linux: 1001-2500 This PR causes many rebuilds on Linux and should target the staging branches. labels Jun 5, 2025
@mweinelt
Copy link
Member

mweinelt commented Jun 6, 2025

With 1001-2500 you really should keep targeting staging. This is roughly 4.5k rebuilds spread out over 4 targets.

@dtomvan
Copy link
Contributor Author

dtomvan commented Jun 6, 2025

Never thought about that. The actual number of rebuilds is 2*linux+2*darwin... Also because there's 3 python module updates this PR tanks a whole lot more rebuilds...

@MattSturgeon
Copy link
Contributor

guess because the rebuild count comes from the vim plugins which isn't deemed as heavy for hydra?

IIRC vim plugins aren't built by hydra, and they're unlikely to be transitive dependencies of anything that is. So as far as hydra is concerned, a vim plugins update is close to zero rebuilds.

@dtomvan
Copy link
Contributor Author

dtomvan commented Jun 25, 2025

So anything still blocking this? AFAICT 0.25.6 is still the latest version of TS, so that should be fine.

@GaetanLepage
Copy link
Contributor

So anything still blocking this? AFAICT 0.25.6 is still the latest version of TS, so that should be fine.

@mrcjkb any objection to merging this?

@mrcjkb
Copy link
Member

mrcjkb commented Jun 29, 2025

So anything still blocking this? AFAICT 0.25.6 is still the latest version of TS, so that should be fine.

@mrcjkb any objection to merging this?

Nope, no objections on my end

@GaetanLepage GaetanLepage merged commit 797fdaa into NixOS:staging Jun 29, 2025
17 of 18 checks passed
@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jun 29, 2025
@dtomvan dtomvan deleted the dtomvan/push-rpxtlysmrrpn branch June 29, 2025 12:33
@trofi
Copy link
Contributor

trofi commented Jun 30, 2025

Bisect says b0ca5ddc9916f66d40648f20757ed7f56b33d501 tree-sitter: 0.25.4 -> 0.25.6 broke hash checks in staging as:

$ nix build nixpkgs/staging#tree-sitter -L
...
tree-sitter> Fetching https://crates.io/api/v1/crates/yoke-derive/0.7.5/download -> tarballs/yoke-derive-0.7.5.tar.gz
error: hash mismatch in fixed-output derivation '/nix/store/dxyarsk54pxzjfkv2wwbmcng4zz2yy41-tree-sitter-0.25.6-vendor-staging.drv':
         specified: sha256-/KCvLsbb6DullLpRoSYbxtSsm/TMc6o0Y/QmK6BN748=
            got:    sha256-sGh16M7cbT5ct1sT2FcUUoIQFcoOftTuQ0aSCjtkTEs=

@dtomvan
Copy link
Contributor Author

dtomvan commented Jul 1, 2025

Ouch, I might've done the conflict resolution wrong:

  • I started this PR off of master (bad idea...)
  • I then rebased to staging
  • I saw 3 cargoHash values in the conflict markers and chose the wrong one, you can see in the diff of this PR that it was left unchanged.

My bad. I'm sorry for messing up. Fixing it right now. Might take a while because I want to reproduce the build and make sure I'm correct this time.

@vcunat
Copy link
Member

vcunat commented Jul 10, 2025

python3Packages.tree-sitter still remains broken on the current staging-next branch #422427

ValueError: Incompatible Language version 15. Must be between 13 and 14

@dtomvan
Copy link
Contributor Author

dtomvan commented Jul 10, 2025

I think it's outdated and they haven't released an update yet: https://github.com/tree-sitter/py-tree-sitter/releases

So I guess that package should get meta.broken = true;... Or we could set it to an unstable version somewhere after this PR

@vcunat
Copy link
Member

vcunat commented Jul 10, 2025

A report estimates the number of jobs blocked by this as ~850 so far, so this might be quite a significant breakage.

@dtomvan
Copy link
Contributor Author

dtomvan commented Jul 10, 2025

Because of a minor version bump, or because tree-sitter reverse dependencies aren't up-to-date with TS 0.25 yet? I don't suspect reverting helps here. I'm sorry if this PR is a nuisance to you, just wanted to update a package...

@dtomvan dtomvan mentioned this pull request Jul 16, 2025
13 tasks
dtomvan added a commit to dtomvan/nixpkgs that referenced this pull request Jul 25, 2025
This commit prevents the update script from touching the files listed
under `pinnedGrammars`. This way breakages cannot accidentally be caused
again (re: NixOS#414233 breaking the heck out of staging-next et al)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 1001-2500 This PR causes many rebuilds on Darwin and should most likely target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 1001-2500 This PR causes many rebuilds on Linux and should target the staging branches. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants