Logs for #nikola for 2015-05-11

13:11:39 <[Tritium]> ralsina_: re 1325 - the point is to allow one to create a new page or post in $PWD instead of the default location.  Helpful if you have a deep post structure.
13:12:07 <ralsina_> <[Tritium] yes, but that is not impossible
13:12:28 <ralsina_> and OTOH it makes it harder to choose the filename from the site root
13:13:06 <ralsina_> But in any case, let's write that as a proposal, so we are all seeing the same thing and then we can talk about it
13:13:13 <ralsina_> makes sense?
13:13:26 <[Tritium]> >.> Its...all...in...the bug...
13:14:33 <ralsina_> Can you edit the bug description into a proposal?
13:14:42 <[Tritium]> Its not killing me, but it is still counterintuitive.
13:15:58 <[Tritium]> Think about this: what other command do you give relative paths to that you expect to be relative to anything but PWD.  Your counterexample is the edge case.
13:24:11 <[Tritium]> updated
13:26:20 <ralsina_> <[Tritium] which is why I am willing to take a look at the issue after it has been sleeping this long
13:26:45 <ralsina_> otherwise I could keep ignoring it just as happily as until now :-)
14:15:50 <bnmnetp> 	Is there a way to set the location of my .doit.db file in conf.py or some other rc file rather than using the --db-file option all the time?
14:18:50 <ralsina_> bnmnetp: not that I know of
14:19:35 <bnmnetp> It looks like DOIT_CONFIG would be the way to go.  Maybe that could be set in config.py if needed, and then __main__.py could just add key value information rather than create it?
17:35:01 <ralsina_> bnmnetp: we are already setting *some* doit config using DOIT_CONFIG ... could you file a bug so we can set more taking values from conf.py?
17:36:40 <bnmnetp> Yes, I will file a bug... I was actually just working on a change/PR so that DOIT_CONFIG could be defined in conf.py and then extended in __main__
17:39:15 <bnmnetp> But... now that I'm testing I'm not sure that is the right approach
17:43:11 <ralsina_> well, it's the one I was thinking about :-)
17:43:26 <ralsina_> it's sort of a very very advanced option, tho
18:01:00 <ChrisWarrick> let’s not tell people about it publicly
18:01:39 <ralsina_> Let's bury it in the manual or something
18:02:00 <ralsina_> but not even a default config setting, since it can break things in awesome and surprising ways
18:02:29 <ChrisWarrick> make it completely undocumented
18:02:39 <ChrisWarrick> the 5 users will ask us or read __main__.py
18:32:00 -GitHub[nikola]:#nikola- [nikola] ralsina created fix-1325 (+1 new commit): http://git.io/vURoR
18:32:00 -GitHub[nikola]:#nikola- nikola/fix-1325 9ea4522 Roberto Alsina: Fix #1325
18:32:21 -GitHub[nikola]:#nikola- [nikola] ralsina opened pull request #1712: Fix #1325 (master...fix-1325) http://git.io/vURoX
18:51:34 <ChrisWarrick> fuuck
18:51:44 <ChrisWarrick> why is EVERYONE calling fucking nikola list commands
18:51:50 <ChrisWarrick> https://github.com/getnikola/nikola/issues/1713
18:52:28 <ChrisWarrick> ralsina_: that ↑ should be an easy fix, just copy-paste the self.l declaration to github_deploy.py and use it to call real_scan_files.  mind doing it in your free time?
18:53:45 <ChrisWarrick> nikola check*
18:54:36 <ChrisWarrick> ralsina_: nvm, done it myself
18:55:16 -GitHub[nikola]:#nikola- [nikola] Kwpolska pushed 1 new commit to master: http://git.io/vUR9w
18:55:16 -GitHub[nikola]:#nikola- nikola/master e78294e Chris Warrick: fix #1713 -- fix nikola check call in gh deploy
18:57:05 <ralsina_> ChrisWarrick: looks like the extra l argument in real_scan is not even used ... is it meant to be a logger?
18:57:17 <ChrisWarrick> ralsina_: it should be used
18:57:28 <ChrisWarrick> self.l = self._doitargs['cmds'].get_plugin('list')(config=self.config, **self._doitargs)
18:57:43 <ChrisWarrick> ralsina_: this is what we do to get a `nikola list` command instance
18:57:49 <ChrisWarrick> blame doit v0.28.0
18:58:17 <ralsina_> I may be having a senior moment but I still don't see where it's used in real_scan_files
18:58:27 <ralsina_> ah, saw it
18:58:28 <ralsina_> sorry
18:58:41 <ChrisWarrick> no problem
22:37:28 -travis-ci:#nikola- getnikola/nikola#4987 (fix-1325 - 9ea4522 : Roberto Alsina): The build passed.
22:37:29 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/commit/9ea452281577
22:37:30 -travis-ci:#nikola- Build details: http://travis-ci.org/getnikola/nikola/builds/62123874
23:04:35 -travis-ci:#nikola- getnikola/nikola#4989 (master - e78294e : Chris Warrick): The build passed.
23:04:36 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/a573ed9a2b38...e78294e7ee2a
23:04:36 -travis-ci:#nikola- Build details: http://travis-ci.org/getnikola/nikola/builds/62126857