-
Notifications
You must be signed in to change notification settings - Fork 16
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
Licensing issue #35
Comments
ghost
assigned chobie
Dec 15, 2013
Thanks, I'll check license issues. I'll plan to switch to hoedown library in this winter vacation. |
chobie
added a commit
that referenced
this issue
Dec 15, 2013
chobie
added a commit
that referenced
this issue
Dec 15, 2013
FYI, sundown extension added on http://blog.famillecollet.com/pages/PECL-extensions-RPM-status |
This issue should be closed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
Working on packaging sundown extension for Fedora / RHEL / CentOS...
Some minor issues
Extension have been relicensed under PHP License, OK, but some header still refer to previous MIT LIcense (ex examples/Man.php)
Some files have no license header (ex php_sundown.h)
sundown extension have no LICENSE file, but license text is include in README.mardown, which state ".. the above copyright notice and this permission notice appear in all copies.". So please add this file in "package.xml" (with role="doc").
In all case, even if the extension sources are PHP, the resulting binary are "MIT and PHP"
N.B. this extension is not suitable for official repo because of bundled sundown library. I will wait for the switch to hoedown to see if we can build a shared library and use it (meanwhile, be in my personal repo).
The text was updated successfully, but these errors were encountered: