[Box Backup-commit] Re: #3: Merge Win32 branch

boxbackup-dev at fluffy.co.uk boxbackup-dev at fluffy.co.uk
Sat Nov 18 00:13:14 GMT 2006


#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.



More information about the Boxbackup-commit mailing list