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

Enable the possibility to not bundle rapidjson with the package? #60

Closed
SylvainCorlay opened this issue Jan 23, 2017 · 7 comments
Closed

Comments

@SylvainCorlay
Copy link

Both rapidjson and python-rapidjson have been bundled for conda-forge,

However, the latter makes use of the rapidjson submodule.

It would probably make sense for the setup.py to handle the case where an installation prefix for rapidjson is provided instead of a submodule.

For reference, such a setup.py has been done for pyzmq which can either bundle zeromq or not.

@lelit
Copy link
Contributor

lelit commented Jan 23, 2017

Can you give some more information on this? Is it just some kind of option so that you can build python-rapidjson using a different path where the compiler search for the rapidjson includes, or what?

@SylvainCorlay
Copy link
Author

Yes exactly. For example, the pyzmq wheel from pypi bundles libzmq. The one from conda uses the libzmq from conda which is its own package.

lelit added a commit that referenced this issue Jan 26, 2017
This allows to use a local copy of the RapidJSON include files, without the
need of cloning its repository into the ./rapidjson submodule.

This fixes issue #60.
@lelit
Copy link
Contributor

lelit commented Jan 26, 2017

Sylvain, does this fulfill your needs?

@SylvainCorlay
Copy link
Author

@lelit thanks! I will try this for the conda-forge recipe and get back to you.

@lelit
Copy link
Contributor

lelit commented Feb 17, 2017

Just curious, did that help?

@lelit
Copy link
Contributor

lelit commented Mar 2, 2017

The proposed change is included in version 0.0.9, please reopen the issue if that's not enough.
Thank you!

@lelit lelit closed this as completed Mar 2, 2017
@SylvainCorlay
Copy link
Author

THanks @lelit and sorry for the late reply. I am updating the conda recipe to 0.10.0 and testing the functionality.

lelit added a commit that referenced this issue Aug 2, 2018
This targets issue #110, and fixups commit c4df408:
I'm quite surprised that this went unnoticed when I closed issue #60... sorry.
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