Created
October 31, 2012 14:35
-
-
Save bengourley/3987379 to your computer and use it in GitHub Desktop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[1:57pm] AvianFlu: | |
bengourley: from the looks of it, st is just very buggy | |
[1:59pm] redir_ joined the chat room. | |
[2:03pm] jetienne left the chat room. (Quit: jetienne) | |
[2:03pm] bradleymeck joined the chat room. | |
[2:03pm] bradleymeck was granted voice by kohai. | |
[2:06pm] sreeix left the chat room. (Quit: sreeix) | |
[2:07pm] TheJH left the chat room. (Ping timeout: 268 seconds) | |
[2:10pm] Sly: | |
Everyone doing okay in here? Need any help? | |
[2:14pm] bengourley: | |
AvianFlu: how do you mean? What bugs does it have? | |
[2:15pm] lpin joined the chat room. | |
[2:15pm] AvianFlu: | |
bengourley: what you're seeing seem to be bugs in the aggressive caching it does | |
[2:15pm] AvianFlu: | |
I haven't had time to dig through the code | |
[2:15pm] AvianFlu: | |
but the latest flatiron release is broken because of an attempted switch to st | |
[2:15pm] AvianFlu: | |
which will likely be reverted later today | |
[2:15pm] AvianFlu: | |
luckily it doesn't seem to be a libuv problem | |
[2:16pm] bengourley: | |
Oh ok. Are there failing tests for flatiron that highlight the bugs then/ | |
[2:17pm] AvianFlu: | |
Sly: ^^ | |
[2:17pm] Sly: | |
bengourley, what kind of bug are you looking for? The tests try to cover every aspect of flatiron usage, but it's not guaranteed that we have your usage case covered. | |
[2:18pm] AvianFlu: | |
Sly: he's been seeing st serve the wrong files intermittently | |
[2:18pm] Sly: | |
Ah. | |
[2:18pm] AvianFlu: | |
like, something.js when stuff.css was requested | |
[2:18pm] AvianFlu: | |
that kind of thing | |
[2:19pm] Sly: | |
I don't think we have a test case for that, because I don't believe ecstatic had that problem. | |
[2:19pm] chrisdotcode joined the chat room. | |
[2:19pm] bengourley: | |
So how did you decided that st broke flatiron? | |
[2:20pm] AvianFlu: | |
because production apps stopped working | |
[2:20pm] Sly: | |
bengourley, most likely because it breaks other tests in the flatiron test suite. | |
[2:20pm] AvianFlu: | |
Sly: that was what I was asking you to tell him about lol | |
[2:20pm] Sly: | |
Oh. So he wants to test it himself? | |
[2:20pm] joshsmith joined the chat room. | |
[2:20pm] joshsmith was granted voice by ChanServ. | |
[2:20pm] bengourley: | |
Oh dear! | |
[2:20pm] Sly: | |
Should be able to run `npm test flatiron` if you want to run the tests yourself, bengourley. | |
[2:21pm] bengourley: | |
It would be good to have something tangible to go back to Isaacs with for this: | |
[2:21pm] bengourley: | |
https://github.com/isaacs/st/issues/9 | |
[2:21pm] bengourley: | |
I have struggled to produce a reduced test case, or even reproduce the bug sometimes | |
[2:22pm] AvianFlu: | |
bengourley: it may only happen on sunOS-derivatives | |
[2:26pm] bengourley: | |
Which apps are breaking in production, how and who owns them? | |
[2:26pm] AvianFlu: | |
bengourley: our apps | |
[2:26pm] AvianFlu: | |
we could not use the latest version of flatiron with st | |
[2:26pm] AvianFlu: | |
we had to revert to the older versions with ecstatic | |
[2:27pm] AvianFlu: | |
bengourley: after seeing isaacs' response, I think I can get him to address this | |
[2:27pm] dylang joined the chat room. | |
[2:27pm] bengourley: | |
Ok, cool. Will you post on that issue? | |
[2:28pm] AvianFlu: | |
yes | |
[2:29pm] bengourley: | |
Great, thanks | |
[2:30pm] sirkitree joined the chat room. | |
[2:35pm] coderarity left the chat room. (Ping timeout: 268 seconds) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment