On Ma, 2007-07-02 at 21:39 +0200, lars wrote:
> Hi Friedel,
>
> > I _think_ the second hunk in your patch might not be what we want, but
> > I'm not sure. (I just see that I changed it in my current diff.)
> >
> > goalchildren = {}
> > allchildren = []
> > for childname in self.project.browsefiles(dirfilter=dirfilter,
> > depth=depth, includedirs=True, includefiles=False):
> > - allchildren.append(childname+"/")
> > + allchildren.append(childname + os.path.sep)
> > for childname in self.project.browsefiles(dirfilter=dirfilter,
> > depth=depth, includedirs=False, includefiles=True):
> > allchildren.append(childname) initial = dirfilter
>
> as far as I can see, 'childname' as returned by 'browsefiles' (defined in
> projects.py) consists of real pathnames (basically produced by 'getpofiles'
> in 'potree.py').
> Thus 'childname' should be extended by os.path.sep instead of '/' to stay
> platform independent.
>
Perhaps I was experimenting with a few things, so I'll still look at
this. Your patch is almost definitely an improvement, it just doesn't
solve the problem with the goals, so we'll still need to investigate
further.
The problem with the goals are that we might want to store things in the
unix way even if we are on a windows environment (I don't know, I'll
still need to think about it).
>
> > In my diff I stayed with "/" and I think it has to do with the goal
> > problem I described earlier in the thread. Have you been able to
> > reproduce the problem?
>
> I do not (easily) have a windows environment around, so I could not test it -
> sorry!
> Is it broken with and without the patch?
>
> Lars
No, it definitely works on unix at the moment. I'm currently testing it
all under wine. Windows python installed with msiexec and installing all
the python modules in my wine installation with the windows python
interpreter :-)
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Translate-pootle mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/translate-pootle
