Skip to content

fix: support DEBUG in environment - #53

Merged
henryiii merged 1 commit into
masterfrom
henryiii-patch-1
Aug 11, 2021
Merged

henryiii merged 1 commit into
masterfrom
henryiii-patch-1

Conversation

@henryiii

@henryiii henryiii commented Aug 3, 2021

Copy link
Copy Markdown
Collaborator

A few updates on rereading, and support for DEBUG being set in the environment.

@ax3l ax3l left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thank you! 👍

Comment thread setup.py
def build_extension(self, ext):
extdir = os.path.abspath(os.path.dirname(self.get_ext_fullpath(ext.name)))

# required for auto-detection of auxiliary "native" libs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you want to remove the comment? I found this a bit surprising why this is needed, so I added it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

-> #57

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I was looking at a Python 3 only version, which has the end separator automatically (due to pathlib) and has less need of the comment. Don't think I really intended to remove the comment here.

@henryiii
henryiii merged commit 5d580d9 into master Aug 11, 2021
@ax3l

ax3l commented Aug 11, 2021 •

Copy link
Copy Markdown
Contributor

@henryiii sorry, I added a comment after approval. Can we leave that comment in? It's pretty nifty setuptools magic why this end-of-path separator is needed -> #57

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