git cl issue gives wrong number |
|||||||
Issue descriptionVersion: Ubuntu 12.04 OS: Linux What steps will reproduce the problem? (1) on issue https://codereview.chromium.org/2263113003/ (2) commit patch locally (3) git cl issue What is the expected output? git should show issue number 2263113003 What do you see instead? Issue number: -2031854293 (https://codereview.chromium.org/-2031854293 Not able to upload the issue because of this.
,
Aug 23 2016
this cl is causing the above git issue https://codereview.chromium.org/2259043002 if i reset the depot_tools before this cl it works fine.
,
Aug 23 2016
,
Aug 23 2016
Thanks for report! Clearly, 32-bit integer overflow inside "git config". Is your git 32-bit binary? In any case, I'll make a CL to fix this in Python.
,
Aug 23 2016
Cl to fix this: https://codereview.chromium.org/2272613002
,
Aug 23 2016
,
Aug 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+/33a46ffb3f0e7f54f55d269795b52d34b2f1be61 commit 33a46ffb3f0e7f54f55d269795b52d34b2f1be61 Author: tandrii <tandrii@chromium.org> Date: Tue Aug 23 12:53:40 2016 git cl: workaround against integer overflows of git config. R=maruel@chromium.org,phajdan.jr@chromium.org BUG= 640115 Review-Url: https://codereview.chromium.org/2272613002 [modify] https://crrev.com/33a46ffb3f0e7f54f55d269795b52d34b2f1be61/git_cl.py [modify] https://crrev.com/33a46ffb3f0e7f54f55d269795b52d34b2f1be61/tests/git_cl_test.py
,
Aug 23 2016
I think this should be resolved now. I'd really appreciate if you can verify: * your git is actually 32 bit binary $ file `which git` * after syncing depot_tools to 33a46ffb, does things work as befofe?
,
Aug 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/205936cfa527c41dd5be9bfb268d92c5974ad607 commit 205936cfa527c41dd5be9bfb268d92c5974ad607 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Aug 23 13:08:15 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/33a46ffb3f0e7f54f55d269795b52d34b2f1be61 git cl: workaround against integer overflows of git config. (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= 640115 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2270803002 [modify] https://crrev.com/205936cfa527c41dd5be9bfb268d92c5974ad607/infra/config/recipes.cfg
,
Aug 23 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/9e425748f792cce1cd794c6296ccf089d99b63e8 commit 9e425748f792cce1cd794c6296ccf089d99b63e8 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Aug 23 13:14:04 2016
,
Aug 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/03b459eaad26b33c434f4b99302d4c50805f6849 commit 03b459eaad26b33c434f4b99302d4c50805f6849 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Aug 23 13:24:40 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/205936cfa527c41dd5be9bfb268d92c5974ad607 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/33a46ffb3f0e7f54f55d269795b52d34b2f1be61 git cl: workaround against integer overflows of git config. (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= 640115 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2270813002 [modify] https://crrev.com/03b459eaad26b33c434f4b99302d4c50805f6849/infra/config/recipes.cfg
,
Aug 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/03b459eaad26b33c434f4b99302d4c50805f6849 commit 03b459eaad26b33c434f4b99302d4c50805f6849 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Aug 23 13:24:40 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/205936cfa527c41dd5be9bfb268d92c5974ad607 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/33a46ffb3f0e7f54f55d269795b52d34b2f1be61 git cl: workaround against integer overflows of git config. (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= 640115 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2270813002 [modify] https://crrev.com/03b459eaad26b33c434f4b99302d4c50805f6849/infra/config/recipes.cfg
,
Aug 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/98d43381f1c6249486df9b61c4cc40724582e0be commit 98d43381f1c6249486df9b61c4cc40724582e0be Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Aug 23 15:30:56 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/205936cfa527c41dd5be9bfb268d92c5974ad607 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/33a46ffb3f0e7f54f55d269795b52d34b2f1be61 git cl: workaround against integer overflows of git config. (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= 640115 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2272673002 Cr-Commit-Position: refs/heads/master@{#413741} [modify] https://crrev.com/98d43381f1c6249486df9b61c4cc40724582e0be/infra/config/recipes.cfg
,
Aug 25 2016
I have verified and it works fine now.
,
Aug 25 2016
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by siva.gun...@samsung.com
, Aug 23 2016