pboettch / json-schema-validator

JSON schema validator for JSON for Modern C++
Other
466 stars 134 forks source link

Provide up-to-date release to better support vcpkg #207

Closed SamVanheer closed 1 year ago

SamVanheer commented 1 year ago

The latest release of this library dates back to 2 years ago.

vcpkg ties port updates to releases for convenience, so it would be helpful if a new release were made available to allow the vcpkg port to be updated as well.

See here for the vcpkg port configuration: https://github.com/microsoft/vcpkg/tree/master/ports/json-schema-validator

The patches provided there might be worth a look to see if they can be applied to the project directly if they are still needed.

When a release is available it would be great if you could also request an update to the port so they can add the new release.

See also the issue and pull request that added the library: https://github.com/microsoft/vcpkg/issues/11080 https://github.com/microsoft/vcpkg/pull/11599

pboettch commented 1 year ago

Well, I wasn't even aware that this project is provided by vcpkg. I'm relatively out of time. If someone could take care of that it would be great help.

SamVanheer commented 1 year ago

I've checked the vcpkg patches to see which ones are still relevant:

So all that's needed is for a new version to be released, then a request to update the port should be made here: https://github.com/microsoft/vcpkg/issues/new/choose

They'll take care of the rest, assuming there's no problems the new version should be made available shortly.

Is there anything else that needs doing before a new release can be made available?

SamVanheer commented 1 year ago

On second thought, forward-find-package uses find_dependency instead of find_package which causes it to use the parameters provided for the original find_package call. That's probably worth changing.

SamVanheer commented 1 year ago

Once #221 has been merged all vcpkg patches will have been merged. All that's left then is to do a new tag release to allow the vcpkg port to be updated.

SamVanheer commented 1 year ago

I've been experimenting with making the HEAD commit into a port and it looks like #195 will be a problem. Because the tests and examples are forced on if it's the root file vcpkg will print these warnings:

The following EXEs were found in /bin or /debug/bin. EXEs are not valid distribution targets.

D:/a/halflife-unified-sdk/halflife-unified-sdk/vcpkg/packages/json-schema-validator_x86-windows/bin/json-schema-validate.exe D:/a/halflife-unified-sdk/halflife-unified-sdk/vcpkg/packages/json-schema-validator_x86-windows/bin/readme-json-schema.exe

The following EXEs were found in /bin or /debug/bin. EXEs are not valid distribution targets.

D:/a/halflife-unified-sdk/halflife-unified-sdk/vcpkg/packages/json-schema-validator_x86-windows/debug/bin/json-schema-validate.exe D:/a/halflife-unified-sdk/halflife-unified-sdk/vcpkg/packages/json-schema-validator_x86-windows/debug/bin/readme-json->schema.exe

I will look into resolving this problem.

pboettch commented 1 year ago

If we disable tests and bins by default, I'd be OK with that if it helps.

SamVanheer commented 1 year ago

I've created a pull request that fixes the issue with no downsides.

pboettch commented 1 year ago

Release about to be pushed

SamVanheer commented 1 year ago

I've made the update request here: https://github.com/microsoft/vcpkg/issues/28028