Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Pierre-Anthony Lemieux <pal@sandflow.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Subject: Re: [FFmpeg-devel] [RFC PATCH 2/2] tests/fate/source-check: Relax BSD licence check
Date: Sat, 25 May 2024 13:43:48 -0700
Message-ID: <CAF_7JxAus1YJV90Zx5dSbR8pQ3VzE4w7UKQTLyJfxRashisDZA@mail.gmail.com> (raw)
In-Reply-To: <GV1P250MB0737CB6F11B87A647FBD0BE58FEB2@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM>

On Wed, May 22, 2024 at 4:38 AM Andreas Rheinhardt
<andreas.rheinhardt@outlook.com> wrote:
>
> Several files already had standard license header (namely
> 2-clause BSD files), yet due to the 80 char line length limit,
> they were not treated as such by source-check.sh (which
> fate-source uses). Therefore relax the BSD check.

LGTM

>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> This is the first variant. The second variant follows shortly.
>
>  tests/fate/source-check.sh | 2 +-
>  tests/ref/fate/source      | 5 -----
>  2 files changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/tests/fate/source-check.sh b/tests/fate/source-check.sh
> index 658823fc0b..4d7e175784 100755
> --- a/tests/fate/source-check.sh
> +++ b/tests/fate/source-check.sh
> @@ -11,7 +11,7 @@ git grep -L -E "This file is part of FFmpeg|This file is part of libswresample|"
>  "Permission is hereby granted to use, copy, modify, and distribute this|"\
>  "Permission is granted to anyone to use this software for any purpose|"\
>  "This work is licensed under the terms of the GNU GPL|"\
> -"Redistribution and use in source and binary forms, with or without modification|"\
> +"Redistribution and use in source and binary forms, with or without|"\
>  "This library is free software; you can redistribute it and/or|"\
>  "This program is free software; you can redistribute it and/or modify|"\
>  "Licensed under the Apache License|"\
> diff --git a/tests/ref/fate/source b/tests/ref/fate/source
> index 723e2e06c7..a3beb35093 100644
> --- a/tests/ref/fate/source
> +++ b/tests/ref/fate/source
> @@ -1,15 +1,10 @@
>  Files without standard license headers:
>  libavcodec/file_open.c
> -libavcodec/ilbcdata.h
> -libavcodec/ilbcdec.c
>  libavcodec/interplayacm.c
>  libavcodec/log2_tab.c
>  libavcodec/reverse.c
> -libavcodec/riscv/startcode_rvb.S
> -libavcodec/riscv/startcode_rvv.S
>  libavdevice/file_open.c
>  libavdevice/reverse.c
> -libavfilter/af_arnndn.c
>  libavfilter/file_open.c
>  libavfilter/log2_tab.c
>  libavformat/bitstream.c
> --
> 2.40.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

      parent reply	other threads:[~2024-05-25 20:44 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-22 11:03 [FFmpeg-devel] [PATCH 1/2] doc/mips: Update list of files with MIPS copyright notice Andreas Rheinhardt
2024-05-22 11:04 ` [FFmpeg-devel] [RFC PATCH 2/2] tests/fate/source-check: Relax BSD licence check Andreas Rheinhardt
2024-05-22 12:40   ` Rémi Denis-Courmont
2024-05-25 20:43   ` Pierre-Anthony Lemieux [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAF_7JxAus1YJV90Zx5dSbR8pQ3VzE4w7UKQTLyJfxRashisDZA@mail.gmail.com \
    --to=pal@sandflow.com \
    --cc=andreas.rheinhardt@outlook.com \
    --cc=ffmpeg-devel@ffmpeg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git