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

Skip extension modules if running egg_info #162

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

altendky
Copy link
Contributor

@altendky altendky commented May 4, 2019

WIP for:

  • An actually decent solution

@altendky
Copy link
Contributor Author

altendky commented May 4, 2019

I am running pip-tools on Azure to build my requirements lock files and it's having trouble with javabridge because of the Java detection. From what I can tell so far the ext_module parameter doesn't seem to affect the results of running egg_info. I'm going to ask around a bit to see if I can find any issues that might arise from doing this but figured I'd share it sooner than later.

As always, thanks for your consideration.

@altendky
Copy link
Contributor Author

altendky commented May 6, 2019

Well, responses have been about as good as you might expect for something like this... I think "ugh" and "ack!" summarize them well. No concrete alternatives identified yet. For now I'm just using this branch in my project and it's fine so I'll keep looking for something more proper before requesting this be considered for merging.

@LeeKamentsky
Copy link
Owner

Meh, it sure does look like a hack, but I can merge it if you want. I doubt it would do much harm.

@altendky
Copy link
Contributor Author

I dunno... I guess go for it. I've tried in a few different places a few times to get an alternative as opposed to just disgust and have failed.

@altendky altendky changed the title [WIP] Skip extension modules if running egg_info Skip extension modules if running egg_info Oct 16, 2019
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