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

Splitting C++ REST SDK #68

Open
kreuzerkrieg opened this issue Feb 7, 2016 · 4 comments
Open

Splitting C++ REST SDK #68

kreuzerkrieg opened this issue Feb 7, 2016 · 4 comments

Comments

@kreuzerkrieg
Copy link
Contributor

I was facing somewhat annoying "issue". sometimes I just need the PPLX, sometimes just to parse or create a piece of JSON. But whatever I need I have to add the whole C++ REST SDK NuGet package. Not a big issue but maybe it is a good idea to have the PPLX and JSON as standalone packages?
In addition, the PPLX is not a full port of PPL, for example it has no parallel algorithms or parallel_invoke functionality, any plans to add it?

@ras0219-msft
Copy link
Contributor

This is a really interesting idea, but it would be a large restructuring of the project so we need to be careful and ensure it's the right thing to do.

As an experiment, adding macros to selectively disable parts of the library might be an interesting approach (we currently have a CPPREST_EXCLUDE_WEBSOCKETS macro which disables websockets; this also removes the need for Boost and OpenSSL on Windows).

@ghost
Copy link

ghost commented Sep 2, 2016

+1

I only need PPLX. It would be good if it could be split from the cpprestsdk project

@cmazakas
Copy link

cmazakas commented May 23, 2017

I was actually looking into making the cross-platform port of the PPL stand-alone. ras, you mention that it would require a large restructuring of the code. Is it not simply possible to copy the pplx directory and then create a new CMakeLists.txt file from that?

This section of the PPL is actually pretty useful and really should be available to everyone because there's no good libraries out there that seem to match what MS has created here.

@jason-ha
Copy link
Contributor

I have project with the same needs. My project also needed narrow character width support on Windows. I tackled the latter first and have completed infrastructure to isolate components include pplx and json. Please see #1204. In CMake you may depend on cpprest_json and/or cpprest_pplx.
Note: I have not tested out referencing both in my project yet, but it should play out just fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants