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

Added php implementation of fnmatch for platforms not supporting it. #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jgmdev
Copy link

@jgmdev jgmdev commented Sep 26, 2019

So I have spend some time playing around with Samsung Dex and configuring my phone as a development machine that is portable and power efficient. Found about this project in the process
of discovering the best auto completion method that would work properly on vim on android.
But couldn't build the phpctags phar file because it seems that the androind platform or the
php build that is shipped with termux app repository doesn't supports fnmatch.

Took the fnmatch php implementation from the php.net comments section and added a link
on the comments of the source. Sorry if I messed the formatting a bit, but I just ran a default
configuration of php-cs-fixer after having to remove lots of characters that where causing errors
do to copy pasting from php.net site to the Termux android app which seems to not have proper copy/pasting support.

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.

1 participant