OXT-1073 : mirror upstream layer repos#240
Conversation
Finds the submodules of the master branch of openxt.git and adds them to the local git mirror alongside the OpenXT repos. Signed-off-by: Christopher Clark <christopher.clark6@baesystems.com>
Signed-off-by: Christopher Clark <christopher.clark6@baesystems.com>
|
Your code is incredibly hard to read, and would make build setups more complicated, less flexible and prone to errors (like if a submodule gets added later, or if the branch you want to build doesn't use the same submodules). If we really want this (I don't), then git submodules is probably not the right tool for us, and we should replace it entirely. If your end-goal is to reduce build time, there are many other areas that could be improved. TL;DR: that's a -1 for me. |
|
Well, I'm saddened to hear that these two commits, each with fewer than twenty lines of commented shell code, with clear variable names, exceed your complexity tolerance threshold for the project build system. I made these changes by tuning my local build to reduce the time to get bitbake active and submitted the PR since it should be valuable to anyone else using the project's recommended build system.
Though they are challenging to read, regular expressions have been a standard UNIX tool syntax for a long time. Their use for this task, parsing and amending the necessary config file, is reasonable.
Either of those cases work just fine. A comment in the patch: |
Improve the prior two commits by removing use of "sed", add further intermediate variables with explanatory names and use git to access its configuration files. Signed-off-by: Christopher Clark <christopher.clark6@baesystems.com>
OpenEmbedded core takes a long time to clone from source, so this speeds up the build.
Use copies of upstream layers in the local git mirror, if available.
Make them available by default when setting up the git mirror.
Signed-off-by: Christopher Clark christopher.clark6@baesystems.com