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
coursera-dl: init at 0.10.0 #39338
coursera-dl: init at 0.10.0 #39338
Conversation
@GrahamcOfBorg build coursera-dl |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: coursera-dl Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: coursera-dl Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: coursera-dl Partial log (click to expand)
|
The following will enable tests: diff --git a/pkgs/applications/misc/coursera-dl/default.nix b/pkgs/applications/misc/coursera-dl/default.nix
index 1f5d7107aa4..419d91c5709 100644
--- a/pkgs/applications/misc/coursera-dl/default.nix
+++ b/pkgs/applications/misc/coursera-dl/default.nix
@@ -1,8 +1,7 @@
-{ stdenv, fetchFromGitHub, fetchurl, pandoc, python2 }:
+{ stdenv, fetchFromGitHub, fetchurl, pandoc, python3, glibcLocales }:
let
- pythonPackages = python2.pkgs;
-
+ pythonPackages = python3.pkgs;
in pythonPackages.buildPythonApplication rec {
name = "coursera-dl-${version}";
version = "0.10.0";
@@ -14,10 +13,23 @@ in pythonPackages.buildPythonApplication rec {
sha256 = "0m3f6ly8c3mkb8yy2y398afswqgy17rz159s1054wzxpb4f85zlb";
};
- buildInputs = with pythonPackages; [ beautifulsoup4 ConfigArgParse keyring mock pandoc pytest requests urllib3 ];
+ nativeBuildInputs = with pythonPackages; [ pandoc ];
+
+ buildInputs = [ glibcLocales ];
propagatedBuildInputs = with pythonPackages; [ beautifulsoup4 ConfigArgParse keyring pyasn1 requests six urllib3 ];
+ checkInputs = with pythonPackages; [ pytest mock ];
+
+ preConfigure = ''
+ export LC_ALL=en_US.utf-8
+ '';
+
+ checkPhase = ''
+ # requires dbus service
+ py.test -k 'not test_get_credentials_with_keyring' .
+ '';
+
meta = with stdenv.lib; {
description = "CLI for downloading Coursera.org videos and naming them";
homepage = https://github.com/coursera-dl/coursera-dl; |
Thanks. I updated the PR with your feedback. |
This is probably related to: #38991 |
@GrahamcOfBorg build coursera-dl |
Success on x86_64-linux (full log) Attempted: coursera-dl Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: coursera-dl Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: coursera-dl Partial log (click to expand)
|
Motivation for this change
New package
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)