[FFmpeg-devel] [PATCHv3] On2 VP7 decoder
James Almer
jamrial at gmail.com
Sun Mar 30 10:47:27 CEST 2014
On 30/03/14 5:14 AM, Vittorio Giovara wrote:
> Nice troll attempt, but won't work. Rather than having an horrible
> source history like ffmpeg where you see every commit alternated with
> a merge (btw why the name qatar? are you afraid to use Libav name?)
> and makes git blame/history tracking MUCH harder, Libav prefers to
> have linear history with *single* *atomic* patches (as much as
> possible). This is so that you can see what's changed in a single
> revision, rather than having to navigate countless (and possibly
> distant) commits.
Just wanted to make a comment about this.
When Libav ported Clément's and Ronald's VP9 decoder from ffmpeg, it was indeed
done in a single patch/commit that included a handful of fixes and some cosmetic
changes by four or so developers.
Both authors asked in Libav's list if they could get a list of the changes and
know what developer contributed with what change, to properly credit them when
they ported the fixes back to ffmpeg. Instead of a reply with the information, or
even a negative for that matter, they got absolutely nothing. The request was
completely ignored.
They had to waste time using diff to find the actual code changes (Filtering all
the cosmetics which probably made the diff output pretty big), split them and
commit them to ffmpeg's tree, each with a comment listing several developers and
saying that the fix was by one of them.
That attitude doesn't really support your claims of trying to make things easier
for people.
More information about the ffmpeg-devel
mailing list