| === ajmitch_ is now known as ajmitch | ||
| balloons | renato_: your mp does fix calendar :-) | 00:03 |
|---|---|---|
| renato_ | balloons, good | 00:05 |
| renato_ | balloons, but the clock still broken? | 00:06 |
| balloons | renato_: yes I updated the MP with my notes. Essentially clock is unchanged | 00:07 |
| balloons | renato_: any thoughts on what's happening with clock/ | 00:14 |
| renato_ | balloons, no idea | 00:16 |
| renato_ | I will discuss with szombi tomorrow to see if he can filter the alarms | 00:16 |
| renato_ | on alarm api | 00:16 |
| balloons | renato_: ok, so we'll hold another day then to see if we can fix clock also? | 00:19 |
| balloons | thanks for digging into this | 00:19 |
| === tsimpson_ is now known as tsimpson | ||
| dholbach | good morning | 06:31 |
| === zequence_ is now known as zequence | ||
| dpm | zsombi, so to give you some background: we're making the Weather app convergent, and it works well on N4 and N10: we've got a default phone layout and a tablet layout | 08:37 |
| dpm | However, on the N7 we've got an issue whereby we've got a footer that should expand the whole width of the screen in landscape, but for some reason is stuck at a fixed size in between | 08:38 |
| dpm | let me show you a pic... | 08:38 |
| dpm | zsombi, here it is: http://imgur.com/4a8bXJk,h3mrXJM#1 the "The Weather Channel" footer has width = mainView.Width, and strangely enough, it doesn't have that width on the N7 | 08:40 |
| zsombi | dpm: so it should go all along to the edge of the screen, right? | 08:40 |
| dpm | zsombi, exactly. Strangely enough, that works on N4 and N10, but on N7 it fails as in the picture. Here's the bit of code that does that: http://bazaar.launchpad.net/~dpm/ubuntu-weather-app/n7-tweaks/view/head:/components/LocationTab.qml#L168 | 08:41 |
| dpm | we have been trying to debug this for a couple of days to get a convergent layout in for MWC, but we were a bit tapping in the dark, | 08:42 |
| dpm | and I thought you might spot something obvious or perhaps help us think of a workaround | 08:42 |
| === chriadam_ is now known as chriadam|away | ||
| dpm | morning popey, have you seen balloons e-mail? Can we get weather and all the other apps uploaded to the store? | 09:11 |
| popey | yeah. | 09:12 |
| zsombi | dpm:guys, ConditionalLayout can take only ONE component. Check the documentation: the ConditionalLayout.layout default [property is a Component. While this does not give error on parsing, it may not work properly | 09:12 |
| zsombi | dmp: quote from ConditionalLayout doc: "There must be only a single component defined in the ConditionalLayout which will be instantiated upon activation." | 09:13 |
| zsombi | dpm: in the code I see loads of components declared under the ConditionalLayout, so no wonder the items are not laid out properly | 09:13 |
| popey | dpm: has balloons run the AP tests on these do you know? | 09:14 |
| zsombi | dpm: so the code should embedd all those Rectangles and ItemLayout components in ONE single element | 09:15 |
| dpm | popey, all I know is the e-mail we got from him | 09:20 |
| dpm | zsombi, ok, thanks a lot for looking. We've got no weather developers around, so I'll try to give it a go | 09:21 |
| zsombi | dpm: ok, good luck :) | 09:22 |
| dpm | and patience :) not having a device to test makes it even more "fun" | 09:22 |
| zsombi | dpm: you can alter the MainWidth to have 90 GU for instance, then the ConditionalLayout will activate straight away | 09:23 |
| dpm | popey, the r197 package from Jenkins did pass the tests. My guess is that he built a new .click manually based on that | 09:23 |
| dpm | zsombi, yeah, to activate the layouts I can do manual resizing on the desktop, what's been challenging has been to trigger the not-full-width footer, as on the desktop it works well | 09:24 |
| zsombi | dpm: that's weird! | 09:25 |
| dpm | zsombi, wait, all the elements are already inside one single Rectangle, it's just that the indentation is off | 09:26 |
| === mpt_ is now known as mpt | ||
| zsombi | dpm: ah, ok... the indenting made me think it's not :) | 09:26 |
| dpm | yeah, just noticed that too, let me quickly fix that | 09:26 |
| zsombi | dpm: in QtC Ctrl+A, Ctrl+I, then Strl+S :) | 09:31 |
| dpm | argh, too late, already fixed in manually in qt creator :) | 09:33 |
| dpm | but nice to know | 09:33 |
| dpm | zsombi, ok, the indentation should now be correct, so I guess it's back to the drawing board: http://bazaar.launchpad.net/~dpm/ubuntu-weather-app/n7-tweaks/view/head:/components/LocationTab.qml#L168 | 09:33 |
| zsombi | dpm: question: doesn't the Page fill the entire MainView client area? | 09:35 |
| zsombi | dpm: I don't see reason to use the mainView property to stick to the app's width | 09:36 |
| zsombi | dpm: if some component in the hierarchy overrides that property, then you'd get in trouble | 09:36 |
| zsombi | dpm: which may also be the case | 09:37 |
| JamesTait | Good morning all; happy Friday, and happy International Mother Language Day! :-D | 09:37 |
| dpm | zsombi, so you're suggesting to use parent.width instead? (I'm not sure if the Page fills the whole area) | 09:37 |
| zsombi | dpm: if the Page does fill it, then yes, use that. | 09:38 |
| zsombi | dpm: but I'd use anchors instead of setting the width | 09:38 |
| zsombi | dpm: I see the developer doesn';t really use those... which is bad | 09:38 |
| dpm | parent.width works on desktop | 09:39 |
| dpm | let me try with anchors. Right now we anchor the footer to the bottom, but nowhere else | 09:39 |
| dpm | zsombi, that works as well (on desktop) | 09:40 |
| dpm | zsombi, so you're suggesting to use anchoring (left, right) to the parent instead of specifying the width? | 09:41 |
| dpm | for the footer | 09:41 |
| zsombi | dpm: right | 09:41 |
| dpm | ok, cool, let's give it a spin. popey, would you mind trying a new .click on the device? | 09:41 |
| zsombi | dpm: and in general in the code, whenever you have parent.width, use anchors | 09:42 |
| dpm | ok | 09:42 |
| zsombi | dpm: same for the heights. | 09:42 |
| popey | dpm: sure | 09:42 |
| zsombi | dpm: this is a really bad practice... | 09:42 |
| zsombi | dpm: also, Label's anchor.verticalCenter will not give the proper result | 09:43 |
| zsombi | dpm: it should be the Label's verticalAlignment property be used | 09:43 |
| dpm | ok, fixing that too | 09:44 |
| dpm | zsombi, and the value assigned should also be parent.verticalAlignment? | 09:45 |
| zsombi | dpm: nope, you shoudl anchor it's top and bottom to the corresponding parent anchorLines, then use Label.verticalAlignment: Text.AlignVCenter | 09:46 |
| zsombi | dpm: so, anchors.top: parent.top; anchors.bottom: parent.bottom; verticalAlignment: Text.AlignVCenter | 09:47 |
| dpm | gotcha | 09:47 |
| * zsombi goes out 4 lunch | 09:48 | |
| === zsombi is now known as zsombi|lunch | ||
| dpm | popey, sorry, my p.c.c account seems to be failing, so no wget. Can you get the .click from http://ubuntuone.com/59GSo0qyNze0m9TGwV70Kj ? | 09:53 |
| popey | sure | 09:53 |
| dpm | hi nik90, reading this: https://bugs.launchpad.net/ubuntu/+source/qtorganizer5-eds/+bug/1282129 - do you think we need a new clock release to limit the date events are returned for? | 09:56 |
| ubot2 | Launchpad bug 1282129 in qtorganizer5-eds (Ubuntu) "Clock and calendar tests fail with static void QOrganizerEDSEngine::itemsAsyncListed(ECalComponent*, time_t, time_t, FetchRequestData*) " [High,Triaged] | 09:56 |
| m-b-o | dpm popey hey | 10:31 |
| popey | hey m-b-o | 10:31 |
| m-b-o | popey dpm reading back the logs .... | 10:32 |
| dpm | m-b-o, good news, would you mind reviewing https://code.launchpad.net/~dpm/ubuntu-weather-app/n7-tweaks/+merge/207616 :-) | 10:32 |
| m-b-o | dpm yes :) | 10:33 |
| dpm | m-b-o, ah, just a minute, I'll remove those print statements. The diff looks big, but it's simply because it fixes the indentation | 10:34 |
| m-b-o | dpm: ubuntu-weather-app.desktop is added. ok? | 10:34 |
| dpm | m-b-o, yeah, I'll remove it to make sure Jenkins doesn't trip on it, but I'm planning to add it post-MWC to make it possible again to do Ctrl+F12 | 10:35 |
| m-b-o | dpm: ready? | 10:41 |
| dpm | m-b-o, ok, it's now ready for review. Please have a look at the description of the MP, which includes a link to a simplified diff to make it clearer to see the real changes between all those indentation fixes. If possible, please test on a N4 as well, to make sure it still works there, and we'll get cwayne to test on a N10 in the afternoon | 10:41 |
| m-b-o | okay | 10:42 |
| dpm | I need to run now, will be back in ~1h | 10:42 |
| * dpm is really excited about getting convergence in! | 10:42 | |
| === lool- is now known as lool | ||
| nik90 | dpm: hey good morning | 11:21 |
| nik90 | dpm: Concerning your question about a new clock release, the answer is it depends. If the Alarms API provides filtering support by data, perhaps I need adjust that in the clock as well. If not, then there is nothing much I can from the clock app in which case we do not need a new release. | 11:22 |
| nik90 | s/data/date | 11:23 |
| renato_ | balloons, I have a fix for you to test, are you around? | 11:34 |
| renato_ | nik90, hi | 11:38 |
| renato_ | nik90, do you have the device to test a patch for me? | 11:38 |
| === zsombi|lunch is now known as zsombi | ||
| nik90 | dpm: Can you check if the MP https://code.launchpad.net/~charlesk/indicator-datetime/alarms/+merge/204420 has landed on the phone or not. It concerns the alarms notification fix for the indicator-datetime package. | 12:25 |
| dpm | hi renato_, balloons won't be around until in ~3h time | 12:26 |
| dpm | nik90, just a sec, just came back from an errand and I need to check how we're looking for Weather | 12:26 |
| nik90 | dpm: np | 12:28 |
| dpm | fginther, ok, we got weather working with tablet layout across all devices. When you're around, could you please trigger a click build for weather and we'll upload that to the store? | 12:41 |
| zsombi | nik90: hey dude | 12:42 |
| dpm | fginther, could you tell us for what time of the day the click builds are scheduled? This might save you some pinging (i.e. if I know when they're getting build and the changes are in that time window, I'd just wait for the build to happen) | 12:42 |
| nik90 | zsombi: hi | 12:43 |
| zsombi | nik90: we're trying to sort out the alarm API issues, and seems you are not cleaning the alarms made during the clock app test, right? | 12:43 |
| nik90 | zsombi: yes, I am not sure how exactly to clear the alarms database after a test | 12:44 |
| nik90 | I was planning on working on it with elopio and balloons later | 12:44 |
| zsombi | nik90: you remove all alarms that have special message | 12:44 |
| nik90 | zsombi: special message? | 12:45 |
| zsombi | nik90: when you create an alarm, you give a message text, right? | 12:45 |
| nik90 | zsombi: yeah I give it the alarm a label | 12:46 |
| nik90 | zsombi: as a temp solution, it wouldn't be hard adding a "swipe delete alarm" action after a test is done to clean up after itself | 12:46 |
| zsombi | nik90: awesome, so you give the "test_XXXX" label to it, right? | 12:46 |
| nik90 | zsombi: yes | 12:46 |
| zsombi | nik90: IT MUST BE :) | 12:46 |
| zsombi | nik90: otherwise all test alarms will land in the Alarm DB... | 12:47 |
| nik90 | zsombi: the proper way of doing this as suggested by renato and elopio was not the swipe delete alarm action | 12:47 |
| zsombi | nik90: which is not a good idea to pollute the Alarm DB with test stuff :) | 12:47 |
| nik90 | zsombi: since if the test fails midway, the test alarm still remains in the db | 12:47 |
| nik90 | zsombi: also the alarms AP tests were only enabled 2 days ago :) .. so I can patch it up to delete the alarm after the test | 12:48 |
| zsombi | nik90: ehh.... so there's no emergency cleanup? :( baaaaad test suite | 12:48 |
| nik90 | zsombi: well that's the thing, other stuff in the clock app uses u1db which creates a local db in the application db. This allows for a proper clean up | 12:49 |
| nik90 | zsombi: in the case of alarms, the EDS doesn't make it easier to do this | 12:49 |
| nik90 | zsombi: I just need more time to do the cleanup. It was ignored just postponed for after MWC | 12:50 |
| zsombi | nik90: once the Alarm API will have the ability to create separate collection for each app, that would get solved, as test alarms will land in a separate collection, thus separate DB | 12:50 |
| zsombi | nik90: ok, np, just wanted you to know that the cleanup is an absolute must :) | 12:50 |
| nik90 | zsombi: lol I mean it *wasn't* ignore just postponed :) | 12:51 |
| zsombi | nik90: ok :) | 12:51 |
| nik90 | zsombi: completely agree on your cleanup | 12:51 |
| nik90 | zsombi: I will have a bug report to track this | 12:51 |
| zsombi | nik90: kewl | 12:51 |
| zsombi | thx!!! | 12:51 |
| nik90 | kewl ? | 12:52 |
| daker | cool | 12:53 |
| nik90 | lol | 12:53 |
| zsombi | nik90: as daker says :D | 12:53 |
| nik90 | zsombi: just so you know, I was able to fix the convergence "animation" issue by using the default layout option ... thnx for that :) | 12:57 |
| zsombi | nik90: ;) I knew you can make it | 12:58 |
| zsombi | nik90: it is always like that: choose a default layout, then implement the other ones :) | 12:59 |
| zsombi | nik90: for code clarity, you can also have the ConditionalLayouts in different files | 12:59 |
| nik90 | zsombi: yeah I overlooked the documentation there | 13:00 |
| nik90 | zsombi: wow really? I will look into that in the coming weeks..I like separating stuff into smaller files for clarity | 13:00 |
| zsombi | nik90: the tutorial also uses some layouts like that... for clarity, the default layout for instance is in a separate file | 13:01 |
| zsombi | nik90: well, these are all QML elements after all :) | 13:01 |
| zsombi | nik90: so you can declare them in separate files as you wish :) | 13:01 |
| nik90 | zsombi: I guess I am stating an obvious fact here, but nice conditional layout tutorial | 13:02 |
| zsombi | nik90: thx, glad it's understandable :) | 13:02 |
| === gatox is now known as gatox_brb | ||
| renato_ | charles, me and zsombi are discussing about alarms on EDS, we have a problem to query for alarms because right now the alarm API query for events without a limit of dates | 13:33 |
| renato_ | this is causing a problem because every alarm is a recurrence event and this cause the function to take longer to process and return "unlimited" events | 13:33 |
| zsombi | charles_: ^^ | 13:33 |
| renato_ | zsombi, can we go with my weekly patch until we decide about that? | 13:36 |
| renato_ | nik90, balloons could you guys test it: https://code.launchpad.net/~renatofilho/ubuntu-ui-toolkit/limit-alarms-fetch/+merge/207629 | 13:36 |
| zsombi | renato_: ok, just make a big FIXME note that this is a temporary patch till we get a better setup | 13:37 |
| renato_ | popey, dpm , do you know any EDS expert that can help us to decide about that? | 13:37 |
| popey | renato_: ted? | 13:38 |
| popey | (I don't know, no) | 13:38 |
| zsombi | renato_: based on the structure, I do not see the way to set occurrences to the Alarm... | 13:39 |
| renato_ | zsombi, ECalComponentAlarms ha a list of ECalComponentAlarmInstance | 13:40 |
| zsombi | renato_: yep... hmm... all these would also be stored in a DB, so not sure whether we solve anything with it :( let's see when charles_ gets online... | 13:41 |
| dpm | renato_, I'd hint at ted, kenvandine or seb128. Or perhaps desrt | 13:41 |
| zsombi | renato_ or if not today, then let's try to catch up with him next week, maybe popey could get us on a meeting, like the one we had this week | 13:42 |
| dpm | renato_, someone on #ubuntu-desktop might be able to help with EDS | 13:42 |
| seb128 | renato_, dpm: kenvandine is probably your best guess | 13:42 |
| renato_ | zsombi, popey yes a meeting would be nice | 13:44 |
| popey | ok, I'll schedule it | 13:46 |
| popey | done | 13:48 |
| === Pici` is now known as Pici | ||
| === gatox_brb is now known as gatox | ||
| fginther | dpm, I'm building weather now. I'm also trying a change to the job where it should check trunk hourly and build a new package if trunk has changed | 14:24 |
| dpm | fginther, that sounds great, thanks! | 14:25 |
| ahayzen | popey, ping | 14:30 |
| popey | ahayzen: hey | 14:31 |
| popey | fginther: i bounced 197 weather back too because the key.js was missing the api key | 14:31 |
| ahayzen | popey, has the move to cmake broken the script u gave me to test the music-app on device? http://paste.ubuntu.com/6721815/ | 14:31 |
| popey | yes | 14:32 |
| popey | ⍨ | 14:32 |
| popey | ahayzen: http://pastebin.ubuntu.com/6949462/ can be used to build a click locally | 14:33 |
| ahayzen | popey, ok, how do i then deploy it? | 14:33 |
| fginther | popey, dang... | 14:34 |
| popey | ahayzen: http://paste.ubuntu.com/6971167/ | 14:34 |
| ahayzen | popey, thanks i'll try tht :) | 14:34 |
| popey | fginther: so please can 197 be resubmitted first, with key, so I can test and accept that as fallback, then the later version as per dpm request? | 14:35 |
| popey | np | 14:35 |
| fginther | popey, sure, let me figure out what's up first | 14:36 |
| balloons | do you need the rev197 branch I used popey? | 14:36 |
| balloons | by default the rev197 branch isn't exactly what you are looking for | 14:36 |
| fginther | popey, what I would do is rebuild with revno 197 of trunk | 14:37 |
| popey | balloons: i don't know what fginther needs, but it needs to be the 197.2 + key.js, thats all | 14:43 |
| fginther | balloons, I need some more context if there was a special branch used | 14:45 |
| ahayzen | popey, tht doesn't seem to work for me it complains tht grilo isn't installed, is this because we aren't bundling it within the click package? | 14:50 |
| popey | ahayzen: that would be https://bugs.launchpad.net/music-app/+bug/1282723 | 14:50 |
| ubot2 | Launchpad bug 1282723 in Ubuntu Music App "Need cmake rule to build the grilo plugin" [High,Fix released] | 14:50 |
| ahayzen | popey, yep which is 'fix released' :/ | 14:51 |
| popey | sergiusens: is there an easy fix for the music devs to be able to do that? | 14:51 |
| popey | yeah, but fixed on backend, no good for you | 14:51 |
| balloons | popey: fginther I included the cmake file and icon from newer versions so everything built properly | 14:53 |
| fginther | balloons, ah, so this was a one-off package that you built? | 14:53 |
| balloons | popey: fginther I pushed it here, the diff is small. lp:~nskaggs/ubuntu-weather-app/custom-rev-197 . That said, the new version is pushed for your review, including key this time | 14:53 |
| balloons | fginther: yes, the straight build of 197 popey rejected for various reason, like the icon :-) | 14:54 |
| sergiusens | popey, I can craft something, but is it for now now? | 14:56 |
| sergiusens | ahayzen, what's the plan for grilo btw; is it going away? | 14:57 |
| sergiusens | jim has made excellent progress on the media hub and it may land in the archives next week | 14:57 |
| ahayzen | sergiusens, yep we should be moving to the new mediascanner API and media-hub soon | 14:57 |
| sergiusens | (not image though) | 14:57 |
| popey | 19:02:29 < jhodapp> hey guys, so media-hub is fully working for music-app playback now | 14:58 |
| popey | 19:02:54 < jhodapp> I'm working to get it pushed up to launchpad and then put into the build, although it won't get used by default until I land the video portion too | 14:58 |
| popey | 19:03:14 < jhodapp> but I'll let you know how you can utilize it from music-app if interested | 14:58 |
| popey | ^^^ what he said yesterday | 14:58 |
| sergiusens | popey, he put the landing task upon me though ;-) | 14:58 |
| ahayzen | popey, thts awesome, i saw the G+ post :) | 14:58 |
| popey | hah sergiusens | 14:58 |
| popey | sergiusens: the problem is we have bugs and the developers ( ahayzen ) can't debug on device. | 14:59 |
| sergiusens | ahayzen, how are you building now? | 15:00 |
| ahayzen | sergiusens, i just tried the script popey gave me to build locally http://pastebin.ubuntu.com/6949462/ | 15:01 |
| ahayzen | sergiusens, and then something similar to this to deploy http://paste.ubuntu.com/6971167/ | 15:01 |
| sergiusens | popey, have you tried click-buddy --dir [trunk] --provision ? | 15:02 |
| sergiusens | it does all that | 15:02 |
| popey | woah, nice | 15:05 |
| popey | yeah, do that ahayzen ☻ | 15:05 |
| balloons | ahayzen: http://www.theorangenotebook.com/2014/02/click-buddy-and-you-how-to-test-your.html | 15:05 |
| popey | \o. | 15:05 |
| ahayzen | popey, sergiusens, IIRC i tried tht yesterday to test dpm's branch but i'll try again :) | 15:05 |
| sergiusens | still doesn't solve the plugin thing... | 15:05 |
| renato_ | balloons, hei, could you test the fix on SDK for the clock app? | 15:05 |
| sergiusens | I can give you an instant gratification hack if you want | 15:06 |
| sergiusens | ahayzen, ^^ | 15:06 |
| renato_ | balloons, https://cohttps://code.launchpad.net/~renatofilho/ubuntu-ui-toolkit/limit-alarms-fetch/+merge/207629de.launchpad.net/~renatofilho/ubuntu-ui-toolkit/limit-alarms-fetch/+merge/207629 | 15:06 |
| balloons | renato_: sure thing | 15:06 |
| ahayzen | sergiusens, wht would this hack be? | 15:06 |
| renato_ | balloons, the correct link: https://code.launchpad.net/~renatofilho/ubuntu-ui-toolkit/limit-alarms-fetch/+merge/207629 | 15:06 |
| sergiusens | ahayzen, after installing the click on the device do | 15:07 |
| dpm | sergiusens, would it make things easier if we merged the plugin's branch into the app's branch so that it gets built from a single project in LP? | 15:11 |
| === charles_ is now known as charles | ||
| sergiusens | ahayzen, http://paste.ubuntu.com/6971350/ | 15:13 |
| sergiusens | dpm, yes, it would; like reminders? | 15:13 |
| dpm | sergiusens, yes, exactly | 15:13 |
| sergiusens | dpm, that it would; but you said music is soon going to be a pure breed app :-) | 15:13 |
| sergiusens | dpm, wouldn't mind it for terminal and filemanager though | 15:14 |
| dpm | sergiusens, yes, good point. Let's leave it for terminal and filemanager, which will definitely need it | 15:14 |
| charles | renato, i-datetime has the same problem as qtorganizer5-eds there; both are using an unbounded query in the ECalClientView | 15:14 |
| sergiusens | ahayzen, did that solve the issue? | 15:15 |
| charles | it's a little easier to tweak this to meet our needs in the C alarm API and in datetime, since we know we only care about (for example) [now ... now + 1 month] | 15:15 |
| ahayzen | sergiusens, it doesn't appear to have fixed it :/ | 15:16 |
| balloons | renato_: looking good so far | 15:16 |
| ahayzen | sergiusens, this is how i'm installing http://pastebin.ubuntu.com/6971374/ | 15:16 |
| charles | renato_: but is there an API in qtorganizer s.t. ubuntu-ui-toolkit can do the same kind of thing, where its alarm model can specify a range | 15:16 |
| sergiusens | ahayzen, lines 9 through 13 aren't needed | 15:17 |
| sergiusens | ahayzen, well, 9 through 19 aren't | 15:18 |
| ahayzen | sergiusens, does the --provision install on device for u? | 15:18 |
| sergiusens | ahayzen, after installing the click, did you go through my pastebin? | 15:18 |
| sergiusens | ahayzen, yes, the click and the tests | 15:18 |
| ahayzen | sergiusens, i did... i'll try again | 15:18 |
| balloons | renato_: so I ran ubuntu_clock_app.tests.test_alarm and no more eds errors. I would say the fix worked. I can check the other apps to make sure it didn't break them. I'll run calendar at least | 15:19 |
| renato_ | balloons, ok nice | 15:20 |
| renato_ | I will fix the comments on the MR which should not cause any regression, and then we can do the last test before ask to release it | 15:21 |
| sergiusens | ahayzen, if you want a quicker have 'mount -o remount,rw / && apt-get install qtdeclarative5-qtgrilo0.1 qtdeclarative5-qtpowerd0.1 -y && mount -o remount,ro /' | 15:23 |
| ahayzen | sergiusens, nope does it make a difference it is running from /opt/click.ubuntu.com/com.ubuntu.music/1.1.350/ | 15:23 |
| sergiusens | ahayzen, do find /opt/click.ubuntu.com/com.ubuntu.music/1.1.350/ | 15:23 |
| sergiusens | and pastebin that | 15:23 |
| sergiusens | ahayzen, how are you running ? | 15:23 |
| ahayzen | sergiusens, clicking on it from the dash | 15:23 |
| ahayzen | sergiusens, http://pastebin.ubuntu.com/6971404/ | 15:24 |
| sergiusens | ahayzen, perfect; with a previous search to refresh the desktop files? | 15:24 |
| ahayzen | sergiusens, ah....thts wht the pkill unity8 was doing? | 15:25 |
| sergiusens | ahayzen, so you created /opt/click.ubuntu.com/com.ubuntu.music/1.1.350/lib/arm-linux-gnueabihf but copied nothing into it :-) | 15:25 |
| sergiusens | ahayzen, hmmm; add a -r to the cp ;-) | 15:25 |
| ahayzen | sergiusens, do i need to modify the preinstalled to be 350? cp /usr/share/click/preinstalled/.click/users/\@all/com.ubuntu.music/plugins/* lib/arm-linux-gnueabihf/ | 15:26 |
| ahayzen | sergiusens, or should just the -r work? | 15:26 |
| sergiusens | ahayzen, just the -r | 15:26 |
| ahayzen | sergiusens, magic! | 15:27 |
| ahayzen | sergiusens, it works, thank you so much :D | 15:27 |
| sergiusens | ahayzen, so depending on the ETA for music to migrate; what would you prefer? | 15:27 |
| sergiusens | hmmm, I know what to do as a transparent hack at least | 15:28 |
| ahayzen | sergiusens, i don't mind doing this hack each time (i have a bash script to automate it) but others may | 15:28 |
| ahayzen | sergiusens, do we know wht the eta of the mediascanner/media-hub are because we'll move to it as soon as it is available and has feature parity | 15:29 |
| sergiusens | ahayzen, no idea about media scanner | 15:31 |
| sergiusens | don't even know who works on that | 15:31 |
| sergiusens | popey, do you know? | 15:31 |
| balloons | renato_: so calendar passes fine as well. I approved both the mp's on that basis and linked the original bug to them | 15:31 |
| popey | sergiusens: someone under thostr_ I think | 15:35 |
| popey | fginther: balloons any ETA on weather 197+fun? | 15:35 |
| popey | ooh, i see 197.3 | 15:36 |
| * popey hugs fginther and balloons | 15:36 | |
| popey | balloons: fginther weather 197 published to store | 15:42 |
| sergiusens | popey, would you rather have .$(date %YYYYMMdd) instead of revno? | 15:42 |
| popey | I'd rather have revno_date | 15:43 |
| sergiusens | well, we would need to add min and sec | 15:43 |
| dpm | fginther, ok, now that we've got the fallback safely in the store, can we get a build of trunk? Then we'll test that and we can determine if it's good enough for MWC | 15:43 |
| dpm | fginther, that'd be for weather again ^ | 15:43 |
| fginther | dpm, building | 15:44 |
| dpm | awesome | 15:44 |
| popey | balloons: fginther file manager has the same armhf/all issue that sergiusens fixed for music app with https://code.launchpad.net/~sergiusens/music-app/arch/+merge/207635 | 15:44 |
| popey | should do same for file manager? | 15:44 |
| popey | (and terminal)? | 15:44 |
| sergiusens | yes, I am aware | 15:44 |
| sergiusens | I need to do that once konsole plugin finishes building | 15:45 |
| popey | ok, magic | 15:45 |
| popey | like a well oiled machine | 15:45 |
| ahayzen | So when i take a branch of the music-app in QtCreator it doesn't show any of the qml/js files just the CMakeLists and the folders, anyone know wht is going on? | 16:13 |
| dpm | fginther, can I use this weather click for testing? https://jenkins.qa.ubuntu.com/view/All/job/weather-app-click/120/ - I wasn't sure because the build is marked as red | 16:37 |
| fginther | dpm, let me fix that and retry | 16:38 |
| dpm | ok, cool | 16:41 |
| renato_ | balloons, could you run the test plan for this MR: https://code.launchpad.net/~renatofilho/qtorganizer5-eds/limit-number-of-events/+merge/207542 | 18:21 |
| renato_ | and if works do the top approval? | 18:21 |
| renato_ | I want to release this today | 18:21 |
| balloons | renato_: I can't top approve; but yes I will run through it again | 18:32 |
| renato_ | balloons, just paste the test plan | 18:33 |
| renato_ | I will ask other person to top approval | 18:33 |
| balloons | renato_: pasted | 18:38 |
| renato_ | balloons, thanks | 18:39 |
| fginther | dpm, I hope you already saw this, but the last build of weather-app-lcick was green | 19:39 |
| dpm | fginther, I did, thanks, trying to upload to the store now | 19:39 |
| === _salem` is now known as _salem | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!