On 1/2/19 7:52 PM, Thomas De Schampheleire wrote:
El lun., 31 dic. 2018 a las 1:53, Mads Kiilerich
(<[email protected]>) escribió:

+    hg log -r "$rev"
+    hg update "$rev"
+
+    cleanup
+    virtualenv -p "$(command -v python2)" "$venv"
+    source "$venv/bin/activate"
+    pip install --upgrade pip setuptools
+    pip install -e .
+    pip install -r dev_requirements.txt

I have found that in case of "different but compatible constraints", it
is more stable to install both the app and dev requirements at once: pip
install --upgrade -e . -r dev_requirements.txt

We should perhaps change to recommend that everywhere?
'everywhere' would be in this script and in docs/contributing.rst, right?
In other places there is no installation of dev_requirements?


Yes, and scripts/make-release .


+    pip install python-ldap python-pam

Should we just include these in dev_requirements.txt ?
I think both of these packages have dependencies expected on the host
system. python-ldap requires the ldap.h header file, normally provided
by the host system.
For 'core' developers like you and me, we could reasonably expect them
to 'sacrifice' their host system and install stuff they would not
otherwise need, but for casual developers it may be a bit too much.
(just my thoughts)


Yeah. Just an annoying extra step I often forget when testing in a new virtualenv.

I can solve it by keeping
pip install --upgrade -e . -r dev_requirements.txt python-ldap python-pam
in my hash history ;-)

/Mads


_______________________________________________
kallithea-general mailing list
[email protected]
https://lists.sfconservancy.org/mailman/listinfo/kallithea-general

Reply via email to