[Box Backup-commit] Re: #3: Merge Win32 branch
boxbackup-dev@fluffy.co.uk
boxbackup-dev@fluffy.co.uk
Sat, 18 Nov 2006 00:13:14 -0000
#3: Merge Win32 branch
-----------------------+----------------------------------------------------
Reporter: chris | Owner: chris
Type: task | Status: new
Priority: normal | Milestone: 0.11
Component: bbackupd | Version: trunk
Resolution: | Keywords: win32 merge branch
-----------------------+----------------------------------------------------
Comment (by martin):
* [1050] + [1051] ok
* [1052] ok
* [1053] ok
* [1054] ok
* [1055] ok
* [1058] ok - this case insensitive stuff should probably be more
generic. I believe HFS+ on Mac OS X is also case insensitive so this and
other related changesets should work there too
* [1059] ok
* [1062] ok
* [1064] ok
* [1070] ok
* [1071] ok
* [1072] I'd like to know the underlying issue here, it's not clear to me
why this is the right fix
* [1073] ok
* [1075] ok
* [1076] why are you using TRUE instead of true?
* [1077] ok
* [1078] I presume it works, so ok to merge. style comments: in the
regexp you could have used {{{ (....){5} }}} instead of having five of
them. Not sure why you couldn't just {{{..\..\..\..\..}}} instead. Also
why is require using \ but open using /?
* [1079] macros are nasty, prefer inline fns. Why undefining fstat only
in these files? (needs explanation I think)
* [1080] ok
--
Ticket URL: <http://bbdev.fluffy.co.uk/trac/ticket/3#comment:184>
Box Backup <http://www.fluffy.co.uk/boxbackup/>
An open source, completely automatic on-line backup system for UNIX.