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

Find major/minor versions of DLL #3

Merged
merged 1 commit into from
Jan 3, 2018
Merged

Find major/minor versions of DLL #3

merged 1 commit into from
Jan 3, 2018

Conversation

fredrikaverpil
Copy link
Contributor

Instead of hardcoding e.g. python35.dll, this change will now successfully detect e.g. python36.dll too.

@mherrmann
Copy link
Owner

Good change, thanks. I'll merge it right now. A tiny code style preference: Have the closing parenthesis with the same indentation as the opening assignment:

python_dlls = (
    ...
) # here

I'll simply change that manually after merging.

Thanks!

@mherrmann mherrmann merged commit 1fc4eda into mherrmann:master Jan 3, 2018
mherrmann pushed a commit that referenced this pull request Jul 20, 2020
t0mpr1c3 added a commit to t0mpr1c3/fbs that referenced this pull request Aug 1, 2023
* Python 3.8

* Update requirements.txt
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.

2 participants