Update vcpkg install command for jsoncpp#1624
Closed
albaropereyra22 wants to merge 3 commits intoopen-source-parsers:masterfrom
Closed
Update vcpkg install command for jsoncpp#1624albaropereyra22 wants to merge 3 commits intoopen-source-parsers:masterfrom
albaropereyra22 wants to merge 3 commits intoopen-source-parsers:masterfrom
Conversation
Added --classic option, now required.
Contributor
|
Can you explain why |
Author
|
They changed the formatting, I’m not an expert, I just looked through help realized there was a classic option tried it and it worked like it’s suppose to in the guide. Instead of rewriting the guide like it needs to be, I just added the —classic option in the interim. Show something that works for now until an expert writes the updated guide.
I hope this answers your question.
…Sent from my iPhone
On Nov 12, 2025, at 12:05 AM, Jordan Bayles ***@***.***> wrote:
baylesj
left a comment
(open-source-parsers/jsoncpp#1624)
Can you explain why --classic is now necessary? Does JsonCpp have to be modified to work with manifests / the "new" mode?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.
|
Contributor
|
The cause of this I think is actually that your project has a vcpkg.json file and vcpkg expected you to modify your vcpkg.json file install of attempting to install JsonCpp. I'll update the README. The |
Contributor
|
Attempting to address this issue here: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added --classic option, now required.