[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread Chris McDonough
Chris McDonough added the comment: I merged a modified version of your patch to the trunk, thanks Sean! -- status: chatting -> resolved __ Repoze Bugs __ __

[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread Sean Gillies
Sean Gillies added the comment: Okay, here's the latest patch. 100% test coverage and *much* less code. It works with the jinja2 environment, adjusting its auto_reload attribute to match the settings whenever get_source is called. __ Repoze Bugs

[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread admin
System message: __ Repoze Bugs __ ___ Repoze-dev mailing list Repoze-dev@lists.repoze.org http://lists.repoze.org/listinfo/repoze-dev

[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread Chris McDonough
Chris McDonough added the comment: This looks almost right Sean. But it's possible (even probable) that the ISettings utility won't be registered when you manufacture a BfgLoader. So I think the call to _auto_reload needs to happen in get_source each time (or at least memoize its results on th

[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread Sean Gillies
Sean Gillies added the comment: New patch including loader which implements the mtime based reloading suggested in jinja2.loaders. And doctests. __ Repoze Bugs __ autoreload.patch Description: Bin

[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread admin
System message: __ Repoze Bugs __ ___ Repoze-dev mailing list Repoze-dev@lists.repoze.org http://lists.repoze.org/listinfo/repoze-dev

[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread Sean Gillies
Sean Gillies added the comment: Oops, dropped the patch. Here it is. -- status: unread -> chatting __ Repoze Bugs __ autoreload.patch Description: Binary data

[Repoze-dev] [issue88] Autoreloading for repoze.bfg.jinja2

2009-06-08 Thread Sean Gillies
New submission from Sean Gillies : Autoreloading isn't supported in repoze.bfg.jinja2, but adding it was easy enough. Test included. -- messages: 235 nosy: sgillies priority: feature status: unread title: Autoreloading for repoze.bfg.jinja2 topic: repoze.bfg