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

libevent missing Python dependency #44

Closed
jakirkham opened this issue Sep 13, 2016 · 4 comments
Closed

libevent missing Python dependency #44

jakirkham opened this issue Sep 13, 2016 · 4 comments
Assignees

Comments

@jakirkham
Copy link
Contributor

Appears the libevent recipe is not open sourced AFAICT or at least it is not in this repo. In any event, I'm going to put this issue here as I can't think of a more appropriate place to put it ATM. Should add libevent does appear in the anaconda channel. So this seems reasonable.

After recently working on a libevent recipe in PR ( conda-forge/staged-recipes#1491 ), it has become clear to me that libevent depends on Python. This is because it installs a script to the bin directory called event_rpcgen.py. Not only does it depend on Python, but that script appears to hard-code Python 2. Thus libevent should depend on Python 2.x for run and build.

@jakirkham
Copy link
Contributor Author

Any thoughts on this?

@jakirkham
Copy link
Contributor Author

Friendly nudge. Would love to understand what plans there are to fix this issue.

@ilanschnell
Copy link
Contributor

I would not consider libevent a Python package, just because it contains a Python script, because it does not install a Python library (into site-packages) which you can import. The plan is to remove the script event_rpcgen.py (in order to avoid confusion) and not add Python as a dependency.

@jakirkham
Copy link
Contributor Author

That sounds completely reasonable. Will follow suit then. Thanks.

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

2 participants