| *** zoli__ has quit IRC | 00:01 | |
| *** zoli__ has joined #baserock | 04:30 | |
| *** zoli__ has quit IRC | 04:43 | |
| *** zoli__ has joined #baserock | 05:06 | |
| * straycat clicks a change | 07:23 | |
| straycat | oh, no, can't see that, my session's expired | 07:23 |
|---|---|---|
| * straycat goes back to the list of changes | 07:23 | |
| * straycat clicks a change | 07:23 | |
| straycat | can't actually see the diff, just a load irrelevant information about the change | 07:24 |
| * straycat clicks the file | 07:24 | |
| * straycat clicks back | 07:24 | |
| * straycat clicks the other file | 07:24 | |
| * straycat clicks back | 07:24 | |
| * straycat looks for the change this change depends on | 07:24 | |
| straycat | oh no my session expired again | 07:25 |
| straycat | Okay screw this, I'm just going to fetch refs and read the commit log from now on, and if we want inline comments 2.11 will let us write a client for that, or maybe once we get https gertty will turn out to be okay | 07:30 |
| * straycat uses gerrymander to see if anyone's commented on any of his changes | 07:32 | |
| paulsherwood | ouch. straycat - maybe stick with the ML, until others convince you that the rough edges have been ironed out? | 07:40 |
| * straycat nods | 07:43 | |
| * paulsherwood can see how fresh-faced young sofware engineers happy to try anything turn into bearded ones who only use commandline tools and bark a lot :) | 07:45 | |
| straycat | haha | 07:46 |
| Kinnison | paulsherwood: yep | 07:49 |
| Kinnison | paulsherwood: Did you get anywhere with that suggestion from me last night? | 07:50 |
| *** mike has joined #baserock | 07:54 | |
| *** mike is now known as Guest70582 | 07:55 | |
| *** a1exhughe5 has joined #baserock | 08:06 | |
| *** gfinney has joined #baserock | 08:15 | |
| *** gfinney has joined #baserock | 08:15 | |
| *** mdizzle has joined #baserock | 08:46 | |
| *** bashrc_ has joined #baserock | 09:02 | |
| *** gary_perkins has joined #baserock | 09:04 | |
| *** ssam2 has joined #baserock | 09:17 | |
| *** ChanServ sets mode: +v ssam2 | 09:17 | |
| jonathanmaw | SotK: I'm trying to get GENIVI Demo Platform working on a Jetson, and note that you've got some definitions for the GENIVI Demo Platform. How complete are those definitions? | 09:24 |
| *** rdale has joined #baserock | 09:27 | |
| ssam2 | weird error from system-version-manager: http://paste.baserock.org/egesadicih | 09:27 |
| *** tiagogomes_ has joined #baserock | 09:29 | |
| ssam2 | this must be due to a change of behaviour in the 'mount' command | 09:30 |
| ssam2 | we upgraded to util-linux 2.25.2 on 19th Feb | 09:31 |
| richard_maw | that's not necessarily a `mount` failure, we saw something similar with a btrfs bug | 09:34 |
| ssam2 | right... this is indeed btrfs | 09:36 |
| ssam2 | seems there were quite a lot of changes in libmount, but nothing that says 'stop people mounting stuff twice' | 09:36 |
| ssam2 | wow, actually there are loads of errors in 'dmesg' | 09:37 |
| SotK | jonathanmaw: I don't remember entirely... I think I'd got to the point where I could deploy a system with Weston/Wayland + Qt + the GENIVI browser POC and was trying to get fuel-stop-advisor to build nicely | 09:46 |
| *** edcragg has joined #baserock | 09:48 | |
| ssam2 | seems that mounting the filesystem on a different machine and/or running btrfsck has fixed whatever problem I was having | 09:48 |
| *** Krin has joined #baserock | 10:10 | |
| *** mauricemoss_ has quit IRC | 10:31 | |
| *** simonh_ has joined #baserock | 10:31 | |
| *** wschaller has joined #baserock | 10:33 | |
| *** locallycompact has quit IRC | 10:40 | |
| jjardon | Hi! Im trying to add some comment to a morphology but doesnt seems to work: http://paste.baserock.org/jusojucave this works fine though http://paste.baserock.org/ivotapazow . Any idea what is the prefered way to do this? | 10:49 |
| paulsherwood | Kinnison: i got into a tangle | 10:51 |
| Kinnison | jjardon: You're missing a backtick | 10:51 |
| Kinnison | paulsherwood: oops :) | 10:51 |
| paulsherwood | Kinnison: you did manage to completely shock me | 10:52 |
| Kinnison | paulsherwood: Oh? | 10:52 |
| * Kinnison hides the taser behind his back | 10:52 | |
| paulsherwood | Kinnison: i started by doing http://git.baserock.org/cgi-bin/cgit.cgi/baserock/baserock/definitions.git/commit/?h=baserock/ps/separate-bootstrap-stratum&id=ae60ae2d64dba72c35377a7f8a22d02894804553 | 10:53 |
| Kinnison | yeah that won't work at all | 10:53 |
| paulsherwood | it seems to me if stage1 and stage2 are only build depends, it *should* work? | 10:53 |
| Kinnison | Nope | 10:53 |
| Kinnison | because build-essential-bootstrap contains no non-bootstrap chunks | 10:53 |
| paulsherwood | interesting 'logic' here :-) | 10:54 |
| Kinnison | As I said last night, bootstrap chunks exist *ONLY* as explicit build-dependencies of non-bootstrap chunks in the same stratum | 10:54 |
| * Kinnison perhaps didn't say that clearly enough | 10:54 | |
| paulsherwood | i hoped that ybd would be happier, but it just exposed other problems in the recursion | 10:54 |
| * Kinnison has prepred a fresh baserock chroot and acquired ybd from your master | 10:55 | |
| * Kinnison may have a tweak | 10:55 | |
| paulsherwood | Kinnison: i think you said it clearly enough... but it's still not clear enough to me. sorry, i can't think about this now. | 10:55 |
| paulsherwood | as i said, this little exercise did also blow a hole in the ybd recursion implementation, so it's worse than i thought | 10:55 |
| Kinnison | Hmm | 10:56 |
| Kinnison | OOI have your fixes to build-essential's b-deps got into maste ryet? | 10:56 |
| paulsherwood | no, i don't believe so | 10:56 |
| * Kinnison shall have to operate from your branch then | 10:57 | |
| * paulsherwood doesn't quite know why... it had 2 +1s :-) | 10:57 | |
| Kinnison | It has two +1s but Sam's comment indicates he'd like a fix to the comments in the morphology | 10:57 |
| paulsherwood | ah, missed that | 10:57 |
| jjardon | Kinnison: sorry, I copy incorrectly; still doesnt work with the missing backtick addded: http://paste.baserock.org/yibusuxiki | 10:58 |
| Kinnison | jjardon: aah | 10:58 |
| Kinnison | jjardon: that's because the way shell allows environment variables to be set requires that they be the first token | 10:58 |
| Kinnison | jjardon: You probably need to shove an 'env' in there | 10:58 |
| Kinnison | otherwise the ``d comment will be the first shell token | 10:59 |
| Kinnison | hence your error | 10:59 |
| * SotK wonders how sending a V2 patch to Gerrit works | 11:08 | |
| jjardon | Kinnison: thanks, that worked | 11:09 |
| rdale | SotK: i got the impression that if you push it with the same change id as v1, it just works | 11:09 |
| jjardon | SotK: "force" push it to the same ref/to/master/<topic> | 11:09 |
| jjardon | SotK: I mean, simply push to that special ref, you dont have to use -f | 11:10 |
| SotK | jjardon: does it matter if the change-ids are different? | 11:11 |
| Kinnison | I think the changeids have t omatch for it to line up the patches | 11:13 |
| ssam2 | yes | 11:13 |
| * SotK wonders how his change-ids ended up changing in the first place, and fixes it | 11:13 | |
| jjardon | SotK: no sure, until know I always rework the patches I already sent, so they have same change-id. Try it, you can alwyas discard the old patches if it doesnt work | 11:13 |
| Kinnison | ssam2: You +1d Paul's change for build-essential build-deps -- any hope that since it has two +1s it might get merged? | 11:17 |
| jjardon | Kinnison: you gave me a +1 for the second commit of the series "no_name" ; that change is a consecuence of the first commit, but you didnt review it, maybe you want to take a look? | 11:20 |
| Kinnison | Can you give me a change URI? | 11:20 |
| * Kinnison finds navigating the list of changes hard | 11:20 | |
| jjardon | sure: http://gerrit.baserock.org/#/c/43/ | 11:21 |
| jjardon | (seems the idea is not very popular already :)) | 11:21 |
| Kinnison | You appear to have two -1s already | 11:22 |
| * Kinnison reads | 11:22 | |
| * Kinnison agrees with the -1s | 11:22 | |
| Kinnison | Do you want me to review it formally? | 11:22 |
| Kinnison | the 'name:' field is meant to be a doublecheck that people haven't done something stupid :) | 11:22 |
| jjardon | Kinnison: if you can, and change your review of the change in the README to be consistent | 11:23 |
| Kinnison | okay | 11:24 |
| ssam2 | Kinnison: I think that it needs a comment explaining why build-essential treats the build-depends field differently to all the other strata, so I don't want it merged as-is | 11:24 |
| Kinnison | ssam2: Aah, I saw it as a precursor to fixing it so build-depends always behaves like paul's change :) | 11:24 |
| ssam2 | it could be... I'm not 100% sure on that now | 11:24 |
| ssam2 | it certainly won't happen overnight anyway, so we need to document why things are inconsistent in the mean time if we merge this patch | 11:25 |
| Kinnison | :) | 11:25 |
| Kinnison | Understood | 11:25 |
| ssam2 | i think the next step towards making build-depends always behave like that would be to write a migration script | 11:26 |
| jjardon | Kinnison: for me is a bit annoying because when I create a bla.morph I have to put a field in the morphology like name: blah (and it has to be exactly the same) | 11:26 |
| ssam2 | then we can see the actual implementation | 11:26 |
| jjardon | but im fine with the -1 | 11:26 |
| Kinnison | :) | 11:26 |
| jjardon | maybe a better way to solve this is to allow morphologies to be inline in the stratum | 11:27 |
| Kinnison | I'd like that, but it's an even bigger change | 11:28 |
| * Kinnison wants to see some smaller cleaner version updates happening first | 11:28 | |
| Kinnison | incl. migration scripts | 11:28 |
| jjardon | so we can fix the problem of some morph files not being moved when you move chunks around | 11:28 |
| Kinnison | However I'm still way behind on things (ca. 1100 mails still left to deal with in baserock-dev :) | 11:29 |
| * straycat would hope for more discussion of the branch morph rfc before any further version updates occur | 11:30 | |
| jjardon | seems people is out of ideas about what "smaller" updates should be done next | 11:30 |
| straycat | esp since they'll inevitably conflict with the series and force me to rework it | 11:30 |
| richard_maw | straycat: I'll take a look after I get back from lunch | 11:30 |
| straycat | richard_maw, awesome thanks :) | 11:31 |
| jjardon | straycat: I think you are mixing 2 things here: I think you want to discuss how we want to act when we break compatibility. That doesnt mean we can not keep doing (not breaking) changes. | 11:32 |
| bashrc_ | looks like there are strange entries in self.pluginmgr.location | 11:33 |
| bashrc_ | '/usr/lib/python2.7/site-packages/morphlib/plugins', '/', 'u', 's', 'r', '/', 'l', 'i', 'b', '/', 'p', 'y', 't', 'h', 'o', 'n', '2', '.', '7', '/', 's', 'i', 't', 'e', '-', 'p', 'a', 'c', 'k', 'a', 'g', 'e', 's', '/', 'm', 'o', 'r', 'p', 'h', 'l', 'i', 'b', '/', 'p', 'l', 'u', 'g', 'i', 'n', 's' | 11:33 |
| Kinnison | looks like someone extend()ed instead of append()ing | 11:34 |
| straycat | jjardon, I don't really follow, the discussion is really about whether we should try and get ourselves away from the problem of unversioned definitions by branching morph, such that one branch supports unversioned/v0 and v1 and the other branch supports v1,v2,... | 11:37 |
| SotK | Kinnison: I sent a v2 (and a v3, I messed up a rebase) | 11:39 |
| Kinnison | I will try and get to that today | 11:39 |
| jjardon | straycat: whats the problem with "unversioned definitions = version 0" ? My point is that when you want to make a change in morph and the result will not support version 0, then we should think about removing support for version 0, not before. It would be weird that we encourage people to use latest morph but they cannot build its no-so-old definitions | 11:43 |
| straycat | It depends on whether we have users of unversioned/version 0 who need to be supported, if we do then branching seems better, so that we can continue to maintain version0/unversioned whilst still being able to reduce code complexity (breaking compatibility) in master | 12:00 |
| *** wschaller_ has joined #baserock | 12:07 | |
| *** wschaller has quit IRC | 12:10 | |
| * bashrc_ spotted a bug | 12:11 | |
| pedroalvarez | that one must be the last thing that needs fixing to finish Morph :) | 12:19 |
| Zara | omg | 12:21 |
| pedroalvarez | tiagogomes_: I don't really know why, but your pysendfile can't be merged :/ | 12:48 |
| tiagogomes_ | pedroalvarez, does it show any error information | 12:49 |
| pedroalvarez | looks like someone added another lorry after pymox, so it's a merge conflict | 12:50 |
| tiagogomes_ | again ¬_¬ | 12:50 |
| pedroalvarez | hah | 12:50 |
| * tiagogomes_ needs to be faster | 12:50 | |
| pedroalvarez | at least franred spotted a Json error :) | 12:50 |
| franred | :) | 12:52 |
| tiagogomes_ | Ok, I sent another patch | 12:53 |
| pedroalvarez | franred: can you change your -2? | 12:54 |
| franred | pedroalvarez, tiagogomes_, merged | 12:55 |
| tiagogomes_ | \o. | 12:55 |
| pedroalvarez | I think we shouldnt use -2's when the patch can be merged with fixes | 12:55 |
| pedroalvarez | otherwise we need to wait for whoever has -2'ed it to merge the patch | 12:55 |
| franred | pedroalvarez, I didn't know that the -2 stay after sending a reworked version I though it disappear as the -1s | 12:56 |
| pedroalvarez | edcragg: I've reviewed your armv8 patches, they need some changes | 13:12 |
| *** pdar has quit IRC | 13:17 | |
| *** pdar has joined #baserock | 13:26 | |
| jonathanmaw | hrm, gstreamer-0.10 is failing to build for me, because the version of the gstreamer-common submodule it uses has some kind of parsing error with bison's verson. | 13:38 |
| jonathanmaw | the correct solution is to update the 0.10 version of the gstreamer-common code to include this patch. | 13:39 |
| jonathanmaw | how do I edit a submodule's code so that it gets picked up by baserock. I gather my gstreamer-0.10 that I've checked out with `morph edit` will be used if available | 13:40 |
| jonathanmaw | and that includes a ref to gstreamer-common | 13:40 |
| jonathanmaw | if I add a new ref to gstreamer-common, can morph find it? | 13:41 |
| ssam2 | submodules are a nightmare | 13:42 |
| ssam2 | you need a new commit of the gstreamer-0.10 repo that updates the submodule to the commit that you need | 13:43 |
| ssam2 | morph effectively does 'git submodule init; git submodule update' to fetch the submodules, i think | 13:43 |
| pedroalvarez | this commit fixed the parsing error in gstreamer-common: http://git.baserock.org/cgi-bin/cgit.cgi/delta/gstreamer-common.git/commit/?id=dae0ad20f831b32d9fc68caa8f26b359276c3c5f | 13:44 |
| jonathanmaw | so it'll be fetched from my local or remote artifact cache | 13:44 |
| jonathanmaw | so I'll need a branch pushed to git.baserock.org for gstreamer-common | 13:44 |
| ssam2 | no, you'll need a branch pushed for gstreamer-0.10, if I understand correctly | 13:45 |
| ssam2 | the commit is already present in gstreamer-common.git | 13:45 |
| ssam2 | but you need a commit in the parent repo that references the submodule commit | 13:45 |
| jonathanmaw | ssam2: the commit in gstreamer-common is for later versions of gstreamer, not the gstreamer-0.10 branch. I tried changing gstreamer-0.10 to use that ref and it didn't go well. | 13:46 |
| ssam2 | ah, right. | 13:51 |
| ssam2 | is there any way to avoid using gstreamer 0.10? it's totally unmaintained | 13:51 |
| ssam2 | I think | 13:51 |
| jonathanmaw | I'm not sure. | 13:52 |
| jonathanmaw | it looks like our qt strata use gstreamer-0.10 | 13:53 |
| SotK | yeah | 13:53 |
| radiofree | jonathanmaw: just don't add qt5-tools-qtmultimedia | 13:53 |
| radiofree | but if anyone wants to use qt5-multimedia then sadly it's still stuck at gstreamer 0.10 | 13:54 |
| * jonathanmaw needs to figure out if the GENIVI Demo Platform needs qt5-multimedia. | 13:55 | |
| paulsherwood | i believe it does | 13:56 |
| SotK | I think that is what I found last year | 13:57 |
| pedroalvarez | there has been some porting effort: http://permalink.gmane.org/gmane.comp.handhelds.openembedded/64265 | 13:57 |
| jonathanmaw | the yocto GDP seems to use meta-qt5, which refers to qtmultimedia | 13:57 |
| radiofree | it's a massive pain in the arse | 13:57 |
| radiofree | qtwebkit uses gstreamer 1.0, that's it | 13:57 |
| jonathanmaw | yep, this meta-qt5 includes patches to port qtmultimedia to gstreamer 1.0 | 13:58 |
| radiofree | are they the ones by yoann something? | 13:58 |
| * jjardon wondering if they sent those patches upstream | 13:59 | |
| radiofree | jjardon: if they're by yoann, then they are qt patches | 13:59 |
| jonathanmaw | radiofree: yep, by yoann | 13:59 |
| jjardon | radiofree: Martin Jansa | 13:59 |
| jjardon | http://permalink.gmane.org/gmane.comp.handhelds.openembedded/64265 | 13:59 |
| Krin | i need to get pyserial into baserock as a dependency of tempest. there are no official git repository of it, only .tar.gz on the main site and an svn repo. what would be the best was of going about getting this into baserock? | 13:59 |
| radiofree | jjardon: that's the patch to add the patches :P | 14:00 |
| SotK | Krin: write a lorry for the SVN repo and test it on your devel machine to see if it works/isn't really slow I think | 14:00 |
| SotK | then use SVN if it works, but the tar if it doesn't | 14:00 |
| jonathanmaw | git.baserock.org indicates there's a wip/gstreamer-1.0 branch | 14:00 |
| jjardon | oh, rigth. So why they are not upstream yet? | 14:01 |
| radiofree | yep, it's also on https://qt.gitorious.org/qt/qtmultimedia/ if it ever loads for you | 14:01 |
| Krin | we can lorry in SVN's? well thats something i didnt know. thanks SotK | 14:01 |
| radiofree | jjardon: it is being worked on by digia, it's probably just not ready yet | 14:01 |
| radiofree | even though it's been a wip branch for ages now... | 14:01 |
| radiofree | jonathanmaw: i would have thought yocto are patching it because they don't include gstreamer 0.10 | 14:02 |
| radiofree | if should work fine with 0.10 | 14:02 |
| radiofree | s/if/it/ | 14:02 |
| radiofree | https://qt.gitorious.org/qt/qtmultimedia/source/c1dbca5f109b34e8b3a58e241ff40a457e8e2dc6:config.tests/gstreamer | 14:02 |
| radiofree | IT'S HAPPENING! | 14:02 |
| jjardon | oh, nice, seems to they restart to work on in last decemnber | 14:04 |
| radiofree | it's in the 5.5 branch now https://qt.gitorious.org/qt/qtmultimedia/source/08058f8483bd1a681edc9aa0d31d3d28a96349b7:config.tests/gstreamer | 14:04 |
| jjardon | \o/ | 14:04 |
| tlsa | Can anyone review http://paste.baserock.org/zedaricuru please? | 14:18 |
| tlsa | it's to update nano tarball to v2.4.0 | 14:22 |
| rdale | when we update a tarball based lorry git rep, are the previous commits kept, or is the whole repo overwritten? | 14:23 |
| richard_maw | previous commits are kept | 14:23 |
| rdale | ah ok | 14:23 |
| pedroalvarez | tlsa: looks ok +1 | 14:28 |
| ssam2 | tlsa: looks fine. I can merge it if you push it to Gerrit | 14:28 |
| tlsa | oh, right I forgot about gerrit | 14:29 |
| tlsa | i'll need to get hooks, amend a change-id onto it and stuff | 14:30 |
| pedroalvarez | seriously, is not that much | 14:31 |
| ssam2 | but merging it will be 1 click, instead of a lengthy process | 14:31 |
| tlsa | pedroalvarez: yeah, didn't say it was | 14:31 |
| tlsa | just that I forgot about it, and was going to push direct to master since its just a one line change | 14:32 |
| *** edcragg has quit IRC | 14:36 | |
| *** edcragg has joined #baserock | 14:38 | |
| tlsa | ssam2: done: http://gerrit.baserock.org/#/c/79/ | 14:39 |
| radiofree | how do i change my e-mail address in gerrit | 14:41 |
| ssam2 | merged, thanks | 14:41 |
| radiofree | i can't - 500 Internal server error | 14:41 |
| ssam2 | radiofree: it's tied to your OpenID I think | 14:41 |
| radiofree | there's a "register new e-mail" option in the settings, but that 500's | 14:42 |
| ssam2 | ah, probably because it can't send emails | 14:42 |
| ssam2 | i'll hopefully fix that this week | 14:42 |
| radiofree | so should we always morph check * gerrit branches now? | 14:44 |
| radiofree | if we intend to send patches? | 14:44 |
| radiofree | when i click "sign in" it's automatically signing me in with the baserock openid :\ | 14:45 |
| radiofree | i suppose this is some cookie thing | 14:46 |
| tiagogomes_ | would a patch for adding tcpdump to tools be appreciated? | 14:49 |
| radiofree | sure | 14:50 |
| *** jonathanmaw has quit IRC | 14:52 | |
| radiofree | so i did http://fpaste.org/202715/27295262/ | 14:54 |
| radiofree | and it failed | 14:54 |
| tiagogomes_ | wdutch, just saw your patch for libpcap. Are you planning to add tcpdump as well? | 14:54 |
| radiofree | are you supposed to see some "change-id" in your commit message? | 14:57 |
| ssam2 | you're supposed to have one, yes | 14:57 |
| radiofree | how do you add that from an existing branch from gbo? | 14:57 |
| wdutch | tiagogomes_: I have no plans to - for openwrt I just need that lib | 14:57 |
| ssam2 | radiofree: commit --amend, after installing the commit hook | 14:57 |
| radiofree | i have a branch on gbo, want to submit the top two patches, i did "reword" | 14:57 |
| radiofree | ah | 14:57 |
| ssam2 | if you have the commit hook then reword might work too, I don't know | 14:58 |
| radiofree | oh | 14:58 |
| radiofree | i already have that | 14:58 |
| radiofree | ! [remote rejected] HEAD -> refs/for/master/qt5-upgrade (change 49 closed) | 14:58 |
| ssam2 | the error message '(change 49 closed)' suggests that you're using the Change-Id from Change 49 | 14:58 |
| ssam2 | seems a bit weird though | 14:58 |
| radiofree | :\ | 14:59 |
| radiofree | ooh | 14:59 |
| radiofree | it seems that the mirrored branchs on gerrit already have a change id? | 14:59 |
| radiofree | no they don't | 15:00 |
| * wdutch has a go at building a tcpdump chunk | 15:00 | |
| * radiofree is stuck | 15:00 | |
| tiagogomes_ | it builds fine as long you add libpcap as its dependency | 15:01 |
| pedroalvarez | hehe, many of us using tcpdump these days :) | 15:03 |
| radiofree | ahah! | 15:03 |
| radiofree | oh wait i think i did it wrong | 15:03 |
| pedroalvarez | wdutch: should be easy, let us know if you need help :) | 15:03 |
| radiofree | it's split up my two commits into two different things http://gerrit.baserock.org/#/c/82/ and http://gerrit.baserock.org/#/c/83/ | 15:03 |
| wdutch | tiagogomes_: do you think libpcap and tcpdump should share a lorry? | 15:04 |
| tiagogomes_ | pedroalvarez is a must for having a clue of the dark magic that Neutron/openvswitch do | 15:05 |
| pedroalvarez | :) | 15:05 |
| radiofree | how does it handle patch series? do they all have to have the same change-id? | 15:05 |
| tiagogomes_ | wdutch no strong opinion | 15:06 |
| ssam2 | radiofree: each patch has to be an individual 'change' with its own change ID | 15:07 |
| ssam2 | you can link them with 'topics' (push to refs/for/master/topicname) | 15:07 |
| pedroalvarez | wdutch: can we persuade you to use gerrit? :P | 15:07 |
| ssam2 | radiofree: if you're working on a big feature you could also have a staging branch, so push everything to refs/for/staging/myfeature/... which hopefully saves half your feature getting committed to master and half of it not | 15:08 |
| wdutch | pedroalvarez: I forgot that things have changed | 15:09 |
| * wdutch goes to read the wiki | 15:09 | |
| pedroalvarez | :) | 15:09 |
| radiofree | i see | 15:11 |
| * radiofree doesn't | 15:11 | |
| radiofree | i did push them to a single topic name though | 15:12 |
| radiofree | * [new branch] HEAD -> refs/for/master/qt5-upgrade | 15:12 |
| radiofree | it's fine for them to appear as two different things then? | 15:12 |
| pedroalvarez | radiofree: yes | 15:14 |
| radiofree | ah ok | 15:15 |
| radiofree | so if my icu patch got another +1, it would be auto-merged? | 15:21 |
| radiofree | or do both my patches need +2 before that happens? | 15:21 |
| pedroalvarez | they can be merged separately (I think) | 15:22 |
| pedroalvarez | unless you create a dependency when submitting the patch for review | 15:22 |
| *** zoli__ has quit IRC | 15:22 | |
| bashrc_ | apparently baserock uses storyboard for bug reports http://wiki.baserock.org/contributing | 15:23 |
| bjdooks | I'm getting: | 15:26 |
| bjdooks | [ TIME ] Timed out waiting for device dev-ttyS0.device. | 15:26 |
| radiofree | bjdooks: one moment | 15:29 |
| radiofree | the rootfs is on a usb stick right? | 15:30 |
| bjdooks | yes | 15:30 |
| pedroalvarez | Krin: have you actually tested that the svn import of pyserial worked? | 15:30 |
| pedroalvarez | Krin: how long it took? | 15:30 |
| radiofree | can you do grep sha1 baserock/build-system-armv7bhf-rootfs.meta | 15:30 |
| radiofree | oh btw it's not actually hf.... | 15:31 |
| bjdooks | i can run init=/bin/sh | 15:31 |
| bjdooks | echo hello > /dev/ttyS0 | 15:31 |
| bjdooks | and I get hello back | 15:31 |
| radiofree | grep sha1 /baserock/build-system-armv7bhf-rootfs.meta | 15:31 |
| bjdooks | "sha1": "eee1143e45133d74fd78fdda450f6d2e57a52028", | 15:32 |
| radiofree | is that from before you get anything from systemd? | 15:34 |
| radiofree | (the [DEPEND]) | 15:34 |
| radiofree | WORKSFORME (on a jetson) | 15:34 |
| radiofree | bjdooks: do you have CONFIG_FHANDLE enabled? | 15:37 |
| Krin | pedroalvarez, it tool about as long as it took me to read the output when i lorried it on it's own | 15:40 |
| rdale | it would be nice to lorry qtwebengine and include that with the qt 5.4 changes | 15:41 |
| bjdooks | radiofree: ah, no | 15:41 |
| Krin | i did it as a seperate lorry to test because it was taking forever to lorry in all of python-packages.lorry | 15:41 |
| pedroalvarez | Krin: cool, that makes sense | 15:41 |
| Krin | but yes it was very quick | 15:41 |
| pedroalvarez | Krin: I made a comment in your patch anyway | 15:42 |
| *** edcragg has quit IRC | 15:43 | |
| *** ssam2 has quit IRC | 15:45 | |
| *** edcragg has joined #baserock | 15:45 | |
| wdutch | could somebody please give me an example of what refs/for/master/my-topic-branch could be in git push gerrit HEAD:refs/for/master/my-topic-branch? Everything I try is getting rejected | 15:46 |
| SotK | What error is it rejected with? | 15:47 |
| wdutch | http://paste.baserock.org/qaqopaxulo | 15:48 |
| pedroalvarez | try without '/'s? | 15:48 |
| SotK | wdutch: you need the refs/for/master bit | 15:48 |
| Zara | wdutch: it's literally HEAD:refs/for/master/ followed by the name of your branch | 15:49 |
| SotK | so refs/for/master/williamholland/libpcap | 15:49 |
| wdutch | aha, thanks! | 15:49 |
| Zara | (I was unsure of that as well, and thought 'refs/for/master' might stand for something; we should make the wiki clearer) | 15:49 |
| bjdooks | Thank you for trying Baserock. | 15:50 |
| bjdooks | baserock login: root | 15:50 |
| bashrc_ | initially I also thought it stood for something, but it doesn't | 15:50 |
| bjdooks | No mail. | 15:51 |
| bjdooks | Linux baserock 4.0.0-rc4+ #23 Wed Mar 25 15:49:27 GMT 2015 armv7b GNU/Linux | 15:51 |
| radiofree | :D | 15:51 |
| rjek | \o/ | 15:51 |
| pedroalvarez | wow | 15:51 |
| * radiofree needs to update his big endian jetson | 15:51 | |
| radiofree | Linux baserock 4.0.0-rc1-00031-g4e4e5ff-dirty #9 SMP PREEMPT Mon Mar 23 17:36:16 GMT 2015 armv7b GNU/Linux | 15:51 |
| *** dabukalam_ is now known as dabukalam | 15:52 | |
| *** dabukalam has joined #baserock | 15:52 | |
| radiofree | think the jetson wins as a build platform though | 15:53 |
| radiofree | ~ # cat /sys/devices/system/cpu/cpu0/cpufreq/cpuinfo_cur_freq | 15:53 |
| radiofree | 2320500 | 15:53 |
| Krin | pedroalvarez, fixed :) | 15:53 |
| radiofree | bjdooks: CONFIG_FHANDLE fixed it then? | 15:53 |
| pedroalvarez | Krin, how easy was to send another version of the same patch? | 15:54 |
| pedroalvarez | I wonder if it's too complex, given that bashrc_ has sent a different patch to gerrit, and abandoned the first version of it | 15:56 |
| * SotK found it fairly painless earlier | 15:58 | |
| Krin | not difficult, just have to copy the change ID of the original patch and put it on the new patch, this fools gerrit into thinking it is the same patch | 15:59 |
| *** ssam2 has joined #baserock | 16:03 | |
| *** ChanServ sets mode: +v ssam2 | 16:03 | |
| franred | Krin, pedroalvarez, should we use https://code.launchpad.net/~testresources-developers/testresources/trunk rather than a tarball for testresources http://gerrit.baserock.org/#/c/81/3/open-source-lorries/python-packages.lorry ? | 16:21 |
| Krin | i don't know any more franred. i'll make it be however it needs to be to get merged. | 16:23 |
| paulsherwood | we should not lorry tarballs, normally unless there's a clear need, for (say) bootstrapping | 16:24 |
| pedroalvarez | or incompetent upstream | 16:25 |
| Krin | ok paulsherwood i'll make those changes now | 16:25 |
| *** zoli__ has joined #baserock | 16:25 | |
| ssam2 | wdutch: somehow you managed to submit your libpcap lorry patch to gerrit twice, and both have been merged | 16:27 |
| wdutch | oops | 16:27 |
| ssam2 | seems harmless though | 16:27 |
| straycat | it's not exactly that they're incompetent, it's that they're not expecting people to build from their source | 16:27 |
| wdutch | I submitted it twice because I couldn't find it the first time and thought I'd done something wrong | 16:27 |
| pedroalvarez | straycat: yes, maybe I was too harsh, sorry | 16:28 |
| Krin | what would the "type": feild be for https://code.launchpad.net/~testresources-developers/testresources/trunk ? | 16:29 |
| rdale | bzr | 16:30 |
| *** zoli__ has quit IRC | 16:43 | |
| straycat | shall we have a gerrit bot then? | 16:44 |
| * straycat has some time to kill | 16:45 | |
| *** a1exhughe5 has quit IRC | 16:51 | |
| straycat | i can also just listen to drum and bass though | 16:51 |
| pedroalvarez | a gerrit bot for changes merged? | 16:57 |
| straycat | and proposals | 16:59 |
| straycat | 's what the openstack folks seem to do | 16:59 |
| ssam2 | i'd prefer emails to an IRC bot, personally. If you have time to spare maybe you could look at finishing off infrastructure.git branch sam/gerrit-exim4 | 16:59 |
| straycat | i'd prefer emails too! heh | 17:00 |
| ssam2 | the 2 commits in that branch could be moved to definitions.git either, i just worked in infrastructure.git because it was close to hand | 17:00 |
| straycat | i'd definitely like a bot, all the other projects have one | 17:01 |
| pedroalvarez | we already have one! :) It's a supybot | 17:05 |
| pedroalvarez | brlogger: hey! | 17:05 |
| brlogger | pedroalvarez: Error: "hey!" is not a valid command. | 17:05 |
| pedroalvarez | he is still there :) | 17:06 |
| straycat | if there's a gerrit module for supybot then cool | 17:06 |
| bashrc_ | ok, is there any special magic which needs to happen in order to enable a plugin? The evidence is mounting that the firehose enable is never called | 17:12 |
| perryl | bashrc_: how do you mean, enable? it could be that firehose_plugin.py needs to go into morphlib/plugins if it isn't already | 17:14 |
| bashrc_ | firehose_plugin.py enable function | 17:14 |
| straycat | pedroalvarez, ? | 17:17 |
| perryl | ahh, i'm not sure in that case i'm afraid :( | 17:18 |
| *** Guest70582 has quit IRC | 17:21 | |
| bashrc_ | and the morph plugin manager only runs after parsing of --config. Probably there's something I'm missing about how plugins are supposed to work. | 17:23 |
| ssam2 | the plugin mechanism is provided by 'cliapp'. I doubt its innards are documented, but it's fairly simple Python code | 17:26 |
| ssam2 | git://git.baserock.org/delta/cliapp branch baserock/morph | 17:26 |
| bashrc_ | is there any parallelism in the way morph starts up? | 17:27 |
| ssam2 | it's all 1 thread | 17:30 |
| bashrc_ | so, it looks like it is reading the plugin paths but just doesn't find firehose | 17:30 |
| *** mdizzle has quit IRC | 17:46 | |
| radiofree | hmm.. you end up with a merge commit for everything now http://git.baserock.org/cgi-bin/cgit.cgi/baserock/baserock/definitions.git/log/ | 17:47 |
| straycat | radiofree, yes but we can have an irc bot! | 17:47 |
| ssam2 | radiofree: we had merge commits for everything before Gerrit, because the policy was to merge with 'git merge --no-ff'. Currently the Gerrit merge policy for all repos is set to 'merge-if-necessary', so we don't get a merge commit unless one is needed | 17:54 |
| *** tiagogomes_ has quit IRC | 17:55 | |
| ssam2 | we could set the policy to rebase-before-merge to reduce the number of merge commits | 17:55 |
| ssam2 | in fact, I meant to do that already on jjardon's suggestion but then didn't | 17:55 |
| ssam2 | i'll do it tomorrow | 17:55 |
| radiofree | ssam2: i mean *everything* in the patch series | 17:55 |
| ssam2 | oh, because each commit is separate | 17:55 |
| radiofree | there's "Merge "Update to qt5 to v5.4.0" and Merge "Fix build of ICU on Arm" | 17:55 |
| ssam2 | yes, that's a bit ugly | 17:56 |
| radiofree | that's going to look horrendous for a large patch series | 17:56 |
| straycat | i think you're missing the bigger picture | 17:56 |
| straycat | we can finally have an irc bot | 17:56 |
| bashrc_ | if gerrit is the future then the existing niggles in it could be patched | 17:56 |
| ssam2 | radiofree: like I said, I'll change the merge policy so that Gerrit rebases each commit before merging | 17:56 |
| radiofree | straycat: :-o | 17:56 |
| * radiofree drops all objectsion then | 17:56 | |
| straycat | :) | 17:56 |
| radiofree | s/objectsion/objections | 17:56 |
| ssam2 | there are only merge commits for those commits because they were not based off latest master at the time they were merged | 17:57 |
| ssam2 | but gerrit has about 10 different merge policies, I think rebase before merge will solve that | 17:57 |
| ssam2 | but, tomorrow, i'm off now | 17:57 |
| *** ssam2 has quit IRC | 17:57 | |
| *** bashrc_ has quit IRC | 18:00 | |
| *** cyndis has quit IRC | 18:06 | |
| *** edcragg has quit IRC | 18:07 | |
| *** rdale has quit IRC | 18:12 | |
| *** cyndis has joined #baserock | 18:20 | |
| *** mdunford has quit IRC | 18:25 | |
| straycat | persia, right so it seems no one wants a 0.x branch | 18:29 |
| * straycat shrugs | 18:29 | |
| *** zoli__ has joined #baserock | 18:30 | |
| *** wschaller_ has quit IRC | 18:50 | |
| straycat | hmm there might be a hack for gerrit that allows you to push a series as one change | 19:03 |
| straycat | apparently you can perform the merge locally then push the merge commit | 19:03 |
| straycat | i also didn't realise but the reviews seem to be per change, so you can't actually give a review on a series as a whole, just the individual commits | 19:04 |
| *** zoli__ has quit IRC | 21:20 | |
| *** gary_perkins has quit IRC | 21:43 | |
| *** gfinney has quit IRC | 22:18 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!