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

Copyrights concerning Issue 2643 and Issue 2732 #2735

Merged
merged 6 commits into from Jul 23, 2016

Conversation

JoesCat
Copy link
Contributor

@JoesCat JoesCat commented Jun 25, 2016

Work in progress to "help" resolve issue #2643 so that issue #2732 can proceed.
Basically a lot of reading, plus implying/adding authors who didn't add their names to the credits, and hopefully acknowledged as "okay" by the authors themselves (preferred they did it, but, well...).

@McSinyx
Copy link

McSinyx commented Jun 27, 2016

I think line 12 in fontforge/ofl.c is missing a comma at the end.

@JoesCat
Copy link
Contributor Author

JoesCat commented Jun 27, 2016

Thanks @McSinyx, you're absolutely right.
I looked at it earlier, and the warning from github said something to the effect of ""Powering down the server NOW" or something similar, so I though it was a glitch due to maybe some sort of server patching. My error for not having compiled before committing this copy-n-paste...after all, it's only comments, what could possibly go wrong here?
;-P
Thanks for the catch.

Copyright: 2007 George Williams
bezctx_ff.c - Provide a gateway into FontForge with added modification.
Contributions: Yuri Chornoivan, Jonathan Hanna, Joe Da Silva, Reuben Thomas,
Franke-Trampe
Copy link
Contributor

Choose a reason for hiding this comment

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

Name is wrong.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

On July 19, 2016 02:42:12 PM Frank Trampe wrote:

Name is wrong.

Sorry about that, should have double checked - my error.

Debian lint highlighted some copyright questions.
bezctx_ps.c from spiro-01 is GPL-2+
Patches and mods below (approx) 10 lines are listed as contributions.
Original bezctx_ps.h file is uncommented but is wrapped in GPL-2+ package.
Adding authors is almost 50% of file contents, but it resolves issue 2643.
Optimized versions using lookup tables exist in several text books,
especially when demonstrating hardware, CPUs, and C/assembler code.
George probably didn't want to copyright name to the file, or forgot to
add it, but it's basically data files holding text information - which
can then be used to make a SIL open source license.

This patch resolves issue 2643 about author and file licence missing
when run through debian lint.
…e#2643

Debian Lint highlighted some copyright questions concerning this file.
Since it's a "generated" file (created by another program), adding a
copyright notice might appear a little odd and maybe not be worth the
extra effort (when you come from a generation of programmers where
computers had very little memory or drive storage space, so you had
to put "extra" effort to save memory/storage space).
Since the project was BSD-3-clause in 2001, seems logical to imply same.
Patches and mods below (approx) 10 lines are listed as contributions.
…orge#2643

Debian Lint highlighted some copyright questions concerning these files.
Since they're "generated" files (created by another program), adding a
copyright notice might appear a little odd and maybe not be worth the
extra effort (when you come from a generation of programmers where
computers had very little memory or drive storage space, so you had
to put "extra" effort to save memory/storage space).
Since the project was BSD-3-clause in 2001, seems logical to imply same.
Patches and mods below (approx) 10 lines are listed as contributions.
@JoesCat
Copy link
Contributor Author

JoesCat commented Jul 23, 2016

will continue further...later...

@JoesCat JoesCat merged commit 3e76016 into fontforge:master Jul 23, 2016
@copyninja
Copy link
Contributor

@JoesCat Thanks for this. Btw I've uploaded fontforge to the Debian Experimental for now. We just need to make sure everything builds fine before uploading it to unstable. We are also waiting for the resolution of #1747 before we can upload to unstable.

@JoesCat JoesCat mentioned this pull request Aug 4, 2016
Omnikron13 pushed a commit to Omnikron13/fontforge that referenced this pull request May 31, 2022
* bezctx_ff.c file based on bezctx_ps.c

Debian lint highlighted some copyright questions.
bezctx_ps.c from spiro-01 is GPL-2+
Patches and mods below (approx) 10 lines are listed as contributions.

* bezctx_ff.h file based on bezctx_ps.h

Original bezctx_ps.h file is uncommented but is wrapped in GPL-2+ package.
Adding authors is almost 50% of file contents, but it resolves issue 2643.

* crctab.c is practically public domain

Optimized versions using lookup tables exist in several text books,
especially when demonstrating hardware, CPUs, and C/assembler code.

* ofl.c and ofl.h created 2007-03-12

George probably didn't want to copyright name to the file, or forgot to
add it, but it's basically data files holding text information - which
can then be used to make a SIL open source license.

This patch resolves issue 2643 about author and file licence missing
when run through debian lint.

* ArabicForms.c created in 2001 - call it BSD-3-clause - issue fontforge#2643

Debian Lint highlighted some copyright questions concerning this file.
Since it's a "generated" file (created by another program), adding a
copyright notice might appear a little odd and maybe not be worth the
extra effort (when you come from a generation of programmers where
computers had very little memory or drive storage space, so you had
to put "extra" effort to save memory/storage space).
Since the project was BSD-3-clause in 2001, seems logical to imply same.
Patches and mods below (approx) 10 lines are listed as contributions.

* utype.c, utype.h created in 2001 - call it BSD-3-clause - issue fontforge#2643

Debian Lint highlighted some copyright questions concerning these files.
Since they're "generated" files (created by another program), adding a
copyright notice might appear a little odd and maybe not be worth the
extra effort (when you come from a generation of programmers where
computers had very little memory or drive storage space, so you had
to put "extra" effort to save memory/storage space).
Since the project was BSD-3-clause in 2001, seems logical to imply same.
Patches and mods below (approx) 10 lines are listed as contributions.
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

4 participants