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

Use proper MIT license #21

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

Conversation

johnnyshields
Copy link

@johnnyshields johnnyshields commented Mar 2, 2025

This project has a modified form of the MIT license which makes it a PITA to use at companies. Even though it would be basically fine to use, any project which does not have a FOSS license following the exact wording of the license needs a customized legal review (i.e. you have to pay lawyers.)

Specifically this language problematic:

Any publication of projects using uroman shall acknowledge its use: "This project uses the universal romanizer software 'uroman' written by Ulf Hermjakob, USC Information Sciences Institute (2015-2020)".
Bibliography: Ulf Hermjakob, Jonathan May, and Kevin Knight. 2018. Out-of-the-box universal romanization tool uroman. In Proceedings of the 56th Annual Meeting of Association for Computational Linguistics, Demo Track.

The clear intent of this language is to ensure the work (uroman) is cited in academic papers. It's better to move this requirement to the README instead.

Moving this wording to the README does not materially change the spirit of the license, nor does it impose any extra burden on users, so I think it's fine for the project maintainer to unilaterally accept it.

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