The web in a box - a next generation web framework for the Perl programming language

IRC log for #mojo, 2017-04-22

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

All times shown according to UTC.

Time Nick Message
00:09 skaji Hi,
00:09 skaji I noticed that Mojolicious with old perl (such as 5.10.1) emitted 'call method "remove" on an undefined value ... during global destruction' or 'Use of uninitialized value $loop ... during global destruction'.
00:10 skaji You can see these in travis ci log too.
00:10 skaji https://travis-ci.org/kraih/mojo/jobs/221403560
00:12 Grinnz skaji: it's known that 5.10 has those warnings due to ${^GLOBAL_PHASE} not existing. they don't tend to harm anything
00:14 Grinnz see http://mojolicious.org/perldoc/Mojolicious/Guides/FAQ#Which-versions-of-Perl-are-supported-by-Mojolicious -- only 5.22 and 5.24 are fully supported
00:15 skaji Grinnz: I see, thanks.
00:47 Lee joined #mojo
01:20 aborazmeh joined #mojo
02:06 noganex_ joined #mojo
04:04 dboehmer_ joined #mojo
06:33 mpapec joined #mojo
06:36 Janos joined #mojo
07:14 dod joined #mojo
07:20 dod joined #mojo
07:37 Vandal joined #mojo
08:16 dod joined #mojo
10:45 punter joined #mojo
11:55 Bloke joined #mojo
12:03 Bloke Has anyone complained about newly generated apps not finding their config files?
12:03 Bloke Mojolicious version 9 required--this is only version 7.30.
12:04 pink_mist uhm, there is no version 9 of Mojolicious
12:04 Bloke That's what /she/ said.
12:05 Bloke It just says 9 'cause i did one of these: perl -MMojolicious\ 9
12:05 Bloke to get it to print out the version
12:06 pink_mist next time just do: mojo version
12:06 Bloke lol, ok
12:10 Bloke https://pastebin.com/C3HgPrKf
12:11 Bloke One's got a - and one's got an _
12:11 pink_mist yeah, looks like that's some form of a bug
12:12 pink_mist perhaps you could open an issue on github
12:12 Bloke from the directory name it creates and from having seen app-mojopaste on the tubes, i wonder if the _ is correct in the directory name too
12:14 nic sri: In the POD for Mojo::DOM (line 443) it says XML detection "can also be disabled" but the immediately following example shows it being enabled
12:14 pink_mist I think the _ is right - at least that's how my test on Mojolicious 6.65 did it
12:14 nic I'm thining it might be clearer if the text said 'can also be enabled/disabled'
12:14 nic s/thining/thinking/
12:15 Bloke pink_mist: I guess the one with the - is the moniker, is there any guidance as to which one should be used here?
12:16 Bloke Also, I saw that it now generates a secret, which is pretty great.
12:18 pink_mist Bloke: actually, scratch my comment - the .conf file should be with - even back in 6.65 as well
12:19 pink_mist (it just doesn't make one by default)
12:20 Bloke so... https://github.com/kraih/mojo/commit/073baf191a4dc8e6a8a7aa89d1d2a1a88eececb5 then?
12:22 pink_mist yeah, the test in that commit just tests if my_app.conf exists - it forgets to test if the Config plugin actually *loads* it :/
12:22 Bloke Lol, computer programs.
12:25 pink_mist nic: that disables XML *DETECTION*, it forces XML semantics to be on
12:26 good_news_everyon joined #mojo
12:26 good_news_everyon [mojo] kraih pushed 1 new commit to master: https://git.io/v9Tvw
12:26 good_news_everyon mojo/master fabe4d3 Sebastian Riedel: better description for forced semantics
12:26 good_news_everyon left #mojo
12:29 sri Bloke: yes, a bug
12:33 good_news_everyon joined #mojo
12:33 good_news_everyon [mojo] kraih pushed 1 new commit to master: https://git.io/v9Tv5
12:33 good_news_everyon mojo/master 56d4d89 Sebastian Riedel: fix a bug in the app generator command where the config file name was not based on the application moniker
12:33 good_news_everyon left #mojo
12:33 pink_mist Bloke++ sri++
12:34 sri jberger: i'm interpreting "jberger has learned a lot about the router today :-P" as there not being a bug :)
12:35 sri btw. re config plugin, every now and then i want it to have a deparate dir option where you just set the directory for the config file
12:35 sri s/deparate/separate/
12:35 sri dir => '/etc'
12:35 sri instead of requiring the absolute path
12:36 pink_mist isn't that https://github.com/kraih/mojo/issues/1075 ?
12:37 sri yes it actually is
12:38 sri i seem to have missed the issue
12:38 sri oh well, if everyone is against it :S
12:43 pink_mist sri: doesn't this test need to be fixed as well? https://github.com/kraih/mojo/blob/master/t/mojolicious/commands.t#L255
12:43 sri no
12:43 sri the config file is still the same in that case
12:43 Bloke It seems like ->home and some platform specific ->moniker+->mode thing would be a sensible default
12:44 pink_mist oh
12:44 Bloke idk if the config plugin already has search path type logic
12:45 sri not yet
12:45 sri that was one of the thoughts behind adding the dir option
12:46 sri dir => ['/etc', app->home->child('etc')]
12:46 Bloke is that in addtion to the ->mode though?
12:46 sri yes
12:46 sri mode handling is separate
12:47 sri i'm not going to work on it though, since the issue showed low interest in the feature
12:50 khfeng joined #mojo
13:00 jberger Correct, the only bug was in muy understanding of conditions
13:01 jberger s/muy/my
13:01 jberger I almost have a real use case for them at $work
13:02 jberger As part of an internal slack bot
13:02 sri i still like the query param checking condition i made for $work http://mojolicious.org/perldoc/Mojolicious/Guides/Routing#Adding-conditions
13:03 jberger You only get 1 (well 2 kinda) webhooks you register
13:03 jberger Then I'm thinking of continuing to route based on the payload
13:04 jberger sri yeah that's neat too
13:04 jberger I liked it even more this time on rereading
13:11 jberger Wow http://news.perlfoundation.org/2017/04/final-grant-report-migrating-b.html
13:12 jberger That was going to be a coup for the Dancer camp
13:13 jberger And with the ACT replacement stumbling too (I don't know what framework that was but I don't think it was mojo)
13:16 sri looks like it died because of Perl6
13:16 sri https://github.com/Perl-Evozon/PearlBee/issues/75#issuecomment-269045241
13:20 sri let this be a warning to you, do not get hung up on Perl6, it's not a sensible investment
13:20 sri not at this time
13:20 sri i'd rather learn more Python than Perl6
13:33 tchaves joined #mojo
13:38 Bloke Are there any writable Config backends?
13:39 Bloke something that I can update from form posts without it being a whole postgres
13:47 Bloke maybe Mojo::SQLite
15:08 PryMar56 joined #mojo
15:11 sh14 joined #mojo
15:19 suede joined #mojo
15:33 jberger Bloke you probably don't want that possible for production
15:33 jberger For Galileo I have a configuration app that you run after install
15:34 jberger It builds a hash and writes it with dumper
15:34 jberger But that's only available during that process and not generally
15:46 MikePM joined #mojo
16:31 Janos joined #mojo
16:53 dod joined #mojo
17:12 kaare joined #mojo
17:17 stryx` joined #mojo
17:21 Janos joined #mojo
17:27 kes joined #mojo
17:28 kes Hi
17:28 kes Is it will be more useful if for request:
17:28 kes /user/5?_method=DELETE
17:29 kes the logged message will be
17:29 kes DELETE "/user5"
17:29 kes instead of
17:29 kes GET "/user/5"
17:31 kes this will allow to grep requests in log file more precisely
17:31 kes cat app.log | grep DELETE
17:34 pink_mist uhm, no
17:42 sh14 joined #mojo
17:52 Bloke is _method a thing?
17:53 Grinnz yes, some form override iirc
17:54 Grinnz https://metacpan.org/pod/Mojolicious::Guides::Routing#HTTP-methods
17:55 Bloke Lol, browsers.
18:01 mat_ joined #mojo
18:31 phillipadsmith joined #mojo
18:31 jabberwok joined #mojo
18:31 dod joined #mojo
18:32 Ralesk joined #mojo
18:33 Grinnz joined #mojo
18:47 jonasbn joined #mojo
18:50 Grinnz that is among the several reasons why i tend to use javascript to call my APIs, rather than browser forms
19:42 disputin joined #mojo
20:13 tchaves joined #mojo
20:16 Janos joined #mojo
20:22 salv0 joined #mojo
20:24 rshadow joined #mojo
20:31 rshadow joined #mojo
21:15 jmac joined #mojo
21:31 jonasbn joined #mojo
21:40 jonasbn joined #mojo
21:51 PryMar56 joined #mojo
22:11 jonasbn joined #mojo
22:20 cheako joined #mojo
22:21 cheako Hello, I'm reading through http://mojolicious.org/perldoc/Mojolicious/Guides/Routing and am starting to find errors and parts that can be explained more exactly.
22:22 cheako qr/(?-xism:^\/user\/show/([^\/.]+))/   this is missing an escaping backslash.
22:22 Grinnz i don't see one
22:23 cheako show\?/
22:23 Grinnz ah
22:24 cheako $r->get('/welcome')->to(controller => 'foo', action => 'welcome');   Shoudln't foo be Foo? so that the module can be called MyApp::Controller::Foo  or is lowercase to name case conversion expected and if it is there should be a note indicating this to avoid confusion.
22:26 Grinnz there is a conversion
22:26 Grinnz https://metacpan.org/pod/Mojolicious::Guides::Routing#Special-stash-values
22:28 cheako I see
22:48 good_news_everyon joined #mojo
22:48 good_news_everyon [mojo] kraih pushed 1 new commit to master: https://git.io/v9T2t
22:48 good_news_everyon mojo/master b72f37d Sebastian Riedel: fix typo in example
22:48 good_news_everyon left #mojo
22:57 disputin joined #mojo
23:03 jonasbn joined #mojo
23:05 disputin joined #mojo
23:33 jonasbn joined #mojo

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