Skip to content

Correct obsolete FSF address in all license notices#879

Open
pemensik wants to merge 1 commit into
avahi:masterfrom
pemensik:license-address
Open

Correct obsolete FSF address in all license notices#879
pemensik wants to merge 1 commit into
avahi:masterfrom
pemensik:license-address

Conversation

@pemensik

@pemensik pemensik commented Mar 4, 2026

Copy link
Copy Markdown
Member
    Fixed with following sed script:
    s|License along with avahi; if not, see <https://www.gnu.org/licenses/>.|License along with avahi; if not, see <https://www.gnu.org/licenses/>.|
    /^[# ] Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307/ d
    /^[# ] USA./ d
    s|along with avahi; if not, see <https://www.gnu.org/licenses/>.|along with avahi; if not, see <https://www.gnu.org/licenses/>.|
    /^[# ] Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA./ d
    /^[# ] Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA/ d
    /^[# ] 02111-1307 USA./ d

Fixed with following sed script:
s|License along with avahi; if not, see <https://www.gnu.org/licenses/>.|License along with avahi; if not, see <https://www.gnu.org/licenses/>.|
/^[# ] Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307/ d
/^[# ] USA./ d
s|along with avahi; if not, see <https://www.gnu.org/licenses/>.|along with avahi; if not, see <https://www.gnu.org/licenses/>.|
/^[# ] Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA./ d
/^[# ] Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA/ d
/^[# ] 02111-1307 USA./ d
@pemensik pemensik added bug documentation Change of text outside of the code. labels Mar 4, 2026
@evverx

evverx commented Mar 4, 2026

Copy link
Copy Markdown
Collaborator

Please add the rationale behind this change to the commit message. For example systemd/systemd@4a8c41f came with

rpmlint complains about the old address, even though it totally
doesn't matter and nobody cares.

@evverx

evverx commented Mar 4, 2026

Copy link
Copy Markdown
Collaborator

Also I'm not sure I understand how that sed script turned

  <signature of Ty Coon>, 1 April 1990
  Ty Coon, President of Vice

to

  <signature of Moe Ghoul>, 1 April 1990
  Moe Ghoul, President of Vice

and where a bunch of other changes unrelated to the address came from.

# License along with avahi; if not, write to the Free Software
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
# USA.
# License along with avahi; if not, see <https://www.gnu.org/licenses/>.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If you're going to touch all the files like this anyway, you might consider adding an SPDX-License-Identifier: line to make it machine-readable as well. (Whether avahi wants SPDX, I don't know, this is a drive-by review).

@adriaandegroot

Copy link
Copy Markdown

Also I'm not sure I understand how that sed script turned

  <signature of Ty Coon>, 1 April 1990
  Ty Coon, President of Vice

to

  <signature of Moe Ghoul>, 1 April 1990
  Moe Ghoul, President of Vice

and where a bunch of other changes unrelated to the address came from.

The sample text on the FSF website at https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt is different from whenever LICENSE was first imported. Nowadays the sample text upstream actually says Moe Ghoul. Note that the SPDX license text sample for LGPL version 2.1 is the "old" one, which matches what is in the avahi repository.

@evverx

evverx commented May 3, 2026

Copy link
Copy Markdown
Collaborator

The sample text on the FSF website at https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt is different from whenever LICENSE was first imported

The commit message should mention that. Currently it just says that the sed script was run and that script couldn't change "Ty Coon" to "Moe Ghoul" obviously.

If you're going to touch all the files like this anyway, you might consider adding an SPDX-License-Identifier: line to make it machine-readable as well

I think with SPDX-License-Identifier the text can be removed as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug documentation Change of text outside of the code. needs-reporter-feedback

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants