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

Code lifted from Keymaster without attribution #126

Open
madrobby opened this issue Feb 7, 2020 · 10 comments
Open

Code lifted from Keymaster without attribution #126

madrobby opened this issue Feb 7, 2020 · 10 comments

Comments

@madrobby
Copy link

madrobby commented Feb 7, 2020

Hey there.

I'm a disappointed that this library lifts code directly from my Keymaster library without any attribution.
Here's my library: https://github.com/madrobby/keymaster

Your library is essentially a substantial clone, reusing most or all of my code.

The MIT license requires that you include the copyright statement. Please add this to the library or I will have to report this to Github.

@jaywcjlove
Copy link
Owner

@jaywcjlove
Copy link
Owner

@madrobby 请告知我添加到什么位置,我添加上去。

@jaywcjlove jaywcjlove pinned this issue Feb 9, 2020
@jaywcjlove
Copy link
Owner

jaywcjlove commented Feb 9, 2020

@madrobby
Copy link
Author

madrobby commented Feb 9, 2020

I don’t speak Chinese, so I don’t know what you’re saying.

The MIT license requires that your include the whole license in any substantial copy of the software; in your case you should include the whole license file (not just a link) in both your README and in your LICENSE file.

Note that this isn’t optional or a goodwill gesture, the license you agreed to when using the code requires you to do so.

@jaywcjlove
Copy link
Owner

My English description is not accurate. Who will submit a PR?

@PabloLION
Copy link

PabloLION commented Aug 29, 2020

I checked madrobby/keymaster and yes they are similar.
In the Chinese readme of Hotkey.js, the owner said it's a lift from and imitation of madrobby/keymaster. And seems that the keymaster repo is out of maintenance, there is no NPM install guide(though I see it is possible in the "issue"), no demo ( I just saw the test.html, although it is not technically a demo with right format) , no Chinese support, etc.
I admit that it is not righteous to copy some code and re-publish it. But when the origin repo is ignoring the PRs since years ago, I think the owner did what he could.

@madrobby
Copy link
Author

I checked madrobby/keymaster and yes they are similar.
In the Chinese readme of Hotkey.js, the owner said it's a lift from and imitation of madrobby/keymaster. And seems that the keymaster repo is out of maintenance, there is no NPM install guide(though I see it is possible in the "issue"), no demo ( I just saw the test.html, although it is not technically a demo with right format) , no Chinese support, etc.
I admit that it is not righteous to copy some code and re-publish it. But when the origin repo is ignoring the PRs since years ago, I think the owner did what he could.

It doesn’t matter, at all, if a project is not maintained or defunct, that doesn’t change the requirements of the licensing.

Again: The MIT license requires that your include the whole license in any substantial copy of the software; in your case you should include the whole license file (not just a link) in both your README and in your LICENSE file.

Note that this isn’t optional or a goodwill gesture, the license you agreed to when using the code requires you to do so.

@phodal phodal mentioned this issue Aug 29, 2020
@phodal
Copy link
Contributor

phodal commented Aug 29, 2020

If you had copy code, you also need to add LICENSE in code.

Chinese version: 如果你直接复制了代码,还需要在对应的代码加上 MIT LICENSE。

Examples in VSCode:

/*---------------------------------------------------------
 * Copyright (C) Microsoft Corporation. All rights reserved.
 *--------------------------------------------------------*/

import { IRawGrammar } from './types';
import * as plist from './plist';
import { DebugFlags } from './debug';
import { parse as manualParseJSON } from './json';

@PabloLION
Copy link

@madrobby. I totally agree. I think one should have the control and possession of what he created. You can report this to GitHub, or PR the README.

@yellow1912
Copy link

I see this one is updated?

https://github.com/jaywcjlove/hotkeys/blob/master/LICENSE

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

No branches or pull requests

5 participants