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

Fixed building on Python 3.10 #16

Closed
wants to merge 5 commits into from
Closed

Fixed building on Python 3.10 #16

wants to merge 5 commits into from

Conversation

bjrubinstein
Copy link
Contributor

No description provided.

Copy link
Owner

@djlampert djlampert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change the import statement for setuptools to combine it with the other imports rather than having two import statements in a row.

import os, sys, setuptools

Copy link
Owner

@djlampert djlampert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the hspfmodel file, on line 54 in the proposed file, change "wdmutil" to "hspfmodel," since this is an error message for the hspfmodel constructor (you copied the text from the wdmutil constructor, I think).
print('You can pass messagepath to the wdmutil constructor.')
In the wdmutil file, on line 35 there appears to be an extraneous print statement. Please delete (unless I am wrong).

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

Successfully merging this pull request may close these issues.

None yet

2 participants