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

Project has no license? #295

Closed
beetleb opened this issue Feb 3, 2014 · 4 comments
Closed

Project has no license? #295

beetleb opened this issue Feb 3, 2014 · 4 comments

Comments

@beetleb
Copy link

@beetleb beetleb commented Feb 3, 2014

Apologies if I missed it, but I can't find a license on this project. Can you please add one?

Thanks.

@Fuco1
Copy link
Owner

@Fuco1 Fuco1 commented Feb 4, 2014

What do you mean? It's in every file on top.

@beetleb
Copy link
Author

@beetleb beetleb commented Feb 4, 2014

Oh, OK - I see it in the .el files.

I'd suggest having an explicit COPY or LICENSE file in the root directory for easy convenience. In principle, you could have a different license in each file, and it'd be the user's responsibility to check each file, which is a pain. If instead you explicitly say that the whole project is GPL v3 in one main file, it spares the user some headaches.

Also, it says ";; You should have received a copy of the GNU General Public License
;; along with Smartparens. If not, see http://www.gnu.org/licenses/." but I don't think you have it in the repository. Creating a COPY or LICENSE file with the full GPL in it would satisfy this.

Anyway, not trying to be a license nazi or anything. I just needed to know the license to ensure I could use it at work, and you've answered my query. Feel free to close the issue if you don't want to take further action.

@beetleb
Copy link
Author

@beetleb beetleb commented Feb 4, 2014

BTW, another way to do it is just to use Github to apply a license:

Fuco1 added a commit that referenced this issue Feb 4, 2014
@Fuco1
Copy link
Owner

@Fuco1 Fuco1 commented Feb 4, 2014

Fair enough.

@Fuco1 Fuco1 closed this Feb 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.