From ae235430958c0a9404250e8d1a89ba3d43b7ecf9 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Dec 09 2017 18:06:43 +0000 Subject: Do not make optional dependencies required This content of the requirements.txt file is processed in the setup.py and added/declared as being what pagure requires to work. But, if you deploy pagure, say with openid login, you do not need flask-oidc, but if you keep it in the list, it's added to the .egg-info/ requires file as being required and when something queries pkg_resources it will complain that flask-oidc is required and missing and make the app crash. So let's keep optional packages not required :) Signed-off-by: Pierre-Yves Chibon --- diff --git a/requirements.txt b/requirements.txt index 64b93bb..3ad8072 100644 --- a/requirements.txt +++ b/requirements.txt @@ -30,14 +30,14 @@ straight.plugin >= 1.4.0-post-1 wtforms # Needed only for local authentication and/or Pagure CI -cryptography +# cryptography # Required only for the `oidc` authentication backend -flask-oidc +# flask-oidc # Required only for the `fas` and `openid` authentication backends -python-fedora +# python-fedora # Required only for the `local` authentication backend -py-bcrypt +# py-bcrypt