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
anki: 2.0.52 -> 2.1.4 #46599
anki: 2.0.52 -> 2.1.4 #46599
Conversation
qtwebkit now builds successfully due to this cb38098 commit. |
cc #33248 |
- # running from source? | ||
- srcFolder = os.path.join(os.path.dirname(__file__), "..") | ||
- webInSrcFolder = os.path.abspath(os.path.join(srcFolder, "web")) | ||
- if os.path.exists(webInSrcFolder): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This path was not found?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think not... I can't really remember because I did this at least a couple of weeks ago
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be web
copied next to the aqt
directory
- ["lame", "rec.wav", processingDst, "--noreplaygain", "--quiet"], | ||
+ [lameCmd, "rec.wav", processingDst, "--noreplaygain", "--quiet"], | ||
- ["lame", processingSrc, processingDst, "--noreplaygain", "--quiet"], | ||
+ [lameCmd, processingSrc, processingDst, "--noreplaygain", "--quiet"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because we have to wrap python applications anyway, you can also add lame
to PATH by setting makeWrapperArgs
: https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/backup/borg/default.nix#L31
Same for mplayer. This way we reduce the number of patches.
def langDir(): | ||
- dir = os.path.join(os.path.dirname( | ||
- os.path.abspath(__file__)), "locale") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens if locale
would be in the anki directory?
- # running from source? | ||
- srcFolder = os.path.join(os.path.dirname(__file__), "..") | ||
- webInSrcFolder = os.path.abspath(os.path.join(srcFolder, "web")) | ||
- if os.path.exists(webInSrcFolder): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be web
copied next to the aqt
directory
New version of Anki. The jump from 2.0 -> 2.1 is somewhat major, especially
considering it moved from python 2 to python 3.
Note that this will not work with versions of qt that aren't 5.9, which
currently does not build on NixOS. I got it to build by applying this patch
I don't really know what's going on there, but once that builds, Anki seems to
build fine and work well.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)