Tomasz Śniatowski
823adab6bd
gn_helper: add missing sys import
...
A recent change added sys usage without the corresponding import. Add it.
See https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6217699
Bug: 393209409
Change-Id: Ib30dc8938ec6e3b56ab35a7b5e2b2071cb9e84ce
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6300492
Auto-Submit: Tomasz Śniatowski <tomasz.sniatowski@xperi.com>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
Reviewed-by: Philipp Wollermann <philwo@google.com>
Reviewed-by: Fumitoshi Ukai <ukai@google.com>
3 days ago
Fumitoshi Ukai
c88ad7af1a
gn_helper: handle absolute path import
...
Bug: 393209409
Change-Id: Ic26b4502ab1744c7af7c0b06de2e4d1b9f2d3980
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6217699
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Philipp Wollermann <philwo@google.com>
Auto-Submit: Fumitoshi Ukai <ukai@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 weeks ago
Fumitoshi Ukai
fd5161870a
gn_helper: find correct directory for //
...
gn finds // by searching .gn in parent directories.
To support out dir is not 2 directories from root (i.e. out/Default),
search .gn in parent directory for '//' location.
tools/licenses/licenses.py runs 'gn gen' in out/Default/$tmp.
for such case, '//' should be '../../..', or fail to import '//path'
Change-Id: I6e5ccbe93cb5e51704f31d4eb558c03560286865
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6051636
Reviewed-by: Philipp Wollermann <philwo@google.com>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
3 months ago
Fumitoshi Ukai
d7d8221581
gn_helper: ignore spaces on gn lines
...
Change-Id: I2f676411a5282c938fc1d095acbbde0bbe6efde3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5668281
Commit-Queue: Junji Watanabe <jwata@google.com>
Auto-Submit: Fumitoshi Ukai <ukai@google.com>
Reviewed-by: Junji Watanabe <jwata@google.com>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
8 months ago
Fumitoshi Ukai
356ef0324e
ninja: error if trying to build for use_remoteexec=true
...
Change-Id: Ia32dd3cba1d1874401c6614f792f212b2cfa60a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5660200
Commit-Queue: Joanna Wang <jojwang@chromium.org>
Reviewed-by: Junji Watanabe <jwata@google.com>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Auto-Submit: Fumitoshi Ukai <ukai@google.com>
Reviewed-by: Joanna Wang <jojwang@chromium.org>
8 months ago