=== Makyo is now known as Makyo|out | ||
bcsaller | frankban: can you test that phantomjs branch again after install phantomjs? :) npm install -g phantomjs Want to get another verification that it works | 13:31 |
---|---|---|
frankban | bcsaller: sure | 13:33 |
frankban | bcsaller: i wonder why phantomjs is not a dependency of mocha-phantomjs | 13:34 |
bcsaller | frankban: I would think so as well, but their might be an issue with phantom also wanting a global install | 13:34 |
frankban | bcsaller: 326 tests complete (6 seconds) 1 test pending, very cool! | 13:34 |
bcsaller | sweet :) | 13:35 |
bcsaller | hey wait! your computer is faster than mine! | 13:35 |
bcsaller | no fair | 13:35 |
frankban | :-) | 13:35 |
gary_poster | hey bcsaller looks like you merged the phantomjs without the review changes | 14:32 |
bcsaller | gary_poster: they should be in there | 14:33 |
bcsaller | docs, removed console comments, what did I miss? (and sorry) | 14:33 |
gary_poster | bcsaller, ah weird. No, it's cool. Your branch (https://code.launchpad.net/~bcsaller/juju-gui/phantom-testing) does not have your changes (verified locally and in LP) but trunk does. I guess that is some lbox magic or something. <shrug> But yes it's in trunk, thanks and sorry for false alarm | 14:37 |
bcsaller | gary_poster: yeah, I didn't repropose/push, just did the submit after the changes | 14:37 |
gary_poster | interesting that lbox works that way. Thanks for the branch bcsaller, really awesome to have that. | 14:37 |
bcsaller | gary_poster: really happy to have this branch in :) | 14:38 |
gary_poster | definitely :-) | 14:38 |
=== Makyo|out is now known as Makyo | ||
teknico | bac, bcsaller, benji, frankban, Makyo, daily call in 2 minutes | 15:28 |
Makyo | Downside to the phantomjs thing: leaving make test-server running while you try to submit causes everything to blow up. | 16:21 |
bcsaller | we could change the port? | 16:25 |
Makyo | Not a big deal, just forgot I had the server running :) | 16:25 |
benji | Makyo: no, over here! what are you doing over there? | 16:56 |
Makyo | Bouncing between windows, apparently :) I'll review after a restart, because apparently a package update requires that. | 16:57 |
benji | actually, a restart sounds good; maybe it will make my camera work again; back in a minute | 16:58 |
benji___ | Why does Ubuntu hate me? Almost every time I reboot I have to rebind my workspace navigation keys. | 17:02 |
bcsaller | benji___: I get that too with things like toggle max state on windows, drives me crazy | 17:04 |
teknico | bcsaller, it happened to me too, so I stopped redbinding them and learnt to love the default bindings :-) | 17:07 |
bcsaller | teknico: I never saw you as a conformist | 17:07 |
teknico | bcsaller, it came as a total shock to me too, it must be the age :-) | 17:08 |
bcsaller | ha | 17:08 |
=== benji___ is now known as benji | ||
benji | bcsaller: yep, I use a custom maximize toggle too, wierd thing is that one will get reset and not the other | 17:15 |
frankban | charm accepted! \o/ | 17:17 |
teknico | frankban, yes, it was announced on #juju about half an hour ago :-) | 17:25 |
frankban | :-) cool | 17:26 |
benji | <sigh> Attempting to install phantomjs generates an error, therefore I can't land my branch. | 17:52 |
benji | does this make sense to anyone? | 17:56 |
benji | http://paste.ubuntu.com/1566725/ | 17:56 |
benji | guihelp: ^^ | 17:56 |
hazmat | benji, looks like a download error, for the binary from google code, i'd retry | 18:03 |
benji | hazmat: I figured the same thing. I retried, no luck. | 18:03 |
hazmat | benji, your on 32 bit? | 18:04 |
benji | yep | 18:04 |
benji | hmm, it looks like the file it is trying to download 404s (http://phantomjs.googlecode.com/files/phantomjs-1.8.0-linux-i686.tar.bz2) | 18:09 |
hazmat | that might be the delta, its possible the link is bad | 18:09 |
benji | 1.7.0 exists; I will try requesting that version | 18:09 |
hazmat | benji, 1.8.1 is also present http://code.google.com/p/phantomjs/downloads/list | 18:10 |
benji | mm | 18:10 |
Makyo | Is https://bugs.launchpad.net/juju-gui/+bug/1087110 no longer a thing? | 18:10 |
_mup_ | Bug #1087110: makefile and tests are not run automatically before landing <juju-gui:Triaged> < https://launchpad.net/bugs/1087110 > | 18:10 |
benji | hmm, I wonder why it is trying 1.8.0 | 18:10 |
Makyo | benji, there was an update to npm today, for me. Maybe try that? | 18:11 |
Makyo | s/try/check | 18:11 |
benji | Makyo: nope (I have updated recently) | 18:14 |
benji | hmm, different error from "sudo npm install -g phantomjs@1.8.1" | 18:16 |
benji | guihelp: I haven't been able to get npm to install phantomjs; I have two cards gumming up the lane, therefore I am going to subvert lbox and just land the branches in question. You have five (5) minutes to object. | 18:36 |
benji | Time's Up! | 18:42 |
bcsaller | benji: can you manually install http://phantomjs.googlecode.com/files/phantomjs-1.8.1-linux-i686.tar.bz2 | 18:42 |
bcsaller | opps | 18:42 |
bcsaller | too slow | 18:42 |
benji | bcsaller: :) | 18:42 |
benji | bcsaller: I tried (in that I tried pointing npm at that URL as well as wget-ting the file and pointing npm at the file on disk; all failed) | 18:43 |
bcsaller | you're sure it was 1.8.1? | 18:43 |
benji | I also tried "npm install -i phantomjs@1.81" | 18:43 |
bcsaller | and not 1.8.0? that was the link I saw before | 18:43 |
benji | yep, I tried several versions | 18:44 |
bcsaller | but have you tried it w/o npm? | 18:44 |
benji | what does "w/o npm" mean? | 18:44 |
bcsaller | you can download the package directly and see about putting it in the right place | 18:46 |
bcsaller | or at least check if the binary will run from the archive for you | 18:46 |
benji | that's worth trying, I'll see about that | 18:47 |
bcsaller | for me it looks like a stand alone binary, you might have to check its library deps with `ldd` and make sure you're not missing anything as you're skipping normal packaging | 18:49 |
benji | bcsaller: how about just using apt to install it? | 18:50 |
bcsaller | haven't tried, my guess is its too old but you can try that as well | 18:51 |
benji | untarring it into a temp directory gives me a working (apparently) phantomjs; now, I wonder where I would put it | 18:51 |
bcsaller | somewhere in your pat | 18:51 |
bcsaller | path | 18:52 |
bcsaller | it just spawns that with some args | 18:52 |
benji | that confuses me, isn't there some sort of npm packagy, javascripty thing supposed to be going on... | 18:53 |
benji | bcsaller: I was supposed to run this command, right? "sudo npm install -g phantomjs" | 18:53 |
bcsaller | npm is being subverted to serve a binary in this case (and not well) | 18:53 |
benji | hmm | 18:54 |
bcsaller | phantom is basically headless webkit | 18:54 |
bcsaller | or I thought it was, might be something else under the hood | 18:54 |
benji | it is (basically headless webkit) | 18:56 |
benji | ok, I have phantomjs installed; "sudo npm install -g mocha-phantomjs" appeared to work | 18:56 |
benji | running lbox now | 18:56 |
benji | eating avocado... mmm | 18:57 |
benji | yay! the tests failed! | 18:58 |
benji | bcsaller: yay, it lives! Double yay: it cought a test failure that I would have introduced to trunk! | 19:03 |
bcsaller | benji: excellent, sorry about the install trouble | 19:03 |
benji | bcsaller: I will add a note to HACKING that if you have trouble installing phantomjs you can do it manually and outline that; many thanks for the help! | 19:04 |
benji | oh man, I love running the tests in a terminal so much | 19:12 |
bcsaller | yeah, me too | 19:12 |
gary_poster | bcsaller Makyo benji bac, hi. Neither the menu's "view" choice work nor the double-click work to go from the environment to a service. I'll file a bug in 30 minutes or so if no-one else has, but if someone is available to file that would be great. If someone is available to fix, that would be great. | 20:24 |
Makyo | gary_poster, on it, will start with the bug. | 20:24 |
benji | gary_poster: I can file it | 20:24 |
benji | or Makyo can | 20:24 |
gary_poster | Thank you Makyo. Thanks benji :-) | 20:24 |
gary_poster | Makyo, looks like click on a unit no longer works on http://uistage.jujucharms.com:8080/service/wordpress/ either | 20:25 |
bcsaller | gary_poster: a fix for that was applied a day or so ago, maybe someone merged it out | 20:26 |
gary_poster | ok thanks bcsaller, maybe check with Makyo if there is a quick fix | 20:26 |
bcsaller | gary_poster: it was a trivial, found it, I'll just merge the fix, its putting old code back :-/ | 20:37 |
Makyo | bcsaller, where was it? | 20:38 |
bcsaller | Makyo: app.js/handler for *:navigateTo | 20:38 |
Makyo | bcsaller, ah, alright | 20:39 |
bcsaller | I think all I saw was the console.log and removed the handler, read the second line as arguments | 20:40 |
bcsaller | *shameface* | 20:40 |
Makyo | On looking at https://launchpad.net/juju-gui should we update the charm store url? | 20:40 |
Makyo | It's still pointing to the ~juju-gui charm. | 20:41 |
gary_poster | bcsaller, ok thank you. Makyo yes please I think you can | 20:41 |
Makyo | gary_poster, Done and done. | 20:42 |
gary_poster | thank you Makyo! | 20:43 |
benji | protip: if you want to run a subset of the tests in a terminal you can just start the test server (and close the annoying test window it opens) and then run a command like this: mocha-phantomjs "http://localhost:8084/test/index.html?grep=viewport module" | 21:32 |
benji | Since mocha-phantomjs supports reading the test file from the filesystem it would be a nice enhancement to get "mocha-phantomjs test/test_index.html" (or similar) to work | 21:33 |
benji | ...then we wouldn't need a test server to run the tests in a terminal | 21:33 |
bcsaller | benji: or just add .only to the describe or it for the test | 21:37 |
bcsaller | we can change the test-server script to pass through options, grep is a cli option as well, you don't need to modify the url directly | 21:38 |
benji | bcsaller: I didn't see grep in the help for mocha-phantomjs and --grep didn't work (although it didn't complain about it either, which is odd) | 21:42 |
benji | the .only trick is a good one; everyone should know that one, maybe you should mention it in the weekly meeting | 21:43 |
bcsaller | benji: hmm, only mocha takes --grep directly, sorry about that | 21:44 |
* Makyo dogwalks. | 21:56 | |
* Makyo dedogwalks. | 22:23 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!