[CinCV TNG] [PATCH] Removed internal fonts

Johannes Sixt j6t at kdbg.org
Mon Oct 26 08:43:28 CET 2015


Am 26.10.2015 um 04:03 schrieb Sérgio M. Basto:
> Copied from einar branch.

Thanks for your efforts to mention the real author of this change. 
However, with git, you can do better. Here's how I would have done it in 
this case:

   git cherry-pick 22060fce5973    # the font removal patch
   git cherry-pick -n 772c7bec1f75 # the configure.ac fixup
   git commit --amend              # keep 1st patch's commit details
   git format-patch -D -1 HEAD

and then use git-send-email to send out the patch or ensure to keep the 
From:-Line that is in the format-patch file, separated by a blank line 
above the commit message summary.

The -D in the format-patch instruction generates an irreversible patch. 
It basically records only the deletion of files, but does not include 
the (binary) patch text. Even though the patch would not be applicable 
as is, it shows which files must be removed.

(I would have submitted the patch for real, but my Cinelerra tree is 
currently unavailable.)

> Fonts are old and probably not redistributable
> Modern distributions have enough scaleable fonts.

In general, I agree to this change.

-- Hannes



More information about the Cinelerra mailing list