When merge-commits are uploaded to Gerrit, the merged commits have
usually either already been reviewed elsewhere (in another branch), or
may contain automatically-generated commits that would probably not pass
the pre-upload checks (e.g. lacking a Bug: tag). This change makes it so
that the merge commits are not checked by the hooks, but the
merge-commit itself is.
Bug: 31926893
Test: repo upload
Change-Id: Ic25127fdbaea651cd4bb14e250c26ffe9aa064ad
This allows individual projects to enable clang-format pre-upload hook
without worrying about the paths of the various tools.
Bug: 26800693
Test: Added a PREUPLOAD.cfg in external/minijail, got a warning.
Change-Id: Ia9ee738ad3305934e8e6ca50261f96ca3ab30852
Remove the nesting of build/{blueprint,kati,soong}, which should allow
us to use submodules.
Bug: 28001743
Test: Compare build-aosp_arm.ninja before/after
Change-Id: I5ff919cdb0423ebd11d9033fadab8aedf1a8b81d
I've fixed ltrace, but I've also determined that it now builds out of
the box anyway. Plus no-one complained for the whole year that the makefile
was disabled.
Bug: http://b/19715557
Change-Id: I8b8ef7faaf5f2c4328f7eea056cb0b1e9414758e
This doesn't turn on anything (yet), just includes some guidelines for
people who are thinking about changing/enabling things.
Bug: 29606287
Test: `repo upload` still works
Change-Id: I397f53aed417fe9bfd1a00dac2d0d5b380f66e10
These repos are part of Brillo in the Brillo manifest and shouldn't be
included in the main Android manifest. These were added here:
b19edffa2d
Change-Id: I0b67a97c7c7c21a48cb4f4b6f6690f6a417b03e0
Using the --groups option with repo init will restrict which repos are
downloaded to only the specified ones. Unfortunately, the <repohooks>
logic will always fire regardless of the selected group (and cannot be
restricted), which means you'll get an error when trying to upload like:
ERROR: Couldn't find repo hook: ".../tools/repohooks/pre-upload.py"
Add the repohooks repo to every major active group so that it gets
synced/checked out too. Hopefully this should cover most teams.
Bug: 28295071
Test: `repo init -g adt-infra && repo sync` pulled down the repohooks repo
Change-Id: If0295d7bbdd2840625fa79a76a5ecf11792a601a
By default, no hooks are enabled, so this shouldn't impact people.
Bug: 28295071
Change-Id: Ia74f03ea89f7b39faa61f26d7846f5e3cbb3f128
Test: `repo upload` in a repo didn't blow up
This reverts commit 12da222884.
After more discussion, we're going to take a
different approach to handling the repo hooks
(splitting into two repositories so the hooks
can be easily modified without triggering more
spam from repo), so reverting for now.
Bug: 28295071
Change-Id: I948ac019f0a95101f04a9ef4d4070079dea4b4ad
Test: None
By default, no hooks are enabled, so this shouldn't impact people.
Bug: 28295071
Test: `repo upload` in a repo didn't blow up
Change-Id: I46a8476f7eaef80e68d9e7b3cc46741267150b67
It is in the same groups as external/icu as this is intended to
be used by external/icu.
Bug: 30244565
Change-Id: I5d35e43181db39e14be489a20f88710e3a9badcd
This branch of the project has not been touched for 2 years and I
am not aware of anyone using it. All LLDB development is now going
on in the lldb-master-dev branch.
Change-Id: Ibb2dc1c940f52ac14faec57e283272a002183537
A number of dependencies that got removed in r.android.com/129602
got accidentally readded in r.android.com/174547 in Android M merge
to AOSP. This CL removes these unused dependencies again.
Change-Id: I6639add51a7270b81a61743dc00d39b3169978ee
This is being added to replace external/gtest and external/gmock, but
I'm not removing those in this commit to try to minimize transient
build breakages.
libconstrainedcrypto, formerly know as libmincrypt, used to be part of
system/core. Moving the code to external/ has a couple benefits:
* The external/ location more accurately reflects the Google-internal
heritage of the code.
* We're planning for removing dependencies on this code for most
consumers in favor of switching to BoringSSL's libcrypto, so it
doesn't make sense to keep the code in system/core.
* Some rare uses of this code will probably remain (e.g. constrained
environments such as bootloaders, TEE, etc.), having the code in a
separate repo makes it simpler to track for projects outside the
main Android tree.
BUG: 27293141
Change-Id: Iedcf78d83390536ebb9b945eeebaf1715f2d4a8e
libgabi++ from abi/cpp was only useful with stlport and libstdc++,
neither of which is used in the platform any more. The NDK has its own
copy of libgabi++, so remove the platform version.
Change-Id: I3c97a5e49c8b0e9d1269567e0c16da8b5df604aa
We keep some common device-specific files in the Linux prebuilts as
well, so it can't be Linux-specific (for those that prune their repo
trees on Darwin).
Change-Id: Ifd16509e9de79f6265e944ad409ec8ddf2dc8419
Reviewed-on: https://googleplex-android-review.git.corp.google.com/863382
Reviewed-by: Dan Albert <danalbert@google.com>
Presubmit-Verified: TreeHugger Robot <android-build-prod@system.gserviceaccount.com>
Reviewed-by: Bill Yi <byi@google.com>
(cherry-picked from commit 91d5393e9b)
Luis Hector Chavez
Dan Willemsen
Thomas Joseph Avila
Elliott Hughes
Mike Frysinger
Alex Deymo
Dan Albert
Christopher Wiley
Bill Yi
Mike Frysinger
David James
Stephen Hines
Paul Duffin
Alex Klyubin
Dmitry Shmidt
Pavel Labath
Aurimas Liutikas
Xin Li
Nick Kralevich
Mattias Nissler
Igor Murashkin
Colin Cross
Bill Rassieur
Nathan Harold
Josh Gao
Sebastien Hertz
destradaa