Re: [PATCH 1/1] media: Drop FSF's postal address from the source code files

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Sakari,

Thank you for the patch.

On Friday 28 Oct 2016 15:43:53 Sakari Ailus wrote:
> Drop the FSF's postal address from the source code files that typically
> contain mostly the license text. The patch has been created with the
> following command without manual edits:
> 
> git grep -l "675 Mass Ave\|59 Temple Place\|51 Franklin St" -- \
> 	drivers/media/ include/media|while read i; do i=$i perl -e '
> open(F,"< $ENV{i}");
> $a=join("", <F>);
> $a =~ s/[ \t]*\*\n.*You should.*\n.*along with.*\n.*(\n.*USA.*$)?\n//m
> 	&& $a =~ s/(^.*)Or, (point your browser to) /$1To obtain the license,
> $2\n$1/m; close(F);

Do we really need to keep the "To obtain the license, point your browser 
to..." text ?

In any case,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

with a quick review of the loooooong patch directly from your tree.

> open(F, "> $ENV{i}");
> print F $a;
> close(F);'; done
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
> Hi,
> 
> I found this in a few places and decided to remove them all. The script
> could be useful elsewhere, too.
> 
> The actual patch can be found here, it appears to be too large to be
> accepted by vger:
> 
> <URL:http://git.retiisi.org.uk/?p=~sailus/linux.git;a=commitdiff;h=refs/head
> s/fsf-address>

-- 
Regards,

Laurent Pinchart