Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when running pip install python-saml #237

Closed
jafetmorales opened this issue Aug 25, 2018 · 10 comments
Closed

Error when running pip install python-saml #237

jafetmorales opened this issue Aug 25, 2018 · 10 comments

Comments

@jafetmorales
Copy link

This is the output when running pip install python-saml

Collecting python-saml
  Downloading https://files.pythonhosted.org/packages/2b/05/c709c6f23df8a939e324b3136042957256e779f3fb0492fb7b0bfff7d1b4/python-saml-2.4.1.tar.gz (72kB)
Collecting dm.xmlsec.binding==1.3.3 (from python-saml)
  Downloading https://files.pythonhosted.org/packages/02/ba/b534536eb50bc013cd8dc3116c46eee4bff5b9e57e2af23f81afc574a67a/dm.xmlsec.binding-1.3.3.tar.gz (120kB)
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\jmorales\AppData\Local\Temp\pip-install-ueuyv6lm\dm.xmlsec.binding\setup.py", line 72
        print "Warning : cflag %s skipped" % flag
                                         ^
    SyntaxError: Missing parentheses in call to print

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in C:\Users\jmorales\AppData\Local\Temp\pip-install-ueuyv6lm\dm.xmlsec.binding\

This is my setup:

Python version: 3.5
OS: Windows 10 Home
System Type: 64-bit Operating System, x64-based processor
Processor: AMD A10-8700P Radeon R6

@pitbulk
Copy link
Contributor

pitbulk commented Aug 26, 2018

It seems an issue with the installation of the dm.xmlsec.binding dependency.

There is a thread where devs share it experience installing it here:
#30

It seems there were releases on July of [dm.xmlsec.binding(https://pypi.org/project/dm.xmlsec.binding/#history).

I updated python-saml to use the latest, 1.3.7

@travisfraser
Copy link

Do you have plans for a new version of python-saml that has these changes?

@pitbulk
Copy link
Contributor

pitbulk commented Sep 5, 2018

with the latest version is working for you guys?

@travisfraser
Copy link

The last published version does not build since it still pulls in the old dm.xmlsec.binding version.

@pitbulk
Copy link
Contributor

pitbulk commented Sep 5, 2018

Released 2.4.2

@pitbulk pitbulk closed this as completed Sep 5, 2018
@pitbulk pitbulk reopened this Sep 5, 2018
@pitbulk
Copy link
Contributor

pitbulk commented Sep 5, 2018

@jafetmorales and @travisfraser Please verify now it works as expected

@jafetmorales
Copy link
Author

jafetmorales commented Sep 5, 2018

@pitbulk Unfortunately, I still get the error. The problem is that I am using Python 3, so the parentheses is needed when printing something. The dependency dm.xmlsec.binding is only available for Python 2, and python-saml is available only for Python 2 as well. So porting of python-saml to Python 3 so that it works for people like me would require porting dm.xmlse.binding to Python 3 as well. But the guy who makes dm.xmlse.binding apparently does not have a repository anywhere. So he would need to have a repository, or we would need to post an unofficial version of dm.xmlse.binding that works on Python 3. Also, other issues might arise from other python-saml dependencies that we might need to solve to port it to Python 3.

@pitbulk
Copy link
Contributor

pitbulk commented Sep 5, 2018

If you use python3, then why are you not using python3-saml instead python-saml?

Code is compatible and it uses a different xmlsec library.

@jafetmorales
Copy link
Author

My bad, that's in the third sentence of the README. This issue is closed.

@travisfraser
Copy link

We ended up going the python3-saml route even though we are on python 2. This combination works well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants