Skip to content

Unbundle ext/xmlrpc#5640

Closed
cmb69 wants to merge 1 commit into
php:masterfrom
cmb69:cmb/unbundle-xmlrpc
Closed

Unbundle ext/xmlrpc#5640
cmb69 wants to merge 1 commit into
php:masterfrom
cmb69:cmb/unbundle-xmlrpc

Conversation

@cmb69

@cmb69 cmb69 commented May 29, 2020

Copy link
Copy Markdown
Member

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
Copy Markdown
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

cmb69 commented May 29, 2020

Copy link
Copy Markdown
Member Author

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

cmb69 commented May 29, 2020

Copy link
Copy Markdown
Member Author

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.

3 participants