Manuel Klimek via llvm-dev
2017-Nov-23 09:43 UTC
[llvm-dev] [cfe-dev] [Proposal] Automatically Cc: cfe-commits@ on Clang reviews
On Thu, Nov 23, 2017 at 12:58 AM Ben Hamilton <benhamilton at google.com> wrote:> Absolutely — I should have mentioned that we would keep the main rL > project and continue to use it. >Your original email said "Moving forward, only LLVM commits will be identified with the prefix rL (as in https://reviews.llvm.org/rL12345) — each project will get its own unique prefix, which might surprise some users or require changes to tools which bake in the rL prefix". That seems to not be an issue, as long as all commits for all projects are still reachable under the rL tag, right?> > On Wed, Nov 22, 2017, 09:23 Manuel Klimek <klimek at google.com> wrote: > >> +llvm-dev, so we get wider input :) >> >> Given how unfortunate reviews that are started without cc'ing the right >> list are (basically folks need to re-send the review from scratch), I >> support this idea. >> >> Ben, couldn't rL still be available for all projects? (and be the main >> project for LLVM). >> >> >> On Tue, Nov 21, 2017 at 5:18 PM Ben Hamilton via cfe-dev < >> cfe-dev at lists.llvm.org> wrote: >> >>> OK. I confirmed that Stephan's process to send out cross-repo diffs from >>> the monorepo is not affected by my proposal. >>> >>> tl;dr: https://reviews.llvm.org/D40312 (which was sent out on top of >>> D40179 to emulate what happens with a monorepo after my proposal lands). >>> >>> Steps I took (starting with >>> https://llvm.org/docs/GettingStarted.html#for-developers-to-work-with-a-git-monorepo >>> ): >>> >>> ==>>> % git clone https://github.com/llvm-project/llvm-project-20170507/ >>> llvm-project >>> % cd llvm-project >>> % cp llvm/.arcconfig . >>> % mkdir -p .git/info >>> % echo .arcconfig >> .git/info/exclude >>> # Manually apply D40179 into the clang-tools-extra directory >>> % arc export --revision D40179 --git | patch -d clang-tools-extra -p1 >>> patching file .arcconfig >>> % git diff >>> diff --git a/clang-tools-extra/.arcconfig b/clang-tools-extra/.arcconfig >>> index f846581763e..d4a00161bce 100644 >>> --- a/clang-tools-extra/.arcconfig >>> +++ b/clang-tools-extra/.arcconfig >>> @@ -1,4 +1,4 @@ >>> { >>> - "project_id" : "clang-tools-extra", >>> + "repository.callsign" : "CTE", >>> "conduit_uri" : "https://reviews.llvm.org/" >>> } >>> % git commit -a -m "Differential Revision: >>> https://reviews.llvm.org/D40179" >>> % echo "Test 1" >> llvm/README.txt >>> >>> % echo "Test 2" >> clang-tools-extra/README.txt >>> >>> % git commit -a -m "[Test] Monorepo test diff crossing Differential >>> repositories" >>> % arc diff --create --reviewers sberg HEAD^ >>> ==>>> >>> Ben >>> >>> On Tue, Nov 21, 2017 at 8:00 AM Stephan Bergmann <sbergman at redhat.com> >>> wrote: >>> >>>> On 11/21/2017 03:41 PM, Ben Hamilton wrote: >>>> > > My understanding was that, while there currently is no official >>>> > monorepo, it is at least possible and accepted to send single, >>>> > cross-repo patches for review. That would no longer be possible, >>>> right? >>>> > >>>> > I assume cross-repo patches would be constructed by hand, right? Or do >>>> > you mean using the unofficial monorepo from 2016 >>>> > <https://github.com/joker-eph/llvm-project>? >>>> >>>> I'm using <https://github.com/llvm-project/llvm-project-20170507> (as >>>> advertised at >>>> < >>>> https://llvm.org/docs/GettingStarted.html#for-developers-to-work-with-a-git-monorepo >>>> >), >>>> where such a "cross-repo diff" is just a matter of a plain 'git diff'. >>>> >>>> > An example of an existing cross-repo review in Differential would be >>>> > nice—do you happen to have one handy? >>>> >>>> I just happened to file <https://reviews.llvm.org/D40295> a moment ago, >>>> spanning clang and compiler-rt. >>>> >>>> > I think either way, we'd keep the ability to send cross-repo patches >>>> to >>>> > the top-level LLVM repo just the same way you could today. >>>> > >>>> > I'll confirm that if you can help me construct an example cross-repo >>>> > patch for review. >>>> >>>> Thanks, >>>> Stephan >>>> >>> _______________________________________________ >>> cfe-dev mailing list >>> cfe-dev at lists.llvm.org >>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev >>> >>-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20171123/754da9cc/attachment.html>
Ben Hamilton via llvm-dev
2017-Nov-26 18:19 UTC
[llvm-dev] [cfe-dev] [Proposal] Automatically Cc: cfe-commits@ on Clang reviews
Right. I think the commits will show up with multiple (two, I think) projects, which should be fine. On Thu, Nov 23, 2017, 02:43 Manuel Klimek <klimek at google.com> wrote:> On Thu, Nov 23, 2017 at 12:58 AM Ben Hamilton <benhamilton at google.com> > wrote: > >> Absolutely — I should have mentioned that we would keep the main rL >> project and continue to use it. >> > > Your original email said "Moving forward, only LLVM commits will be > identified with the prefix rL (as in https://reviews.llvm.org/rL12345) — > each project will get its own unique prefix, which might surprise some > users or require changes to tools which bake in the rL prefix". > > That seems to not be an issue, as long as all commits for all projects are > still reachable under the rL tag, right? > > >> >> On Wed, Nov 22, 2017, 09:23 Manuel Klimek <klimek at google.com> wrote: >> >>> +llvm-dev, so we get wider input :) >>> >>> Given how unfortunate reviews that are started without cc'ing the right >>> list are (basically folks need to re-send the review from scratch), I >>> support this idea. >>> >>> Ben, couldn't rL still be available for all projects? (and be the main >>> project for LLVM). >>> >>> >>> On Tue, Nov 21, 2017 at 5:18 PM Ben Hamilton via cfe-dev < >>> cfe-dev at lists.llvm.org> wrote: >>> >>>> OK. I confirmed that Stephan's process to send out cross-repo diffs >>>> from the monorepo is not affected by my proposal. >>>> >>>> tl;dr: https://reviews.llvm.org/D40312 (which was sent out on top of >>>> D40179 to emulate what happens with a monorepo after my proposal lands). >>>> >>>> Steps I took (starting with >>>> https://llvm.org/docs/GettingStarted.html#for-developers-to-work-with-a-git-monorepo >>>> ): >>>> >>>> ==>>>> % git clone https://github.com/llvm-project/llvm-project-20170507/ >>>> llvm-project >>>> % cd llvm-project >>>> % cp llvm/.arcconfig . >>>> % mkdir -p .git/info >>>> % echo .arcconfig >> .git/info/exclude >>>> # Manually apply D40179 into the clang-tools-extra directory >>>> % arc export --revision D40179 --git | patch -d clang-tools-extra -p1 >>>> patching file .arcconfig >>>> % git diff >>>> diff --git a/clang-tools-extra/.arcconfig b/clang-tools-extra/.arcconfig >>>> index f846581763e..d4a00161bce 100644 >>>> --- a/clang-tools-extra/.arcconfig >>>> +++ b/clang-tools-extra/.arcconfig >>>> @@ -1,4 +1,4 @@ >>>> { >>>> - "project_id" : "clang-tools-extra", >>>> + "repository.callsign" : "CTE", >>>> "conduit_uri" : "https://reviews.llvm.org/" >>>> } >>>> % git commit -a -m "Differential Revision: >>>> https://reviews.llvm.org/D40179" >>>> % echo "Test 1" >> llvm/README.txt >>>> >>>> % echo "Test 2" >> clang-tools-extra/README.txt >>>> >>>> % git commit -a -m "[Test] Monorepo test diff crossing Differential >>>> repositories" >>>> % arc diff --create --reviewers sberg HEAD^ >>>> ==>>>> >>>> Ben >>>> >>>> On Tue, Nov 21, 2017 at 8:00 AM Stephan Bergmann <sbergman at redhat.com> >>>> wrote: >>>> >>>>> On 11/21/2017 03:41 PM, Ben Hamilton wrote: >>>>> > > My understanding was that, while there currently is no official >>>>> > monorepo, it is at least possible and accepted to send single, >>>>> > cross-repo patches for review. That would no longer be possible, >>>>> right? >>>>> > >>>>> > I assume cross-repo patches would be constructed by hand, right? Or >>>>> do >>>>> > you mean using the unofficial monorepo from 2016 >>>>> > <https://github.com/joker-eph/llvm-project>? >>>>> >>>>> I'm using <https://github.com/llvm-project/llvm-project-20170507> (as >>>>> advertised at >>>>> < >>>>> https://llvm.org/docs/GettingStarted.html#for-developers-to-work-with-a-git-monorepo >>>>> >), >>>>> where such a "cross-repo diff" is just a matter of a plain 'git diff'. >>>>> >>>>> > An example of an existing cross-repo review in Differential would be >>>>> > nice—do you happen to have one handy? >>>>> >>>>> I just happened to file <https://reviews.llvm.org/D40295> a moment >>>>> ago, >>>>> spanning clang and compiler-rt. >>>>> >>>>> > I think either way, we'd keep the ability to send cross-repo patches >>>>> to >>>>> > the top-level LLVM repo just the same way you could today. >>>>> > >>>>> > I'll confirm that if you can help me construct an example cross-repo >>>>> > patch for review. >>>>> >>>>> Thanks, >>>>> Stephan >>>>> >>>> _______________________________________________ >>>> cfe-dev mailing list >>>> cfe-dev at lists.llvm.org >>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev >>>> >>>-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20171126/3dffc5cc/attachment.html>
Manuel Klimek via llvm-dev
2017-Nov-27 11:34 UTC
[llvm-dev] [cfe-dev] [Proposal] Automatically Cc: cfe-commits@ on Clang reviews
On Sun, Nov 26, 2017 at 7:19 PM Ben Hamilton <benhamilton at google.com> wrote:> Right. I think the commits will show up with multiple (two, I think) > projects, which should be fine. >Sounds good!> > On Thu, Nov 23, 2017, 02:43 Manuel Klimek <klimek at google.com> wrote: > >> On Thu, Nov 23, 2017 at 12:58 AM Ben Hamilton <benhamilton at google.com> >> wrote: >> >>> Absolutely — I should have mentioned that we would keep the main rL >>> project and continue to use it. >>> >> >> Your original email said "Moving forward, only LLVM commits will be >> identified with the prefix rL (as in https://reviews.llvm.org/rL12345) — >> each project will get its own unique prefix, which might surprise some >> users or require changes to tools which bake in the rL prefix". >> >> That seems to not be an issue, as long as all commits for all projects >> are still reachable under the rL tag, right? >> >> >>> >>> On Wed, Nov 22, 2017, 09:23 Manuel Klimek <klimek at google.com> wrote: >>> >>>> +llvm-dev, so we get wider input :) >>>> >>>> Given how unfortunate reviews that are started without cc'ing the right >>>> list are (basically folks need to re-send the review from scratch), I >>>> support this idea. >>>> >>>> Ben, couldn't rL still be available for all projects? (and be the main >>>> project for LLVM). >>>> >>>> >>>> On Tue, Nov 21, 2017 at 5:18 PM Ben Hamilton via cfe-dev < >>>> cfe-dev at lists.llvm.org> wrote: >>>> >>>>> OK. I confirmed that Stephan's process to send out cross-repo diffs >>>>> from the monorepo is not affected by my proposal. >>>>> >>>>> tl;dr: https://reviews.llvm.org/D40312 (which was sent out on top of >>>>> D40179 to emulate what happens with a monorepo after my proposal lands). >>>>> >>>>> Steps I took (starting with >>>>> https://llvm.org/docs/GettingStarted.html#for-developers-to-work-with-a-git-monorepo >>>>> ): >>>>> >>>>> ==>>>>> % git clone https://github.com/llvm-project/llvm-project-20170507/ >>>>> llvm-project >>>>> % cd llvm-project >>>>> % cp llvm/.arcconfig . >>>>> % mkdir -p .git/info >>>>> % echo .arcconfig >> .git/info/exclude >>>>> # Manually apply D40179 into the clang-tools-extra directory >>>>> % arc export --revision D40179 --git | patch -d clang-tools-extra -p1 >>>>> patching file .arcconfig >>>>> % git diff >>>>> diff --git a/clang-tools-extra/.arcconfig >>>>> b/clang-tools-extra/.arcconfig >>>>> index f846581763e..d4a00161bce 100644 >>>>> --- a/clang-tools-extra/.arcconfig >>>>> +++ b/clang-tools-extra/.arcconfig >>>>> @@ -1,4 +1,4 @@ >>>>> { >>>>> - "project_id" : "clang-tools-extra", >>>>> + "repository.callsign" : "CTE", >>>>> "conduit_uri" : "https://reviews.llvm.org/" >>>>> } >>>>> % git commit -a -m "Differential Revision: >>>>> https://reviews.llvm.org/D40179" >>>>> % echo "Test 1" >> llvm/README.txt >>>>> >>>>> % echo "Test 2" >> clang-tools-extra/README.txt >>>>> >>>>> % git commit -a -m "[Test] Monorepo test diff crossing Differential >>>>> repositories" >>>>> % arc diff --create --reviewers sberg HEAD^ >>>>> ==>>>>> >>>>> Ben >>>>> >>>>> On Tue, Nov 21, 2017 at 8:00 AM Stephan Bergmann <sbergman at redhat.com> >>>>> wrote: >>>>> >>>>>> On 11/21/2017 03:41 PM, Ben Hamilton wrote: >>>>>> > > My understanding was that, while there currently is no official >>>>>> > monorepo, it is at least possible and accepted to send single, >>>>>> > cross-repo patches for review. That would no longer be possible, >>>>>> right? >>>>>> > >>>>>> > I assume cross-repo patches would be constructed by hand, right? Or >>>>>> do >>>>>> > you mean using the unofficial monorepo from 2016 >>>>>> > <https://github.com/joker-eph/llvm-project>? >>>>>> >>>>>> I'm using <https://github.com/llvm-project/llvm-project-20170507> (as >>>>>> advertised at >>>>>> < >>>>>> https://llvm.org/docs/GettingStarted.html#for-developers-to-work-with-a-git-monorepo >>>>>> >), >>>>>> where such a "cross-repo diff" is just a matter of a plain 'git diff'. >>>>>> >>>>>> > An example of an existing cross-repo review in Differential would be >>>>>> > nice—do you happen to have one handy? >>>>>> >>>>>> I just happened to file <https://reviews.llvm.org/D40295> a moment >>>>>> ago, >>>>>> spanning clang and compiler-rt. >>>>>> >>>>>> > I think either way, we'd keep the ability to send cross-repo >>>>>> patches to >>>>>> > the top-level LLVM repo just the same way you could today. >>>>>> > >>>>>> > I'll confirm that if you can help me construct an example cross-repo >>>>>> > patch for review. >>>>>> >>>>>> Thanks, >>>>>> Stephan >>>>>> >>>>> _______________________________________________ >>>>> cfe-dev mailing list >>>>> cfe-dev at lists.llvm.org >>>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev >>>>> >>>>-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20171127/352b1986/attachment.html>