roll_dep_svn.py fails when same deps exists in multiple deps_os entries and uses variable |
|
Issue descriptionI recently ran into this issue in WebRTC, which is also easily reproducible in Chromium by rolling to an older lss revision: $ roll-dep-svn third_party/lss 348bdf8d32b37c8fb2627df7a0a977d1d640e1fc Pinning src/third_party/lss to revision 348bdf8d32b37c8fb2627df7a0a977d1d640e1fc in ./DEPS $ git diff DEPS diff --git a/DEPS b/DEPS index c36d4dae62be..213269dea509 100644 --- a/DEPS +++ b/DEPS @@ -84,7 +84,7 @@ vars = { # Three lines of non-changing comments so that # the commit queue can handle CLs rolling lss # and whatever else without interference from each other. - 'lss_revision': '63f24c8221a229f677d26ebe8f3d1528a9d787ac', + 'lss_revision': '348bdf8d32b37c8fb2627df7a0a977d1d640e1fc'348bdf8d32b37c8fb2627df7a0a977d1d640e1fc7df7a0a977d1d640e1fc', # Three lines of non-changing comments so that # the commit queue can handle CLs rolling NaCl # and whatever else without interference from each other. The resulting change is invalid and makes autorolling fail for this dependency. The easiest workaround is to modify DEPS to avoid having the same deps multiple times. I think this only affects such deps that share a common variable for the revision.
,
Apr 6 2017
Since https://codereview.chromium.org/2297513002/, I've considered roll-dep-svn to be owned by V8 team, as they are the only users of it. The best solution to this bug is probably to switch to using the non-svn version of deps roller.
,
Apr 18 2017
machenbach: would you consider fixing this or have any ideas on this? It cost me quite some time.
,
Apr 18 2017
Re 2: Seems like webrtc is a user too, which makes them an owner as well? :) It's not broken for our auto-roller and my long-term plan is to move away from it and/or improve roll-dep with the missing features. |
|
►
Sign in to add a comment |
|
Comment 1 by kjellander@chromium.org
, Apr 6 2017