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

The name of this tool #175

Closed
sj-i opened this issue Feb 23, 2022 · 1 comment · Fixed by #223
Closed

The name of this tool #175

sj-i opened this issue Feb 23, 2022 · 1 comment · Fixed by #223
Milestone

Comments

@sj-i
Copy link
Member

sj-i commented Feb 23, 2022

#84 needs extracting the definition of sapi_globals_struct from php-src/main/SAPI.h, and this file is licensed under the PHP license.
And the 4th clause of this license forbids the use of names like "PHP Foo" or "phpfoo" without permission.
Currently, the name of this tool is "php-profiler", so this tool cannot use the definition from files licensed under the PHP license.

There are some options.

a. Changing the name of this tool
b. Getting written permission from [email protected]
c. Avoiding the license problem through binary analysis

It seems that phpspy uses the plan c.
I'm not sure whether this is a legit solution.

I've already sent an email to [email protected] asking the permission. But it seems that the mail doesn't appear in any public mailing lists available, so I guess only a limited number of people can read it, thus it unlikely guides the consensus from the community.

So the last option I have to speed up the tool (via #84) is changing the name of this tool.
Currently, I can't think of a good name for it though. :(

@sj-i sj-i pinned this issue Feb 23, 2022
@sj-i
Copy link
Member Author

sj-i commented Feb 24, 2022

strrev('php-profiler') === 'reliforp-php'

So "Reliforp" and "Reli (for ph*)" are the current candidates.

@sj-i sj-i mentioned this issue Oct 5, 2022
@sj-i sj-i closed this as completed in #223 Oct 5, 2022
@sj-i sj-i added this to the 0.6.0 milestone Oct 5, 2022
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 a pull request may close this issue.

1 participant