Re: [PATCH 3/3] push: add option to push only submodules

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 19, 2016 at 10:25 AM, Brandon Williams <bmwill@xxxxxxxxxx> wrote:
> Teach push the --recurse-submodules=only option.  This enables push to
> recursively push all unpushed submodules while leaving the superproject
> unpushed.
>
> This is a desirable feature in a scenario where updates to the
> superproject are handled automatically by some other means, perhaps a

e.g. Gerrit. (No need to be shy about our code review tool)

>
> code review tool.  In this scenario a developer could make a change
> which spans multiple submodules and then push their commits for code
> review.  Upon completion of the code review, their commits can be
> accepted and applied to their respective submodules while the code
> review tool can then automatically update the superproject to the most
> recent SHA1 of each submodule.  This would eliminate the merge conflicts
> in the superproject that could occur if multiple people are contributing
> to the same submodule.

Code and tests look good to me, I think the commit message
is good enough, but let's hear Junio on this one.

Thanks,
Stefan



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]