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

Backport Azure/azure-sdk-for-cpp#4982 #6

Closed

Conversation

jjerphan
Copy link
Member

@jjerphan jjerphan commented Oct 2, 2023

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

Backport Azure/azure-sdk-for-cpp#4982

Disclosure: this is a request from man-group/arcticdb.

Signed-off-by: Julien Jerphanion <[email protected]>
Co-authored-by: Phoebus Mak <[email protected]>
@conda-forge-webservices
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jjerphan
Copy link
Member Author

jjerphan commented Oct 2, 2023

@conda-forge-admin, please rerender

@jjerphan jjerphan marked this pull request as ready for review October 2, 2023 06:59
Comment on lines +54 to +66
+#if defined(_azure_SUPPORT_SETTING_CAPATH)
+ /**
+ * @brief Path to a directory which holds PEM encoded file, containing the certificate
+ * authorities sent to libcurl handle directly.
+ *
+ * @remark The Azure SDK will not check if the path is valid or not.
+ *
+ * @remark The default is the built-in system specific path. More about this option:
+ * https://curl.se/libcurl/c/CURLOPT_CAPATH.html
+ *
+ */
+ std::string CAPath;
+#endif
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This introduces a new public API. Are we sure it can be backported?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not entirely sure, indeed. Also a maintainer of the Azure SDK for C++ mentioned that the next release should happen sometime soon (see Azure/azure-sdk-for-cpp#4982 (comment)). Probably we wait for it?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That looks like a better choice.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To hopefully clarify, we are planning to release this change as 1.11.0-beta.1 on Thursday, October 5th.
Please note, since this goes to a beta release, and the beta release is for the library already had stable releases, the further Betas do not go to the official vcpkg repo. Usually, the beta changes are baked for a month until they are released in stable release. If you are looking for this change to appear in the vcpkg official repo, you'll probably need to wait until, I'd say, November 10th.
Otherwise, this change will be available in our Beta vcpkg registry - https://github.com/Azure/azure-sdk-vcpkg-betas - it should happen on October 5th.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for chiming in @antkmsft! BTW, when can we expect the next stable release of azure-identity-cpp?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are targeting the November timeframe for the next stable release of azure-identity-cpp, if not sooner.

Is there something particular you are hoping to use from there? We'd love to get some usage feedback on the beta release to help stabilize it.

@jjerphan
Copy link
Member Author

jjerphan commented Oct 2, 2023

Closing in preference to the next release of the SDK.

@jjerphan jjerphan closed this Oct 2, 2023
@jjerphan jjerphan deleted the backport-azure-sdk-for-cpp-4982 branch October 2, 2023 14:49
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

Successfully merging this pull request may close these issues.

4 participants