[PATCH] D29050: [zorg] Add AOSP builder

[Pengxuan Zheng via Phabricator via llvm-commits]

pzheng added a comment.

In https://reviews.llvm.org/D29050#655879, @grosser wrote:

> Thanks for explaining how to setup the buildslave. Could you add the information (the link what to install, the need for java, the information which AOSP repo to checkout, ...) as comment to the AOSPBuilder?


Sure, I will add these information.

> Also, if the AOSP checkout is just a one-time thing, this could still be part of the builder, as long as we make sure we do not clean it in each build, but just update it.

Sorry for the confusion. Let me try to clarify this a bit more. Roughly speaking, there are two steps to checkout AOSP.

1. repo init

This step should only be done once (manually), and therefore should not be included in any subsequent builds. This is part of the one-time setup needed for the build slave. Thus, this step is ommited from the AOSP build factory.

2. repo sync

This step does the actual checkout and is already included in the AOSP build factory. The first time you run "repo sync" after "repo init", it takes hours to finish. This is also part of the one-time setup I mentioned. Fortunately, subsequent "repo sync"s only take a few minutes since it only fetches the latest changes and updates the repo.

> Best,
> Tobias



In https://reviews.llvm.org/D29050#655879, @grosser wrote:

> Thanks for explaining how to setup the buildslave. Could you add the information (the link what to install, the need for java, the information which AOSP repo to checkout, ...) as comment to the AOSPBuilder? Also, if the AOSP checkout is just a one-time thing, this could still be part of the builder, as long as we make sure we do not clean it in each build, but just update it.
>
> Best,
> Tobias





https://reviews.llvm.org/D29050