#nemomobile log for Thursday, 2012-07-05

*** M4rtinK2 has quit IRC00:05
*** M4rtinK has joined #nemomobile00:08
*** M4rtinK has quit IRC00:14
*** dionet has quit IRC00:51
*** KaIRC has quit IRC00:53
*** arcean_ has quit IRC01:03
*** KaIRC has joined #nemomobile01:06
*** himamura has quit IRC01:38
*** jluisn has joined #nemomobile02:00
*** jluisn has quit IRC02:00
*** himamura has joined #nemomobile02:01
*** himamura has joined #nemomobile02:02
*** himamura has quit IRC02:06
*** himamura has joined #nemomobile02:09
*** rcg has quit IRC02:11
*** himamura has quit IRC02:17
*** KaIRC has quit IRC02:18
*** himamura has joined #nemomobile02:20
*** himamura has quit IRC02:23
*** himamura has joined #nemomobile02:25
*** rcg has joined #nemomobile02:28
*** himamura has quit IRC02:29
*** himamura has joined #nemomobile02:32
*** himamura has quit IRC02:35
*** himamura has joined #nemomobile02:35
*** himamura has quit IRC02:39
*** himamura has joined #nemomobile02:43
*** himamura has quit IRC02:47
*** himamura has joined #nemomobile02:50
*** himamura_ has joined #nemomobile02:53
*** himamura has quit IRC02:54
*** himamura has joined #nemomobile02:57
*** himamura_ has quit IRC02:57
*** himamura has quit IRC03:01
*** himamura has joined #nemomobile03:13
*** himamura has quit IRC03:17
*** himamura has joined #nemomobile03:20
*** himamura has quit IRC03:24
*** Estel_ has quit IRC03:30
*** himamura has joined #nemomobile03:40
*** Estel_ has joined #nemomobile03:41
*** furikku has joined #nemomobile03:48
*** the-boss has quit IRC03:59
*** the-boss has joined #nemomobile04:04
*** himamura has quit IRC04:12
iekkumorning04:45
*** lofty306 has left #nemomobile05:14
rcgmorning05:21
Stskeepsrcg: did you have a fix for the calendar crash for kcalcore/qt mobility or how was it?05:21
rcgyes05:22
rcgStskeeps: i attached a updated patch to the bug report05:22
Stskeepsfor various legal reasons we can't accept patches within bugzilla.. we need it submitted through git with a signed-off-by05:22
Stskeepsi should probably note that other places05:22
rcgactually, i intended to ping you so i can learn how to do this "officially"05:23
Stskeeps:nod:05:23
rcghowever, i didn't have the time yet :(05:23
rcgalright, first question: is it sufficient that i simply add this "signed off by /me"?05:24
Stskeepsyes05:24
Stskeepslet me show you05:24
rcgcool05:24
Stskeeps.. or try..05:24
* Stskeeps pokes meego.com with a stick05:24
Stskeepshttp://gerrit.googlecode.com/svn/documentation/2.0/user-signedoffby.html05:25
rcgalright, sounds simple and fair05:27
Stskeepsit's basically so we have some protection in case anybody has submitted bad stuff, and we can find it and remove it05:28
rcgi see05:28
Stskeepsand can point blame to a specific person/commits05:28
Stskeepssame system linux kernel works with05:30
rcgright :)05:36
rcgjust registering @ gerrit05:40
*** Siosm has joined #nemomobile05:46
*** Siosm has quit IRC05:50
rcgStskeeps: does it take some time until an ssh key is accepted in gerrit?05:55
Stskeepsno, shouldn't05:55
Stskeepstell me how you test it?05:56
rcgssh -p 29418 ruedigergad@review.merproject.org gerrit ls-projects05:56
Stskeepssounds about right, did you set username in settings->profile ?05:56
rcgah right05:57
rcgusername was not set05:57
rcgthought i had set it as i registered05:57
rcgyes, works now, set it via the settings in the web gui.. apparently it was empty before05:57
Stskeepsgood05:58
*** himamura has joined #nemomobile05:59
rcgalright: http://review.merproject.org/#change,63606:05
*** Aard has quit IRC06:06
rcgStskeeps: ^06:06
Stskeepsa good start, you also need to modify the spec/yaml file ;)06:07
Stskeepsdoesn't get picked up automatically06:07
*** himamura has quit IRC06:07
Stskeepsget mer platform sdk, add patch to .yaml file, run 'specify06:07
rcgi just replaced an existing patch06:09
rcgthat is, i kinda patched a patch06:09
Stskeepsah06:09
Stskeepswell, then you need to add an entry to .changes too06:09
rcgbut i guess i would need to update the version in the yaml?06:10
Stskeepsnah, no need06:10
rcgah, missed that :)06:10
rcgwill update changes as well06:10
rcgStskeeps: http://review.merproject.org/#change,63706:13
rcgso, this is it?06:13
Stskeepssortof, https://wiki.merproject.org/wiki/Contribution_in_detail#Re-submit_if_original_change_was_rejected06:14
Stskeepsgit checkout the older sha, 0436001340b5b30d5cb74dd87a1b502a693b6411 and amend the commit06:15
Stskeepsthe goal is a proper clean commit, so you amend and re-submit06:15
rcgah, so i should have used --amend instead of simply commiting..06:16
rcghmm.. unfortunately my git skills are rather basic :(06:17
Stskeepsdon't worry :)06:19
Stskeepsit takes a bit to get used to but once it's down in muscle memory, its' quite easy06:19
rcghmm.. did that06:22
rcgbut i think it didnt pick up my change in the changes file?06:23
Stskeepsdid you git add the .changes file, then commit --amend ?06:24
Stskeepswell06:24
Stskeepsok, when you checked out your older sha, the .changes change naturally disappeared06:24
Stskeepsas you went earlier back in the source tree06:24
rcgdid: git checkout 0436001340b5b30d5cb74dd87a1b502a693b64 && vim mkcal.changes && git commit -s --amend && git push origin HEAD:refs/for/master06:25
Stskeepsah, you forgot git add mkcal.changes06:25
Stskeepsyou can just git add mkcal.changes and amend again06:26
rcgah was missing -a06:26
rcgon commit06:26
Stskeepsand remember to read about Change-id06:26
Stskeepsbefore pushing next06:26
rcgah, yeah, sorry06:28
rcgman.. i am really in a hurry...06:29
rcgthis does not compute well for me right now :(06:29
rcgalways making mistakes :(06:29
rcgand keep spamming you guys with new change id's06:29
Stskeepsit's fine, just click 'abandon change' in the end06:30
Stskeepsjust continue some other time :)06:30
rcgalright, i am just on my last try06:31
rcgwith Change-Id: in the last line i simply put "636" or do i put "Change-Id:636"?06:32
StskeepsChange-Id: Ib5a4abac1d8e882d15b67b59474faf1a12ce229006:32
Stskeepssee top of the changes page06:32
rcgi see, and should i prefix it with "Change-Id:"?06:33
Stskeepsjust as is, 'Change-Id: Ib5a4abac1d8e882d15b67b59474faf1a12ce2290'06:34
rcgyay, looks like this time i got it right :)06:35
rcghttp://review.merproject.org/#change,63606:35
Stskeepsyup, i see a patch set 206:35
rcgthanks a lot Stskeeps for the help and patience.. :) and now i gotta head off to shower and sprint to work ;)06:36
Stskeepsenjoy06:36
rcgStskeeps: alright, just quickly abandoned the changes i spammed accidently06:44
rcghope this was the right way to do it06:44
Stskeeps:nod:06:45
*** jukkaeklund has joined #nemomobile07:50
*** rcg-work has joined #nemomobile08:02
*** ronoc has joined #nemomobile08:06
*** slaine has joined #nemomobile08:33
*** cxl000 has joined #nemomobile08:50
*** M4rtinK has joined #nemomobile09:04
*** phaeron has quit IRC09:11
*** M4rtinK has quit IRC09:14
*** arcean has joined #nemomobile09:17
rantomHas there been some planning on what to do with the desktop in Nemo Mobile?09:17
rantomAs far as I know it shows the background wallpaper, nothing else09:17
rantomIn english: is there a need to preserve the desktop as it is now or should it just be an application-grid/etc?09:18
rantomAlthough I could understand if it would be used to display e.g. widgets09:23
*** M4rtinK has joined #nemomobile09:30
*** himamura_ has joined #nemomobile09:33
*** phaeron has joined #nemomobile10:36
*** KaIRC has joined #nemomobile10:37
*** phaeron1 has joined #nemomobile10:40
*** phaeron has quit IRC10:42
*** MohammadAG has quit IRC10:52
*** MohammadAG has joined #nemomobile10:52
*** niqt has joined #nemomobile10:52
*** smyows has joined #nemomobile11:18
*** smyows has joined #nemomobile11:18
*** arcean has quit IRC11:23
*** arcean has joined #nemomobile11:25
*** lizardo has joined #nemomobile11:37
Sage_Problem: nothing provides /usr/bin/ruby needed by meegotouch-applauncherd-tests-3.0.3-3.3.Project.MTF.MW.armv7hl11:54
Sage_...11:54
Stskeepsyes, that's my reaction to some of this stuff too11:55
Stskeepsyou're thinking TDriver11:55
Sage_yes11:55
* Sage_ though ruby was already in Mer:Tools:Testing with tdriver11:56
Sage_ah, no. It is only in my prj. https://build.pub.meego.com/project/packages?project=home%3Asage%3Abranches%3AMer%3ATools%3ATesting11:57
* Sage_ though he pushed those already :/11:59
*** himamura_ has quit IRC12:05
*** jd is now known as Jade12:12
*** lofty306 has joined #nemomobile12:17
*** ronoc has quit IRC12:21
*** jluisn has joined #nemomobile12:29
*** rcg-work1 has joined #nemomobile12:45
*** rcg-work has quit IRC12:46
*** lofty306 has left #nemomobile12:58
*** ronoc has joined #nemomobile13:03
*** ronoc has quit IRC13:09
*** jluisn_ has joined #nemomobile13:16
*** jluisn has quit IRC13:17
*** DocScrutinizer has quit IRC13:26
*** DocScrutinizer has joined #nemomobile13:26
*** ronoc has joined #nemomobile13:37
*** niqt has quit IRC13:38
*** ronoc has quit IRC13:44
*** ronoc has joined #nemomobile13:45
*** niqt has joined #nemomobile14:28
*** niqt has quit IRC14:41
*** jluisn has joined #nemomobile15:21
*** faenil has joined #nemomobile15:21
*** jluisn_ has quit IRC15:23
*** phaeron1 has quit IRC15:44
*** slaine has quit IRC15:44
*** himamura has joined #nemomobile15:47
*** NIN101 has joined #nemomobile15:55
*** himamura has quit IRC16:02
*** himamura has joined #nemomobile16:05
*** himamura_ has joined #nemomobile16:30
*** radiofree has quit IRC16:32
*** radiofree_ has joined #nemomobile16:32
*** himamura has quit IRC16:33
*** himamura_ has quit IRC16:55
*** himamura has joined #nemomobile16:56
*** himamura has quit IRC16:56
*** himamura has joined #nemomobile16:59
*** radiofree_ is now known as radiofree17:02
*** himamura_ has joined #nemomobile17:04
*** himamura has quit IRC17:07
*** dionet has joined #nemomobile17:08
*** NIN101 has quit IRC17:15
*** the-boss has quit IRC17:18
*** rcg-work1 has quit IRC17:19
*** the-boss has joined #nemomobile17:27
*** nsuffys has joined #nemomobile17:41
*** himamura has joined #nemomobile17:55
*** faenil has quit IRC17:57
*** phaeron has joined #nemomobile17:58
*** himamura_ has quit IRC17:58
*** himamura_ has joined #nemomobile18:02
*** himamura_ has joined #nemomobile18:02
*** himamura has quit IRC18:05
*** furikku has quit IRC18:08
*** arcean has quit IRC18:28
*** NIN101 has joined #nemomobile18:44
*** nsuffys has quit IRC19:25
*** ronoc has quit IRC19:34
*** ronoc has joined #nemomobile19:49
rcgSage_: ping20:08
Sage_rcg: pong20:12
rcghow do you prefer the patch?20:13
rcghttp://review.merproject.org/#change,63620:13
rcgi plan to just add myself as "signed-off-by" for the particular change20:14
Sage_rcg: Well, one way is to do: git init; git add *; git commit -m.; CHANGES ; git commit -a; git format-patch -120:14
rcgam currently figuring out how to do the git diff stuff20:14
rcgalright20:15
Sage_but you can also checkout the upstream tree if you want, but that for the tarball works as well20:15
rcgi plan to preserve your "signed-off-by" message and just add me to it20:15
rcgalright, thanks20:15
rcgi think i slowly get into the deeper bits of git20:15
Sage_:)20:16
Sage_that is the way it usually goes20:16
rcgi see :)20:16
rcgthis morning i felt quite stupid when Stskeeps guided me through the process in gerrit.. but i hope i remember everything and don't spam gerrit again ;)20:18
rcgSage_: http://review.merproject.org/#change,63620:36
rcgi _think_ that should have worked20:36
*** ronoc has quit IRC20:44
Sage_rcg: is the notifyOpened used somewhere btw?20:53
rcgit is20:53
rcgiirc in some places20:53
Sage_ok20:54
rcgso i figured letting it return false instead of removing the calls would be easier20:54
rcgalso returning false corresponds to its logic20:54
Sage_well that patch seems ok to me.20:55
rcggreat :)20:55
Sage_.changes needs fix though :)20:56
Sage_- Fixes MER#387: summary...20:56
Sage_not - Fix #MER387: summary...20:56
rcgalright, the very details ;)20:57
rcgbut am getting closer :)20:57
Sage_well, I hope we get at somepoint parser that automatically reports also to bugzilla about changes thus it will be nice to keep syntax same.20:58
rcgsure, didn't want to nag ;)20:59
rcgjust commited the fixed changes file20:59
Sage_rcg: I don't want to nag, but it is MER#387 not #MER387 ;)21:02
*** dionet has quit IRC21:03
rcgah, damn... missed that, gimme just a sec :)21:03
Sage_Well, take 10 ;)21:05
rcgalright21:05
rcghope that's it21:05
rcg;)21:05
Sage_well, gave +1 instead of -2 ;)21:06
*** NIN101 has quit IRC21:06
rcgright :)21:06
rcgwell, that last details were tougher work than i expected ;)21:07
*** jluisn has quit IRC21:10
lbtrcg: it's OK, I just -1'ed one of Sage_'s submits :)21:12
*** smyows has quit IRC21:14
rcglbt: i see :)21:16
rcgit always is kinda relieving to see you are not alone ;)21:17
lbthehe... you should see my attempts...21:17
lbtthe problem is that when you don't do it everyday, you miss things21:17
rcganyway, learned quite a lot.. and i really need to update my git cheat sheet before i loose my .bash_history :)21:17
lbtthat's why I wrote the tutorial.... to remind myself21:18
rcghehe, yeah, right :)21:18
lbtfeels good to get your first change in though ... congrats :)21:19
lbtany comments on the wiki page?21:19
rcgah, well, it's not completely through yet, need another +1 ;)21:22
Sage_lbt: about the 191 I'm not sure if that is relevant21:22
rcgbut thanks, yeah, it is quite interesting to work on the official mer :)21:22
rcgpersonally i found the wiki page very informative but there could be a section that describes how to deal with rejected changes in more detail21:23
Sage_lbt: ah, and 412 is actually fixed already. Just didn't know that there was bug about it :)21:23
rcgpersonally, i missed an example of the command lines used for git21:23
Sage_lbt: will add comment about 41221:23
rcgand also an actual example on how to place the "Change-Id:" entry in the change log21:24
rcgdunno why but i had some blocks in my mind on doing this and a real life example would have helped imho21:25
Sage_lbt: I would have given -2 but you are too nice ;)21:25
lbtSage_: the 191 is more that you're touching the package and 191 should be a trivial thing to avoid 2 change/review cycles21:26
lbtsince vendors should do QA on it21:26
lbtin fact...21:26
lbtwe should actually do an 'outstanding bugs for this package' check21:26
Sage_I always try to do search with package name before pushing any changes to check if I can fix other things as well.21:27
lbtrcg: OK - if you want to make some edits there and ask someone (me, sage, Stskeeps) to validate then that'd be good21:27
Sage_lbt: I didn't get the 191 comment.21:28
lbtSage_: yeah - that should be in the checklist on the wiki actially21:28
lbthttps://bugs.merproject.org/show_bug.cgi?id=191 ?21:29
Sage_yes. the opensslconf.h file is different on intel and arm. On ARM it uses the one provided on upstream and on Intel it uses the one originated from fedora21:30
Sage_I don't know if ARM has any multilib arches that would need that patch21:31
rcglbt: yes, i hope i'll find the time to make some personal notes first and then might edit the wiki21:31
Sage_e.g. on intel you might want to install 64bit and 32bit packages thus the fix is there if I understood correctly.21:32
rcgunfortunately there are only so many hours in a day.. same problem as usual, so much to do, so few time21:32
Sage_Time to sleep.. good night everyone.21:33
lbtSage_: hmm21:33
lbtOK21:34
lbtanyhow, g'night21:34
rcgbtw, how do you guys manage to invest that much time into mer/nemo?21:40
*** lizardo has quit IRC21:58
*** cxl000 has quit IRC22:32
*** DocScrutinizer05 has joined #nemomobile23:15
*** M4rtinK has quit IRC23:30
*** bef0rd has joined #nemomobile23:39

Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!