I tried remove it and I an error in listdir. Massimo
On Apr 19, 7:22 pm, Jonathan Lundell <jlund...@pobox.com> wrote: > On Apr 19, 2010, at 5:05 PM, mdipierro wrote: > > >> Why does the compiled version include the .pyc files in controllers/ > >> (outside of compiled/)? It doesn't seem like those get used, at least not > >> by run_controller_in. > > > I think you refer to line 298 of compileapp.py. I do not recall why it > > is there. I will try remove it and see what breaks. > > Ah, yes, that's where it comes from. > > > > >> And a minor thing: there's a lot of this kind of thing: > > >> path = os.path.join(folder, 'compiled/') > > >> Should the '/' really be part of the call? It sort of defeats the purpose > >> of os.path.join. > > > There are a few cases when in can be removed (and I just did in trunk) > > but most of them are required else listdir does not work properly. > > listdir appends a '/' if there isn't one already. (Though I don't see why it > needs to, as long as it computes n correctly.) > > BTW, the comment for listdir seems to confuse add_dirs with drop. Or > something. And a pair of typos: s/b "regex pattern to filter files" > > -- > Subscription settings:http://groups.google.com/group/web2py/subscribe?hl=en
