FirefighterBlu3 / python-pam

Python pam module supporting py3 (py2 support still exists but is deprecated)
MIT License
110 stars 23 forks source link

Is `six` a missing dependency somehow? #41

Open pmorch opened 9 months ago

pmorch commented 9 months ago

Hi, I can see that six does seem to already be a dependency (something I don't really understand), but it seems to still be missing from 2.0.2:

$ mkdir /tmp/venv

$ cd /tmp/venv

$ python3 -m venv .

$ ./bin/pip install python-pam
Collecting python-pam
  Using cached python_pam-2.0.2-py3-none-any.whl (10 kB)
Installing collected packages: python-pam
Successfully installed python-pam-2.0.2

$ ./bin/python -c 'import pam ; print("hi")'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/tmp/venv/lib/python3.11/site-packages/pam/__init__.py", line 9, in <module>
    from .__internals import PAM_ACCT_EXPIRED
  File "/tmp/venv/lib/python3.11/site-packages/pam/__internals.py", line 2, in <module>
    import six
ModuleNotFoundError: No module named 'six'

$ ./bin/pip install six
Collecting six
  Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Installing collected packages: six
Successfully installed six-1.16.0

$ ./bin/python -c 'import pam ; print("hi")'
hi
svmhdvn commented 2 months ago

I'm running into this issue too, @FirefighterBlu3 would you be able to take a look? Working around this by manually installing this locally for now.

FirefighterBlu3 commented 2 months ago

My apologies, let me fix that up

On Thu, Jun 13, 2024 at 3:14 PM Siva Mahadevan @.***> wrote:

I'm running into this issue too, @FirefighterBlu3 https://github.com/FirefighterBlu3 would you be able to take a look? Working around this by manually installing this locally for now.

— Reply to this email directly, view it on GitHub https://github.com/FirefighterBlu3/python-pam/issues/41#issuecomment-2166591260, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEW5EUJYUKE7TYYO6MJATZHHVRVAVCNFSM6AAAAABJI76ZCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWGU4TCMRWGA . You are receiving this because you were mentioned.Message ID: @.***>

pkoning2 commented 1 month ago

It's still broken in the published version 2.0.2.