fluid-work IRC Logs-2011-05-12

[09:27:51 CDT(-0500)] <michelled> mlam, harriswong: are you still waiting for the FLUID-3878 pull request to go into the project repo?
[09:28:10 CDT(-0500)] <mlam> michelled: Yes, we are
[09:28:28 CDT(-0500)] <mlam> I think colin was going to look at it and maybe make some small tweaks
[09:28:55 CDT(-0500)] <michelled> great
[09:29:48 CDT(-0500)] <michelled> mlam: do you know if justin is looking at the FLUID-4163 pull request?
[09:32:38 CDT(-0500)] <mlam> colin pushed that in already. that pull request should be closed
[09:34:19 CDT(-0500)] <michelled> could you close it?
[09:36:00 CDT(-0500)] <michelled> cindyli: are you still waiting for FLUID-4132 to be pulled into the project repo?
[09:39:01 CDT(-0500)] <mlam> yes i will, michelled
[09:39:08 CDT(-0500)] <michelled> thanks!
[09:39:09 CDT(-0500)] <cindyli> michelled: Bosmon has pulled FLUID-4207 into project repo yesterday.
[09:39:11 CDT(-0500)] <mlam> np
[09:39:20 CDT(-0500)] <cindyli> it also contains changes for 4171
[09:39:27 CDT(-0500)] <michelled> cindyli: cool
[09:39:30 CDT(-0500)] <cindyli> so it's settled
[09:39:34 CDT(-0500)] <cindyli> np, michelled
[09:39:40 CDT(-0500)] <michelled> do you know if the FCK editor changes got in cindyli?
[09:39:47 CDT(-0500)] <michelled> 4132
[09:39:57 CDT(-0500)] <cindyli> no, as i know
[09:40:14 CDT(-0500)] <cindyli> sorry, i almost forgot about 4132
[09:40:16 CDT(-0500)] <michelled> I was glancing at your pull request and I have a question
[09:40:39 CDT(-0500)] <michelled> why did you change the reg exp on line 67?
[09:40:51 CDT(-0500)] <michelled> https://github.com/cindyli/infusion/commit/11cabcffab0aac472ac18cb4b5c96408d80171e1
[09:41:13 CDT(-0500)] <cindyli> ha, i know there might be questions about this
[09:41:16 CDT(-0500)] <cindyli> one sec
[09:41:44 CDT(-0500)] <michelled> you're probably in the middle of stuff - we can talk about it when you have time
[09:41:47 CDT(-0500)] <michelled> don't worry cindyli
[09:42:08 CDT(-0500)] <cindyli> michelled: jslint threw error on the original regex
[09:42:38 CDT(-0500)] <michelled> cindyli: with something like a regular expression I worry about deferring to jslint
[09:42:55 CDT(-0500)] <michelled> I think we should find out who the original author of the reg exp was and talk to them about it
[09:42:59 CDT(-0500)] <cindyli> because of the usage of [^\>\s] which is not encouraged by jslint
[09:43:24 CDT(-0500)] <michelled> let's mention it at standup and see where we get
[09:43:34 CDT(-0500)] <cindyli> sure
[09:44:14 CDT(-0500)] <michelled> hi pulkit - how's it going today?
[09:45:23 CDT(-0500)] <michelled> cindyli: the UIEnhancer work I'm taking on, is it FLUID-4207?
[09:45:43 CDT(-0500)] <pulkit> Hi michelled Sorry I couldn't look at it today. I was a little busy with my reports.
[09:45:44 CDT(-0500)] <cindyli> yes, michelled
[09:46:01 CDT(-0500)] <cindyli> thanks, michelled
[09:46:04 CDT(-0500)] <michelled> pulkit: np - I was just curious - I don't want you to feel stuck
[09:46:39 CDT(-0500)] <michelled> cindyli: thx - I'm getting started now
[09:46:51 CDT(-0500)] <cindyli> cool thx michelled
[09:47:15 CDT(-0500)] <pulkit> michelled: ok. thanks
[09:47:19 CDT(-0500)] <michelled> np
[10:01:31 CDT(-0500)] <pulkit> michelled: Google has asked for additional information about the project. Do you know what I should enter in External resource URL and Project feed URL?
[10:03:05 CDT(-0500)] <michelled> pulkit: I'm not sure - are these URLs for Infusion or for your particular project?
[10:03:37 CDT(-0500)] <pulkit> These are for Image Editor
[10:05:50 CDT(-0500)] <michelled> pulkit: I think you should create a page in the Fluid wiki for the image editor project
[10:05:54 CDT(-0500)] <michelled> then you can link to that
[10:06:39 CDT(-0500)] <pulkit> ok. thanks
[10:06:46 CDT(-0500)] <michelled> np
[10:25:15 CDT(-0500)] <jhung> jessm, jameswy: meet to talk about IDI website structure after Collection Space standup? Maybe around 1p to 1:30p?
[10:28:46 CDT(-0500)] <jessm> jhung: works for me
[12:27:26 CDT(-0500)] <jhung> jessm: lunch is wrapping up, but I think jameswy just got hijacked into a few meet and greets.
[12:27:47 CDT(-0500)] <jessm> jhung: k, i'll remain flexi
[12:28:02 CDT(-0500)] <jhung> jessm: thanks! lunch took longer than expected.
[12:29:51 CDT(-0500)] <jhung> jessm: erin and I are free now.
[12:36:28 CDT(-0500)] <jessm> jhung: ok
[13:34:40 CDT(-0500)] <cindyli> Bosmon: have a minute to talk about ui options
[14:18:31 CDT(-0500)] <mlam> Bosmon: was the IE6 JSON2.js dependency fixed by using console.debug.apply? I updated my fork with yesterday's commits and a bunch of tests now fail in FF whenever fire bug is active.
[14:28:31 CDT(-0500)] <Bosmon> mlam, it was
[14:28:47 CDT(-0500)] <Bosmon> I tried all the tests in FF before I committed and they seemed to work... which ones are you seeing fail?
[14:28:57 CDT(-0500)] <Bosmon> And which version of FF?
[14:29:24 CDT(-0500)] <mlam> FF 3.6 and FF4 . Uploader-tests.html is one for sure.
[14:29:44 CDT(-0500)] <mlam> I can list others if you'd like....i can quickly run them again
[14:30:25 CDT(-0500)] <mlam> they pass when firebug is disabled, but once it's active, the tests break with this message: console.debug.apply is not a function
[14:31:12 CDT(-0500)] <Bosmon> Ok
[14:31:18 CDT(-0500)] <Bosmon> That test definitely runs for me on FF 3.6
[14:31:22 CDT(-0500)] <Bosmon> So this is an interesting nmystery
[14:32:32 CDT(-0500)] <mlam> I'll grab a fresh copy of the upstream master and give it a run
[14:32:55 CDT(-0500)] <Bosmon> It's true that I don't actually check in that branch whether console.debug is a function before I try to apply it
[14:33:02 CDT(-0500)] <Bosmon> But if it is not a function.... what is it?
[14:33:04 CDT(-0500)] <heidi_> newbie q: that.locate doesn't seem to be working in my finalInit function - does that make sense?
[14:33:30 CDT(-0500)] <Bosmon> heidi_ - no, it should work
[14:33:59 CDT(-0500)] <Bosmon> http://wiki.fluidproject.org/display/docs/Component+Lifecycle+and+autoInit
[14:34:02 CDT(-0500)] <heidi_> Bosmon okay thanks, i must be doing some else wacky
[14:34:03 CDT(-0500)] <Bosmon> You can check out the sequence here
[14:34:12 CDT(-0500)] <heidi_> thanks
[14:34:19 CDT(-0500)] <Bosmon> It should even work in a postInitFunction
[14:34:26 CDT(-0500)] <mlam> Bosmon: Ok, it's not running on a fresh clone of the upstream master
[14:34:47 CDT(-0500)] <mlam> ..when firebug is active, that is.
[14:36:50 CDT(-0500)] <Bosmon> Perhaps it's an issue with the particular version of FireBug you are running?
[14:36:55 CDT(-0500)] <Bosmon> Which version do you have
[14:37:24 CDT(-0500)] <colinclark> Hey Bosmon, thanks so much for fixing my IE6 failure last night
[14:37:44 CDT(-0500)] <colinclark> I'm not quite clear what the actual problem was
[14:37:49 CDT(-0500)] <colinclark> I see the change you made, but what motivated it?
[14:38:10 CDT(-0500)] <Bosmon> The issue was that the test was not properly mocking the contextual decision tag for the Uploader's environment...
[14:38:12 CDT(-0500)] <mlam> Bosmon: 1.8.02a
[14:38:18 CDT(-0500)] <Bosmon> It was creating the "real" progressiveEnhancerForComponent
[14:38:29 CDT(-0500)] <Bosmon> Which means it was reponding to the actual browser environment, rather than a mock for it
[14:38:42 CDT(-0500)] <Bosmon> So, on my IE8 which has no Flash installed, the test was guided to instantiate the singleFileUploader
[14:38:51 CDT(-0500)] <Bosmon> Which failed to have the event required by the configuration, onFileSuccess
[14:39:00 CDT(-0500)] <mlam> Bosmon: but on my XP VM, i'm running 1.6.2 on FF3.6
[14:39:17 CDT(-0500)] <colinclark> ah, of course
[14:39:26 CDT(-0500)] <colinclark> Thanks Bosmon, I appreciate that
[14:39:32 CDT(-0500)] <Bosmon> np
[14:39:40 CDT(-0500)] <colinclark> mlam: Lemme try on my Firefoxes
[14:39:42 CDT(-0500)] <Bosmon> So, do you want to try out mlam's issue too?
[14:39:44 CDT(-0500)] <Bosmon> Thanks
[14:40:00 CDT(-0500)] <mlam> ok, cool
[14:41:05 CDT(-0500)] <Bosmon> So, mlam... in your FireBug.... what is console.debug?
[14:41:15 CDT(-0500)] <Bosmon> Can you inspect it?
[14:41:56 CDT(-0500)] <colinclark> I have Firefox 4.0.1 on Mac OS X with Firebug 17.1b3 installed
[14:41:56 CDT(-0500)] <colinclark> A
[14:42:01 CDT(-0500)] <colinclark> All the Uploader tests are passing
[14:42:06 CDT(-0500)] <Bosmon> For me, it is very clearly a function
[14:42:31 CDT(-0500)] <Bosmon> I mean, I can add a check to see whether it is a function... but I wasn't aware of the possibility it could be anything else
[14:42:39 CDT(-0500)] <Bosmon> Does it actually operate when you try to call it?
[14:42:56 CDT(-0500)] <Bosmon> I have Firebug 1.6.2
[14:43:08 CDT(-0500)] <Bosmon> Which I guess is the same one as in your XP VM
[14:44:11 CDT(-0500)] <colinclark> mlam: You're seeing the error when the Firebug console is open only, is that right?
[14:44:26 CDT(-0500)] <mlam> yes thats right colinclark
[14:45:07 CDT(-0500)] <colinclark> Hilariously, the tests also pass in Firefox 2 with Firebug 1.05
[14:45:24 CDT(-0500)] <colinclark> I think I won't try my luck with Firefox 1.5
[14:45:26 CDT(-0500)] <mlam> Bosmon: No, I'm trying to inspect , but I can't
[14:45:38 CDT(-0500)] <Bosmon> You "can't"? :!
[14:45:43 CDT(-0500)] <Bosmon> What do you mean?
[14:47:06 CDT(-0500)] <colinclark> Happiness in Firefox 3.6 with Firebug 1.6.2 on Mac OS X
[14:47:11 CDT(-0500)] <colinclark> Let me try Windows XP now
[14:47:13 CDT(-0500)] <mlam> there we go...ok...ya, console.debug is clearly a function for me too.
[14:47:32 CDT(-0500)] <Bosmon> It is a function?
[14:47:36 CDT(-0500)] <Bosmon> So how does it fail to apply?
[14:48:31 CDT(-0500)] <mlam> Yeah, console.debug IS a function for me....but in the watch panel of firebug, it says the apply is undefined.
[14:48:39 CDT(-0500)] <Bosmon> .....
[14:49:31 CDT(-0500)] <Bosmon> I am wondering whether you have some strange kind of extension installed
[14:49:42 CDT(-0500)] <Bosmon> But it would be an odd coincidence to have it both outside and inside your VM
[14:49:52 CDT(-0500)] <Bosmon> You are seeing the failure in two different machines, right?
[14:50:11 CDT(-0500)] <harriswong> mlam: should I try on my machine as well?
[14:50:18 CDT(-0500)] <mlam> harriswong: yes please
[14:50:28 CDT(-0500)] <colinclark_> Happiness in Firefox 3.0 with Firebug 1.1.0b12...
[14:50:28 CDT(-0500)] <mlam> Bosmon: Yah, in ubuntu and on my XP VM
[14:50:30 CDT(-0500)] <colinclark_> the bad old days
[14:50:32 CDT(-0500)] <Bosmon> I gest these results
[14:50:34 CDT(-0500)] <Bosmon> >>> console.debug.apply
[14:50:35 CDT(-0500)] <Bosmon> apply()
[14:50:35 CDT(-0500)] <Bosmon> >>> typeof(console.debug)
[14:50:35 CDT(-0500)] <Bosmon> "function"
[14:50:35 CDT(-0500)] <Bosmon> >>> typeof(console.debug.apply)
[14:50:35 CDT(-0500)] <Bosmon> "function"
[14:50:36 CDT(-0500)] <harriswong> mlam: can you summary what exactly i have to do please?
[14:50:44 CDT(-0500)] <harriswong> summarize*
[14:50:56 CDT(-0500)] <mlam> just run the Uploader-tests.html file in FF with firebug running
[14:51:02 CDT(-0500)] <harriswong> mlam: will do.
[14:53:13 CDT(-0500)] <harriswong> mlam: i am assuming on master?
[14:53:19 CDT(-0500)] <mlam> yah
[14:56:53 CDT(-0500)] <Bosmon> mlam^
[14:57:06 CDT(-0500)] <Bosmon> Can you paste the results of those in your two machines?
[14:57:14 CDT(-0500)] <mlam> yup
[14:57:23 CDT(-0500)] <mlam> typeof(console.debug.apply)"undefined"typeof(console.debug)"function"console.debug.applyundefinedconsole.debugdebug()
[14:57:26 CDT(-0500)] <mlam> ugh.
[14:57:36 CDT(-0500)] <Bosmon> That's insane
[14:57:52 CDT(-0500)] <colinclark_> mlam: Remind us again the exact version of Firefox and Firebug you're running?
[14:57:57 CDT(-0500)] <Bosmon> Insanely broken
[14:58:21 CDT(-0500)] <mlam> FF4, firebug 1.8.0a2
[14:58:26 CDT(-0500)] <colinclark_> And if you're running multiple versions of Firefox, are you using separate profiles?
[14:58:30 CDT(-0500)] <harriswong> mlam: on FF4 across win7, osx, seems okay.
[14:58:31 CDT(-0500)] <colinclark_> That's a very new version of Firebug
[14:58:55 CDT(-0500)] <Bosmon> But it fails in your 1.6.2 as well, right, mlam?
[14:59:05 CDT(-0500)] <mlam> I'm only running FF4 on ubuntu. FF 3.6 is on my XP VM
[14:59:05 CDT(-0500)] <Bosmon> Can you paste the results from that one too?
[14:59:39 CDT(-0500)] <mlam> I restarted my XP VM and haven't been able to replicate since...
[14:59:40 CDT(-0500)] <colinclark_> Okay, interesting
[14:59:48 CDT(-0500)] <Bosmon> !!!
[14:59:52 CDT(-0500)] <colinclark_> So you've got FF4 with Firebug 1.8.0a2 on Ubuntu
[15:00:01 CDT(-0500)] <mlam> yes
[15:00:03 CDT(-0500)] <colinclark_> and Firefox 3.6 with which version of Firebug on XP?
[15:00:09 CDT(-0500)] <mlam> 1.6.2
[15:00:14 CDT(-0500)] <colinclark_> Okay
[15:00:18 CDT(-0500)] <colinclark_> That's a stable combination
[15:00:22 CDT(-0500)] <colinclark_> and verified to work elsewhere
[15:00:25 CDT(-0500)] <Bosmon> This latter is the same one I have
[15:00:33 CDT(-0500)] <colinclark_> and now it's disappeared for you?
[15:00:34 CDT(-0500)] <Bosmon> mlam, on your problematic configuration, can you try this:
[15:00:40 CDT(-0500)] <Bosmon> http://stackoverflow.com/questions/5538972/console-log-apply-not-working-in-ie9
[15:00:51 CDT(-0500)] <colinclark_> Are you seeing the same failures in FF4/FB1.8.1a2/Ubuntu?
[15:00:54 CDT(-0500)] <Bosmon> In particular try these lines:
[15:00:54 CDT(-0500)] <colinclark_> mlam: ^
[15:00:55 CDT(-0500)] <Bosmon> var log = Function.prototype.bind.call(console.log, console);
[15:00:55 CDT(-0500)] <Bosmon> log.apply(console, ["this", "is", "a", "test"]);
[15:01:15 CDT(-0500)] <mlam> colinclark_: I can try 1.8.1a2
[15:01:29 CDT(-0500)] <colinclark_> sorry
[15:01:33 CDT(-0500)] <colinclark_> I meant 1.8.0a2
[15:01:37 CDT(-0500)] <colinclark_> whatever you've already got installed
[15:01:38 CDT(-0500)] <colinclark_> sorry
[15:01:46 CDT(-0500)] <colinclark_> too many numbers and letters
[15:02:03 CDT(-0500)] <colinclark_> Thank goodness we never actually put out Infusion 1.2.1.1
[15:02:07 CDT(-0500)] <colinclark_>
[15:02:42 CDT(-0500)] <mlam> colinclark_: yes...that's the combo I'm having the issue on.
[15:02:48 CDT(-0500)] <colinclark_> ok
[15:03:08 CDT(-0500)] <colinclark_> Are you on Firefox 4 or 4.01?
[15:04:09 CDT(-0500)] <mlam> 4.0.1
[15:04:30 CDT(-0500)] <colinclark_> I'd suggest going backwards to 1.7.1b3 if you can
[15:04:38 CDT(-0500)] <colinclark_> since you're running an alpha version intended for Firefox 5
[15:06:16 CDT(-0500)] <mlam> haha Bosmon: this is the output from the 2 lines you gave me : this is a test
[15:06:43 CDT(-0500)] <colinclark_> so clearly console.log is an appliable function
[15:09:20 CDT(-0500)] <mlam> Bosmon: colinclark_ : reverting back to 1.7.1b3 made everything work again
[15:09:28 CDT(-0500)] <colinclark_> wicked
[15:11:02 CDT(-0500)] <mlam> soo, what does this mean for the firebug 1.8.0x users?
[15:15:26 CDT(-0500)] <Bosmon> It means we need to apply those 2 lines of code
[15:15:31 CDT(-0500)] <Bosmon> Since clearly they work for you
[15:15:38 CDT(-0500)] <Bosmon> They worked in 1.8, right?
[15:17:12 CDT(-0500)] <Bosmon> We should report it as a bug also, clearly...
[15:18:04 CDT(-0500)] <Bosmon> mlam - can you double-check that the "bind" lines run in Firebug 1.8.0a2?
[15:18:30 CDT(-0500)] <mlam> Yah, I ran those commands with 1.8
[15:18:41 CDT(-0500)] <mlam> ok
[15:19:12 CDT(-0500)] <Bosmon> cindyli - are you there?
[15:19:18 CDT(-0500)] <Bosmon> I mean to talk to you about file permissions
[15:19:26 CDT(-0500)] <Bosmon> It looks like a few more files have got into master with 755 permissions today
[15:19:30 CDT(-0500)] <cindyli> ha Bosmon
[15:19:36 CDT(-0500)] <cindyli> great to hear from u
[15:19:52 CDT(-0500)] <cindyli> thanks for correcting them, Bosmon
[15:20:27 CDT(-0500)] <Bosmon> Looks like it is from justin's merge this time
[15:20:29 CDT(-0500)] <cindyli> not sure how 755 sneaks in. probably because i'm using windows?
[15:20:36 CDT(-0500)] <Bosmon> But I suspect that neither of you are the direct source of them
[15:20:46 CDT(-0500)] <Bosmon> Well, I am using Windows too
[15:20:55 CDT(-0500)] <cindyli> ha. shake hands ..
[15:20:58 CDT(-0500)] <Bosmon> Justin sent round a guide on how to avoid the problem when we first moved to git
[15:21:05 CDT(-0500)] <Bosmon> I wonder if I can dig it out
[15:21:14 CDT(-0500)] <Bosmon> There is a git setting you need to apply
[15:21:16 CDT(-0500)] <cindyli> sure. i guess the first pdf he sent out
[15:21:31 CDT(-0500)] <Bosmon> But the best method is to use Cygwin which lets you see and manipulate the file permissions
[15:21:40 CDT(-0500)] <cindyli> i am using cygwin
[15:21:46 CDT(-0500)] <mlam> Bosmon: Yup, the bind works in 1.8.0a2
[15:21:51 CDT(-0500)] <cindyli> but never pay attention to the file permissions
[15:22:00 CDT(-0500)] <Bosmon> mlam, ok, I will push an update in a sec
[15:22:10 CDT(-0500)] <mlam> ok, cool
[15:22:21 CDT(-0500)] <Bosmon> cindyli - to clear the permissions, you should run the following command after you have added new files to your working copy:
[15:22:49 CDT(-0500)] <Bosmon> find . -type f -exec chmod 644 {} \;
[15:23:02 CDT(-0500)] <Bosmon> That is, before you commit them
[15:23:18 CDT(-0500)] <cindyli> ah, yes, modify all to 644
[15:23:26 CDT(-0500)] <cindyli> thanks
[15:24:07 CDT(-0500)] <Bosmon> selecting only file types is important - if you hit directories with 644, the results are unpleasant
[15:24:24 CDT(-0500)] <cindyli> ic. thx for reminding