Skip to content

ext/curl: suppress -Wdeprecated-declarations #10531

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

Closed
wants to merge 1 commit into from

Conversation

MaxKellermann
Copy link
Contributor

No description provided.

@nielsdos
Copy link
Member

nielsdos commented Feb 7, 2023

LGTM, but you accidentally included one additional commit though.

@MaxKellermann
Copy link
Contributor Author

you accidentally included one additional commit though

Oops, fat-fingered stgit? Fixed now.

Copy link
Member

@devnexen devnexen left a comment

Choose a reason for hiding this comment

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

Makes sense

@devnexen devnexen closed this in 8c8a38a Feb 8, 2023
@devnexen
Copy link
Member

devnexen commented Feb 8, 2023

Thank you !

@bukka
Copy link
Member

bukka commented Feb 10, 2023

Would be probably also good to create issues for the actual deprecations as we might need to eventually do something about them. For example if there is no way around them, we should deprecate such functionality in PHP as well...

MaxKellermann added a commit to CM4all/php-src that referenced this pull request Feb 10, 2023
Disable the warning before including curl_arginfo.h.

(Follow-up for php#10531)
@MaxKellermann MaxKellermann deleted the curl_no_deprecated branch February 10, 2023 10:33
MaxKellermann added a commit to CM4all/php-src that referenced this pull request Mar 7, 2023
iluuu1994 pushed a commit that referenced this pull request Mar 22, 2023
Disable the warning before including curl_arginfo.h.

(Follow-up for #10531)
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