[x264-devel] [PATCH 1/2] Add FFmpeg-style silent build rules

Stephen Hutchinson qyot27 at gmail.com
Sat Apr 5 22:13:41 CEST 2014


On Fri, Apr 4, 2014 at 2:37 PM, James Almer <jamrial at gmail.com> wrote:
> Use git blame.
> As long as there were no cosmetic reindents/renamings that modified the lines
> in question, it should show you the commit that introduced them.
> If something changed the lines after the initial commit then it gets a bit more
> complex.
>

Turns out that that is the case, but traversing backwards through
the blame log shows that aside from later changes to which variables get
put under BRIEF or SILENT (which also happened to bring the patch in line
with what x264 needs), all the important parts of the ifndef V block were
Mans'.  The commit authorship is fixed locally (and all parts of that are now
in the first patch only), I'll send the updated patches shortly.

The configure option is now gone and so is the newline hack (turns out
that using a non-BRIEF variable completely resolves what it was meant to
fix, so CC_SILENT got kept but renamed to CC_VERBOSE).

The install rules patch is deprecated, and has been replaced with a
much more straightforward method.


More information about the x264-devel mailing list