Skip to content

Add missing license#30

Open
licenser1971 wants to merge 1 commit intodomsec:masterfrom
licenser1971:patch-1
Open

Add missing license#30
licenser1971 wants to merge 1 commit intodomsec:masterfrom
licenser1971:patch-1

Conversation

@licenser1971
Copy link
Copy Markdown

Hello,

thank you for publishing open-source work on GitHub!

It seems your project is missing some licensing information.

The readme, logo, and project name have been copied from johansatge/jpeg-autorotate, which is fine, because its MIT license allows derivative work. But in order to respect said license, you also need to credit the original project; hence, this pull request.

(It may not be obvious, but the name of a project, its readme, and all other assets are part of it, and are covered by its license. A license does not only cover the source code of a project!)

Thank you and have a nice day!


2

1

3

4

@domsec
Copy link
Copy Markdown
Owner

domsec commented Nov 7, 2021

Hi,

First and foremost, sorry for getting to this PR later than I would have liked to. Secondly, I'd like to thank you for bringing to my attention the issues outlined in this PR. I have provided my thoughts on the issues raised.

Logo

I fully agree the logo is very similar. As such, the current logo for this project will be changed.

ReadMe

A revamp of the ReadMe has been planned for quite some time. Adjusts will be made to sections that are too similar to the mentioned project.

Name

A quick Google search leads to numerous open-source projects and websites with either a similar or identical name, most of which were created before the project you mentioned. Therefore, stating the mentioned project as the original project would be disingenuous to the others that came before it. I would feel more comfortable and think it would be best to give credit in general perspective to the similarly named projects instead of just a single one. I'm open to any suggestions you may have.

@licenser1971
Copy link
Copy Markdown
Author

Hello,
thanks you for your reply.

I don't think the issue here is the similarity; the mentioned project was just clearly used as a base for yours, which is totally fine, because it's license allows it. but it is also the reason why it should be credited.

Changing the Readme to make it different doesn't cancel the fact that it is still a "derivative work", as stated in the original license. The same thing applies to the logo.

I agree the name of the project is used by many others people. The point is not to associate ownership of the name to a specific project.

Copy link
Copy Markdown
Owner

@domsec domsec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After considering your comments, I believe my suggested modifications provide a more comprehensive and accurate credit to the original project.

Since it has been some time since our last conversation in this PR (apologies for such a length of time), I will plan to push these changes myself after 2 weeks if such modifications are not pushed.

Comment thread README.md

### Original project

- [johansatge/jpeg-autorotate](https://github.com/johansatge/jpeg-autorotate)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be best to clearly state where the inspiration for this library originated from and the elements that are either the same, similar, or modified from the original project, as mentioned in your comments.

Suggested change
- [johansatge/jpeg-autorotate](https://github.com/johansatge/jpeg-autorotate)
The inspiration for this library (including the name and logo) originated from [johansatge/jpeg-autorotate](https://github.com/johansatge/jpeg-autorotate), available under the [MIT License](https://github.com/johansatge/jpeg-autorotate/blob/master/license.md). Additionally, portions of the README file (including the project description and features) are derived from this project, but with some modifications.

Comment thread README.md
- Stroke width decreased for all icons by 0.25
- Detached arrows from [sync-alt](https://fontawesome.com/icons/sync-alt?style=solid)

### Original project
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be best to place this subsection first, before the Libraries subsection.

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.

2 participants