Updated to 2.1.65, since upstream has had more releases, and it was just bumping hashes pretty much.

@GrahamcOfBorg build anki

Checking the checkboxes at e.g. Tools -> Preferences -> Review didn't work which seems to work for the anki-bin version. Unfortunately I can't investigate and I also don't know if this PR causes that. I may run on wayland and maybe thats related to the issue you mentioned.

Thanks for the heads up @mohe2015! I appreciate the info.

It's broken for me too, both on nixos-unstable (X + Anki 2.1.61) and with this PR.
Which hey, nice, that means I can reproduce it :D

I think that means it's not related to this PR specifically, but rather the broader package.
As such, I'll look into it separately.

I filed an issue for that: #248357

@euank Thank you for initiating this update. I don't use anki and wanted to gauge your confidence in this update.

We merged #248866 into staging, which among other things updated wheel to 0.41.1. I think anki will be broken until ankitects/anki@e20e7f7 is picked up, which was only added in version 2.1.62. I could try to cherry-pick that change onto the current version, but I only want to take that route if we cannot update it.

As the issue I found seems unrelated to this PR I think we can still merge it.

Thanks for the heads up, @tjni, appreciated.

I'm not sure if we'll need to cherry pick that or not.
Since it's related to wheel packaging, I think it's likely that if the package still builds, there's a decent chance it's fine.

It looks like there's not yet a hydra build for those changes on staging, so checking properly would also require recompiling pretty much the entire world, not just anki.
I think I'll wait on hydra to get something built, see if hydra hits an evaluation error for the current 2.1.61 anki package, and then also see if I can check this update with the staging changes.

It took awhile for staging-next to build, and at the time of this writing it's only finished for aarch64-linux, but there does seem to be an error related to wheel:

FAILED: /build/source/out/wheels/aqt-2.1.61-py3-none-any.whl 
/build/source/out/rust/debug/runner run /build/source/out/pyenv/bin/python python/write_wheel.py qt/aqt /build/source/out/qt/_aqt /build/source/out/wheels/aqt-2.1.61-py3-none-any.whl
Command failed: 

Traceback (most recent call last):
  File "/build/source/python/write_wheel.py", line 189, in <module>
    write_wheel(
  File "/build/source/python/write_wheel.py", line 80, in write_wheel
    return write_wheel_file(
  File "/build/source/python/write_wheel.py", line 46, in write_wheel_file
    with ReproducibleWheelFile(filename, "w") as wheel:
  File "/nix/store/hzpbm0zkg61smdwpj42jmp2k5bansrdw-python3-3.10.12/lib/python3.10/zipfile.py", line 1312, in __exit__
    self.close()
  File "/nix/store/h8mzz6lkk0v83ck53541fpq7n7ngl1s1-python3.10-wheel-0.41.1/lib/python3.10/site-packages/wheel/wheelfile.py", line 195, in close
    self.writestr(self.record_path, data.getvalue())
  File "/build/source/python/write_wheel.py", line 18, in writestr
    raise ValueError("ZipInfo required")
ValueError: ZipInfo required
Exception ignored in: <function ZipFile.__del__ at 0xfffff7114e50>
Traceback (most recent call last):
  File "/nix/store/hzpbm0zkg61smdwpj42jmp2k5bansrdw-python3-3.10.12/lib/python3.10/zipfile.py", line 1821, in __del__
    self.close()
  File "/nix/store/h8mzz6lkk0v83ck53541fpq7n7ngl1s1-python3.10-wheel-0.41.1/lib/python3.10/site-packages/wheel/wheelfile.py", line 195, in close
    self.writestr(self.record_path, data.getvalue())
  File "/build/source/python/write_wheel.py", line 18, in writestr
    raise ValueError("ZipInfo required")
ValueError: ZipInfo required

Then we should probably merge this sooner than later as this update seems to include that fix and seems to work otherwise too.

In addition to the nixpkgs-review above, I've verified that this PR, when merged into current master, gives a working anki binary.

Since the python update broke the older anki that's on master now, I think that lends more importance to merging this as soon as possible.

In addition to the nixpkgs-review above, I've verified that this PR, when merged into current master, gives a working anki binary.

Note that nixpkgs-review also builds against master by default.

Yup, I was just trying to say that I also ran the binary too, since nixpkgs-review only verifies the build, not running the application.

Anki builds on Darwin but crashes on startup. Could you set meta.broken = stdenv.isDarwin?

I've marked it as broken on darwin, thanks for testing that!

Btw, 2.1.66 is out but I didn't want to hold this PR off on that.