Media/WebRTC/libwebrtc Update Process: Difference between revisions

Mention build error due to changes in build (gn) scripts as a third category
(Broke out a separate "Review" section)
(Mention build error due to changes in build (gn) scripts as a third category)
Line 64: Line 64:
=== Operational notes ===
=== Operational notes ===
* Running <code>prep-repo.sh</code> with a fresh github clone of moz-libwebrtc will likely display instructions on how to add recent moz-central changes made in <code>third_party/libwebrtc</code> to the top of the patch stack in github.  This is expected and necessary for successfully vendoring changes into <code>third_party/libwebrtc</code>.
* Running <code>prep-repo.sh</code> with a fresh github clone of moz-libwebrtc will likely display instructions on how to add recent moz-central changes made in <code>third_party/libwebrtc</code> to the top of the patch stack in github.  This is expected and necessary for successfully vendoring changes into <code>third_party/libwebrtc</code>.
* Two main types of errors will cause <code>loop-ff.sh</code> to exit; the first is a rebase conflict, the second is a build error due to api changes in upstream code.
* Three main types of errors will cause <code>loop-ff.sh</code> to exit: the first is a git rebase conflict, the second is a build error due to changes in build (gn) scripts, the third is a build error due to API changes in upstream code.
* When making changes to fix build issues in moz-libwebrtc (any changes under third_party/libwebrtc), mercurial commit messages follow the pattern:
* When making changes to fix build issues in moz-libwebrtc (any changes under third_party/libwebrtc), mercurial commit messages follow the pattern:
  Bug {current-fast-forward-bug-number} - (fix-{upstream-sha}) {description}
  Bug {current-fast-forward-bug-number} - (fix-{upstream-sha}) {description}
24

edits