FFmpeg - 修改提交流程

1 搭建自己的修改环境

修改本身是不可以直接push进FFmpeg的原始仓库里的,所以首先应该Fork一个自己的仓库并建立专用的提交分支。

2 开始进行修改

2.1 准备工作
  • 确保base最新!!!
  • 修改前先编译运行,成功后再进行修改
  • 及时clean,不建议保留无关文件
  • 阅读官方引导文档:FFmpeg Developer
2.2 进行修改
  • 修改时尽量与官方格式保持一致
  • 不允许添加任何行尾空白字符(主要是空格)
  • Makefile外不允许使用TAB
  • API更改需要写入所属模块的logdoc
  • 更改需要在相关的version.h中进行迭代
  • 遵从官方的其他格式指引:Coding Rules
迭代分级:
VERSION_MAJOR - 巨大更改,本次修改后失去向前兼容性
VERSION_MINOR - 正常更改,本次修改影响版本API
VERSION_MICRO - 微小更改,本修修改无其他影响
高版本修改时需要重置低位,如 MINOR++ => MICRO =back-to=> 100
  • 更改尽可能小,不需要划分成原子更改但必须保真每一次更改为当前最简更改
  • 更改不能破坏原功能,完成后应该可以重新编译运行并通过FATE测试:make fate
2.3 进行提交
  • 注册ffmpeg-devel邮箱列表:ffmpeg-devel info page
  • 使用git commit -s -m "[change part] : [change message]"签名后提交
  • 使用git format-patch -v[x] -[n]生成补丁,x是版本,n是打包commit的数量
  • 使用自带工具tool/pacheck检查patch规范,按提示进行修改
  • 按照官方check list进行检查:Patch submission checklist
  • 使用git send-email --compose --no-chain-reply-to --suppress-from --to ffmpeg-devel@ffmpeg.org 0001-xxx.patch
示例:
$ git commit -s -m "lavc, doc: add avs2 codec"
... # other changes
$ git commit -s -m "lavc,doc,configure: add libdavs2 video decoder"
$ git format-patch -v3 -2
$ git send-email --compose --no-chain-reply-to --suppress-from --to ffmpeg-devel@ffmpeg.org v3-0001-lavc-doc-add-avs2-codec.patch v3-0002-lavc-doc-configure-add-libdavs2-video-decoder.patch
... # other steps
  • 必须确保通过了FATE测试
  • 确保send-email返回正确信息
2.4 后续内容
  • 提交成功后应该可以在FFmpeg Patch Work中查找到自己的提交信息,没有查到或不全不要急着重复提交,延时很大
  • 时刻关注注册mail-list的邮箱账户,找到与自己有关的,修改/回复Reviewer的问题,可以直接用邮箱客户端引用来信,并在其基础上添加留言
  • 最终确认接受后不需要自己提交

流程笔记,待更新,欢迎留言指正

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 213,711评论 6 493
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,079评论 3 387
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 159,194评论 0 349
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,089评论 1 286
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,197评论 6 385
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,306评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,338评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,119评论 0 269
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,541评论 1 306
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,846评论 2 328
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,014评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,694评论 4 337
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,322评论 3 318
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,026评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,257评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,863评论 2 365
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,895评论 2 351

推荐阅读更多精彩内容