search

astrofrog / psrecord

Record the CPU and memory activity of a process :chart_with_upwards_trend:
BSD 2-Clause "Simplified" License
595 stars 77 forks source link

specify license more clearly and use proper license headers #4

Closed cgoldberg closed 10 years ago

cgoldberg commented 10 years ago

Hi. I'm looking at your project and it doesn't specify which license you are using. From a glimpse of the LICENSE file, i think it's a BSD-like license? Making this very clear is important to gain contributors and users.

You should use a standard license (GPL, BSD, MIT, etc), specify that in the license file according to their procedure, and also mention it in each source file header. Each source file header should also include copyright info along with license info.

Your project is cool and I'd like to contribute. Let me know if you'd accept a pull request with the licensing/copyrights fixes... and I can go ahead and do it.

regards,

-Corey

astrofrog commented 10 years ago

@cgoldberg - thanks for the feedback. Please do feel free to open a pull request with such fixes!

astrofrog commented 10 years ago

The LICENSE should indeed be a BSD license. I started off with the three clauses but removed the third because there is no 'organization'. Is there a version of the third clause for when there is no organisation?

astrofrog commented 10 years ago

Just saw the text is not wrapper in the LICENSE - I usually do wrap it, but not sure what happened there. Feel free to wrap.

astrofrog commented 10 years ago

@cgoldberg - just to check, are you still planning on submitting a couple of pull requests? If not, then I can try tidying this up.

astrofrog commented 10 years ago

The original issue should be fixed in 5ca4bae62fb4850615dbb2bea18297cb1da8b2af.