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

Reformat License.txt so GitHub recognizes it #687

Merged
merged 3 commits into from
Sep 8, 2018
Merged

Conversation

stakx
Copy link
Contributor

@stakx stakx commented Sep 6, 2018

This follows up on #683.

  • The License.txt is reformatted so that GitHub recognizes the license as BSD 3-Clause. This reformatting consists mostly of line break changes.

  • All source code files are updated to include a two-line copyright notice referring to License.txt. Much shorter than the current 39-line texts.

  • While we're changing the first line of every source code file, let's take the opportunity to normalize UTF-8 BOMs at the same time. (That is, remove them everywhere).

Closes #669.

/cc @kzu, @retslig

@stakx
Copy link
Contributor Author

stakx commented Sep 6, 2018

@kzu, I pinged you in case you want to take a final look. More of a formality, really, since we've already discussed this earlier in #683. If I don't hear back from you, I'll merge this in a few days' time.

NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
SUCH DAMAGE.
* Neither the names of the copyright holders nor the names of its
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This line differs from both the original BSD 3-Clause template (name of the copyright holder → plural form) as well as from the previous version of License.txt (which relisted the copyright holders' names verbatim).

@retslig
Copy link

retslig commented Sep 7, 2018

Looks good to me. Thanks

@stakx stakx merged commit 4abde50 into devlooped:master Sep 8, 2018
@stakx stakx deleted the license branch September 8, 2018 07:58
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.

None yet

2 participants