Skip to content

Commit

Permalink
Clean conda-environment-dev.yml
Browse files Browse the repository at this point in the history
Fix the pinned version for `python-ldap`.

Add some explanatory comments.

Remove `pip` since it is already a dependency of `python`.

Remove `openldap` since it is already a dependency of `python-ldap`.
  • Loading branch information
fabcor-maxiv committed Feb 6, 2024
1 parent b3a3cf4 commit 2cf62ed
Showing 1 changed file with 16 additions and 5 deletions.
21 changes: 16 additions & 5 deletions conda-environment-dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,21 @@ name: mxcubecore
channels:
- conda-forge
dependencies:

### Runtime dependencies

- python >=3.8,<3.11
# Run-time
- openldap
- python-ldap=3.4.0
# Dev
- pip

# We install `python-ldap` from conda because it is "hard" to install.
# On PyPI it is not available as *wheel*, only as *sdist* which requires to be built
# with compilation steps that require a compiler and some header files.
# Installing with conda is much "easier".
#
# IMPORTANT:
# Always make sure that `python-ldap` here is pinned to the same version
# as in the Poetry lockfile `poetry.lock` (not the other way around).
- python-ldap=3.4.3

### Development dependencies

- poetry

0 comments on commit 2cf62ed

Please sign in to comment.