Skip to content

Commit f592a12

Browse files
heptauclaude
andcommitted
fix(ci): fix python-ldap build on Linux and Windows
python-ldap requires system OpenLDAP headers (lber.h). Fix: - Linux: install libldap2-dev + libsasl2-dev before pip install - Windows: skip django-auth-ldap (no OpenLDAP on Windows); LDAP auth is optional and the app works without it Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 11e0a2b commit f592a12

3 files changed

Lines changed: 6 additions & 2 deletions

File tree

.github/workflows/release.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,10 @@ jobs:
6767
run: choco install make --no-progress -y
6868
shell: powershell
6969

70+
- name: Install LDAP headers (Linux)
71+
if: runner.os == 'Linux'
72+
run: sudo apt-get install -y libldap2-dev libsasl2-dev
73+
7074
- name: Cache NW.js
7175
uses: actions/cache@v4
7276
with:

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ dependencies = [
1111
"Django>=5.2",
1212
"CherryPy",
1313
"social-auth-app-django>=5.8.0",
14-
"django-auth-ldap",
14+
"django-auth-ldap; sys_platform != 'win32'",
1515
"pyscrypt",
1616
"pyaes",
1717
"openpyxl",

requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ setuptools
22
Django>=5.2
33
CherryPy
44
social-auth-app-django>=5.8.0
5-
django-auth-ldap
5+
django-auth-ldap; sys_platform != "win32"
66
pyscrypt
77
pyaes
88
openpyxl

0 commit comments

Comments
 (0)