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

Unbundle ext/xmlrpc #5640

Closed
wants to merge 1 commit into from
Closed

Unbundle ext/xmlrpc #5640

wants to merge 1 commit into from

Conversation

cmb69
Copy link
Contributor

@cmb69 cmb69 commented May 29, 2020

According to https://wiki.php.net/rfc/unbundle_xmlprc we unbundle
ext/xmlrpc.

According to <https://wiki.php.net/rfc/unbundle_xmlprc> we unbundle
ext/xmlrpc.
@cmb69 cmb69 added the RFC label May 29, 2020
@carusogabriel
Copy link
Contributor

Idea: document somewhere how to unbundle an extension from the php-src.

It'll be easier for reviewers as well, we check if the X files were modified/removed and there we go.

@carusogabriel carusogabriel added this to the PHP 8.0 milestone May 29, 2020
@cmb69
Copy link
Contributor Author

cmb69 commented May 29, 2020

Idea: document somewhere how to unbundle an extension from the php-src.

Well, I just removed the ext/xmlrpc directory, and then grepped for "xmlrpc". Finally, I did a full build, to check whether nothing was broken in the process.

@cmb69
Copy link
Contributor Author

cmb69 commented May 29, 2020

Thanks! Applied as 4fa3687.

@cmb69 cmb69 closed this May 29, 2020
@cmb69 cmb69 deleted the cmb/unbundle-xmlrpc branch May 29, 2020 14:53
khs1994 added a commit to khs1994-docker/lnmp that referenced this pull request Jun 4, 2020
Signed-off-by: Kang Huaishuai <khs1994@khs1994.com>
khs1994 added a commit to khs1994-docker/lnmp that referenced this pull request Jun 4, 2020
Signed-off-by: Kang Huaishuai <khs1994@khs1994.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants