December 19, 2015 
< | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9 | 10 | 11 | 12 | 13 | 14 | 15 | 16 | 17 | 18 | 19 | 20 | 21 | 22 | 23 | 24 | 25 | 26 | 27 | 28 | 29 | 30 | 31 | >
December 19, 2015 [00:21:03] *** qchris has quit IRC[00:42:13] *** sbeller has quit IRC[01:11:24] *** Puhe has joined #gerrit[01:28:57] *** Puhe_ has joined #gerrit[01:32:45] *** Puhe has quit IRC[02:03:28] *** Puhe__ has joined #gerrit[02:06:45] *** Puhe_ has quit IRC[02:08:30] *** Puhe_ has joined #gerrit[02:11:50] *** Puhe__ has quit IRC[03:22:11] *** Vampire0 has joined #gerrit[03:25:29] *** Vampire0_ has quit IRC[04:58:05] *** Puhe_ has quit IRC[04:59:54] *** Naresh has joined #gerrit[06:31:12] *** Haza` is now known as Haza`Aw[07:07:34] *** gtristan has joined #gerrit[07:08:36] <gtristan> Hi... I am relatively new to using gerrit, and I'm trying to improve my workflow with gerrit which, in some cases works well, and others not so much[07:09:34] <gtristan> When for example, I am working on a branch, and I have a patchset to upload: Gerrit works well and appropriately makes all uploaded patches depend on eachother in the correct order[07:09:56] <gtristan> I was paranoid at first they were not being applied in the right order, but it seems gerrit understands this well[07:10:25] <gtristan> The other scenario though, is: Say I am working on master, and I am pushing out a *lot* of small patches[07:10:51] <gtristan> I want to have 10 or 15 patches applied to my local master branch, so I am building with everything applied[07:11:04] <gtristan> but these 10 or 15 patches are completely unrelated to eachother[07:11:41] <gtristan> For this workflow to work well, I would need a sort of wrapper like git review to upload ONE PATCH for review[07:11:50] <gtristan> like: git review <commit-id>[07:12:32] <gtristan> Upload "that commit" against upstream master, and please do not infer that it depends on other commits I have in my local tree[07:12:41] <gtristan> Does that exist ?[07:14:13] <gtristan> I think this SO post sort of describes it: http://stackoverflow.com/questions/19768959/how-to-review-a-specific-commit-on-git[07:15:46] <gtristan> The proposed solution is obnoxious though, create 15 branches to review them ? Couldnt we have some convenience wrapper to do this for us ?[07:44:26] *** uebera|| has quit IRC[07:48:27] *** uebera|| has joined #gerrit[07:48:27] *** uebera|| has quit IRC[07:48:28] *** uebera|| has joined #gerrit[08:00:06] *** uebera|| has quit IRC[08:04:17] *** uebera|| has joined #gerrit[08:04:18] *** uebera|| has quit IRC[08:04:18] *** uebera|| has joined #gerrit[08:33:33] *** lqi has joined #gerrit[09:20:24] *** Haza`Aw is now known as Haza`[09:28:58] *** lqi has quit IRC[09:42:32] *** lqi has joined #gerrit[09:45:52] *** gtristan has quit IRC[10:32:57] *** Naresh has quit IRC[10:50:27] *** gtristan has joined #gerrit[11:53:58] <gtristan> FWIW: This wrapper of git-review makes the story much more bearable: http://fpaste.org/303117/52231614/[11:55:08] <gtristan> unfortunately it's not very elegant, as it (temporarily) changes the active branch state[11:55:40] <gtristan> would be much nicer to just say "take this commit ID, stage it against that branch, and send it through git review, but NEVER change my current branch state"[12:16:37] *** [HeOS] has joined #gerrit[12:21:56] *** [HeOS] has quit IRC[12:35:27] *** lqi has quit IRC[13:02:35] <pedroalvarez> gtristan: the idea of that script is interesting though[13:03:29] <gtristan> pedroalvarez, I used it to update the lorry patches[13:03:50] <gtristan> they all have origin/master as the parent, now[13:04:08] <pedroalvarez> That's great[13:04:54] <pedroalvarez> So, an script like this but that doesn't compromise your current branch would work for you?[13:05:15] <gtristan> pedroalvarez, I do "get it" now, to be honest yesterday looking at that "wocky" patch status, I was under the impression that the lorry patches were blocking on the definitions patches[13:05:38] <gtristan> but it (was) only the webkit lorry patch (iiuc) which was blocking them, correct ?[13:05:48] <pedroalvarez> Yes[13:05:59] <gtristan> honestly, its a matter of workflow[13:06:24] <pedroalvarez> It may be possible to create dependencies on other projects too, but never explored it[13:06:39] <gtristan> it's a huge overhead to be manually checking out branches one by one, especially when you want to create a lot of small patches in a row[13:06:50] <pedroalvarez> I agree[13:08:15] <pedroalvarez> Btw.. Gerrit is now in 2.11, the topic of this channel suggests me that is not being used... :([13:08:30] <gtristan> honestly the current script "worksforme" even though it does a checkout under the hood[13:08:41] <gtristan> it's just IMO unclean that way[13:09:52] <gtristan> It would be nice to propose that as a feature for https://github.com/openstack-infra/git-review, but I'm uncomfortable proposing a patch which touches the local branch state[13:10:44] <pedroalvarez> Let's continue in #baserock[13:10:49] <gtristan> pedroalvarez, the topic was set in 2013 :)[13:10:55] <gtristan> relating to this channel :)[13:10:56] <gtristan> hehe[13:11:15] <gtristan> just one of those, nobody cared to update the topic[13:18:21] *** [HeOS] has joined #gerrit[13:22:26] *** [HeOS] has quit IRC[14:18:58] *** [HeOS] has joined #gerrit[14:23:42] *** [HeOS] has quit IRC[14:34:33] *** [HeOS] has joined #gerrit[14:39:15] *** [HeOS] has quit IRC[14:48:01] *** [HeOS] has joined #gerrit[14:50:30] *** gtristan has quit IRC[14:53:29] *** [HeOS] has quit IRC[15:09:18] *** gtristan has joined #gerrit[18:24:24] *** Tica2 has joined #gerrit[18:32:17] *** pzd has joined #gerrit[18:33:30] *** pzd_ has quit IRC[18:34:13] *** Tica2 has quit IRC[18:58:51] *** marlinc has quit IRC[19:00:47] *** marlinc has joined #gerrit[19:18:59] *** lqi has joined #gerrit[20:45:29] *** Puhe_ has joined #gerrit[22:02:18] *** lqi has quit IRC[22:19:26] *** lqi has joined #gerrit[22:40:40] *** gigetoo has quit IRC[22:41:26] *** gigetoo has joined #gerrit[23:12:25] *** kad has quit IRC[23:12:54] *** kad has joined #gerrit[23:52:07] *** pamaury has joined #gerrit[23:52:43] *** ^elyse^ has joined #gerrit[23:52:51] *** ^elyse^ has left #gerrit