Camelia, the Perl 6 bug

IRC log for #mojo, 2012-05-12

| Channels | #mojo index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:50 andrefs joined #mojo
01:47 xaka joined #mojo
02:15 tempire hey sri
02:16 tempire if I come to germany, will you buy the pizza?
02:35 ispy_ joined #mojo
02:49 Patterner is that code for a temporary restraining order?
03:28 perlite joined #mojo
04:08 crab is it supported to use ->to('foo#') or ->to('#foo')?
04:11 crab it works, and people do use it, but i'm not sure i should rely on it
04:29 vel joined #mojo
05:32 Foxcool joined #mojo
05:58 crab sri: let me know if it's not intended to be supported, otherwise i'll change the docs to state that either controller or action in foo#bar is optional
06:00 crab what didn't you know you could do? append .patch to github commit URLs, or something else?
06:04 taryk joined #mojo
06:15 nordicdyno joined #mojo
06:42 Vandal joined #mojo
07:34 Htbaa joined #mojo
07:42 * sri goes to get a temporary restraining order
07:43 sri crab: i believe that's already documented
07:43 spleenjack joined #mojo
07:44 sri search shows like a dozen examples
07:44 * tempire BUYS THE PIZZA
07:44 tempire http://www.youtube.com/watch?v=7mZZd4gQrrg
07:51 nordicdyno joined #mojo
07:51 nordicdyno joined #mojo
08:39 AmeliePoulain joined #mojo
08:49 fhelmber_ joined #mojo
09:03 ryozi joined #mojo
09:09 nordicdyno joined #mojo
09:19 taryk joined #mojo
09:31 nordicdyno joined #mojo
10:36 nordicdyno joined #mojo
10:55 arpadszasz joined #mojo
11:14 taryk joined #mojo
11:31 nordicdyno joined #mojo
11:53 spleenjack1 joined #mojo
11:54 Htbaa1 joined #mojo
12:20 marcus good morning.
12:20 purl For you maybe.
12:20 marcus tempire: Are we meeting in germany?
12:22 andrefs joined #mojo
12:23 * marcus is at a shopping center playground.
12:23 marcus At least they provide wifi...
12:27 taryk joined #mojo
13:01 D4RK-PH0ENiX joined #mojo
13:03 nordicdyno joined #mojo
13:10 nordicdyno joined #mojo
13:10 memowe marcus: are you talking about yapc::eu this year? #germany
13:26 nordicdyno joined #mojo
13:32 estrai_ joined #mojo
13:34 estrai_ hi, I found a bug in Mojo::URL::path but I'm not sure is it a bug in documentation or code, last two examples in documentation return exactly the same thing however the intention is that they will return something different: https://metacpan.org/module/Mojo::URL#path
13:35 estrai_ it bothers me, because it means if I need to add something to the path I always need to be sure that I have the trailing slash which isn't perfect
13:40 nordicdyno joined #mojo
14:01 alnewkirk estrai_: #mojo exists
14:01 alnewkirk oops, wrong channel
14:01 alnewkirk estrai_: intention?
14:01 purl i guess intention is to return you to normal balance when my brain goes left.
14:06 estrai_ alnewkirk: this is what I'm talking about:
14:06 estrai_ % perl -MMojo::URL -E "say Mojo::URL->new('http://mojolicio.us/perldoc')->path('Mojo/DOM');"
14:06 estrai_ http://mojolicio.us/Mojo/DOM
14:06 estrai_ in the docs it says it should be "http://mojolicio.us/perldoc/Mojo/DOM"
14:07 estrai_ and to me it makes more sense as it says in the docs
14:15 alnewkirk estrai_: good catch
14:15 alnewkirk tempire: ^
14:21 sri estrai_: hmm, i'm not sure
14:23 alnewkirk I'd probably make the same assumption
14:27 sri i can follow the resposnible code back two years https://github.com/kraih/mojo/commit/8d​51a8ec3f8e220e391803c86781e6f1f510d0b8
14:28 sri it appears to be intentional
14:31 alnewkirk its hard to say, guess it depends on the authors
14:32 inokenty joined #mojo
14:32 alnewkirk on the one hand, having used URI and common-sense tells me that calling the path() method should change abnd/or return the path part of the URL
14:33 ladnaV joined #mojo
14:33 alnewkirk ... /perldoc is part of the path so I'd kinda expect path($foo) to replace that
14:33 alnewkirk though, the docs make it seem as if it will append without the forward slash
14:34 GitHub75 joined #mojo
14:34 GitHub75 [mojo] kraih pushed 1 new commit to master: http://git.io/-pI9pA
14:34 GitHub75 [mojo/master] fixed path merging examples in Mojo::URL - Sebastian Riedel
14:34 GitHub75 left #mojo
14:35 sri marcus came up with the feature, maybe he knows what the correct semantics are
14:36 sri current ones have been active for over 2 years now though
14:40 sri the example is only 7 months old https://github.com/kraih/mojo/commit/8c​3e61039083a0f61b88dbc584fd328931db32b4
14:41 alnewkirk sri++ looks good
14:52 ok joined #mojo
15:30 plicease joined #mojo
15:37 estrai_ I think it might be worth to consider adding ->path_append, it would make clear interface assuming it would be possible to replace path's functionality with what URI does, and even if it would stay the same still saying ->path_append makes the intention very clear
15:41 sri that seems very inconsistent api design wise
15:44 estrai_ it's already inconsistent, all other methods will replace the value with new one
15:44 sri you have a way better chance convincing me to change the current semantics than to make the api ugly
15:44 sri estrai_: false
15:45 estrai_ sorry, only eyed the docs :)
15:47 sri that's not a good start then
15:53 sri here's also the original feature https://github.com/kraih/mojo/commit/a4​ab22ab2c7b5ae385c898299c4c6a481c3fc643
15:55 xaka joined #mojo
16:08 Properler joined #mojo
16:11 mire joined #mojo
16:29 ispy_ joined #mojo
16:40 vel joined #mojo
17:16 andrefs joined #mojo
18:37 arpadszasz joined #mojo
18:53 zivester joined #mojo
18:58 zivester joined #mojo
19:08 tempire marcus: I don't think so.  sri isn't a very good host.  does norway have pizza?
19:11 * sri hides the book titled "How to cook tempire"
19:12 tempire I'd buy that book
19:22 mire joined #mojo
20:12 zivester joined #mojo
20:38 GitHub85 joined #mojo
20:38 GitHub85 [mojo] kraih pushed 1 new commit to master: http://git.io/dsi9mw
20:38 GitHub85 [mojo/master] added merge method to Mojo::Path - Sebastian Riedel
20:38 GitHub85 left #mojo
20:39 sri less code to do more \o/
20:39 sri but we really need to decide about semantics of merging
20:39 * sri pokes tempire, marcus and crab
20:44 GitHub84 joined #mojo
20:44 GitHub84 [mojo] kraih pushed 1 new commit to master: http://git.io/utLfiA
20:44 GitHub84 [mojo/master] more path merging tests - Sebastian Riedel
20:44 GitHub84 left #mojo
20:45 estrai_ sri++
20:46 xaka joined #mojo
20:46 estrai_ I ended up with:     my $uri_path = Mojo::Path->new( $auth_uri->path );
20:46 estrai_ $uri_path->parts( [ @{$uri_path->parts}, $video_id ] );
20:46 sri please no pasting into the channel
20:46 estrai_ 2 lines, that's usually ok
20:49 sri just saying in case you're nor familiar with irc etiquette
20:49 sri s/r/t/
20:50 taryk joined #mojo
20:57 GitHub147 joined #mojo
20:57 GitHub147 [mojo] kraih pushed 1 new commit to master: http://git.io/gxtWcQ
20:57 GitHub147 [mojo/master] better merge examples - Sebastian Riedel
20:57 GitHub147 left #mojo
21:25 zivester joined #mojo
21:41 GitHub23 joined #mojo
21:41 GitHub23 [mojo] kraih pushed 1 new commit to master: http://git.io/PgiCcA
21:41 GitHub23 [mojo/master] fixed small stringification bug in Mojo::Path - Sebastian Riedel
21:41 GitHub23 left #mojo
21:47 GitHub53 joined #mojo
21:47 GitHub53 [mojo] kraih pushed 1 new commit to master: http://git.io/-wtyVg
21:47 GitHub53 [mojo/master] fixed another small stringification bug in Mojo::Path - Sebastian Riedel
21:47 GitHub53 left #mojo
21:55 zivester joined #mojo
22:12 zivester joined #mojo
22:34 mire joined #mojo
22:49 arpadszasz joined #mojo
23:03 tempire I like the way merge works
23:04 tempire very versatile
23:04 tempire I think it will cause some confusion for people who prefer not to read docs
23:04 tempire but I'm ok with that.
23:05 sri well, it follows url semantics in html
23:05 sri similar to how we handle relative Mojo::URL objects
23:06 sri but i screwed up a little with stringification :S
23:10 andrefs joined #mojo
23:24 GitHub111 joined #mojo
23:24 GitHub111 [mojo] kraih pushed 1 new commit to master: http://git.io/QtGwmQ
23:24 GitHub111 [mojo/master] better stringification tests - Sebastian Riedel
23:24 GitHub111 left #mojo
23:37 zivester joined #mojo

| Channels | #mojo index | Today | | Search | Google Search | Plain-Text | summary