#50: Same problems svn -> svn
-----------------------------+----------------------------------------------
  Reporter:  [EMAIL PROTECTED]  |       Owner:  anonymous 
      Type:  defect          |      Status:  assigned  
  Priority:  major           |   Milestone:  VersionOne
 Component:  tailor          |     Version:  0.9       
Resolution:                  |    Keywords:            
-----------------------------+----------------------------------------------
Changes (by Lele Gaifax):

  * status:  new => assigned
  * owner:  lele => anonymous

Comment:

 I'm fixing the cut&paste glitch, thanks for reporting.

 On the other problems you mention:

  1. Why you think ''sincerev'' is more correct than ''sincerev+1''? Here
 ''sincerev'' is the last revision that was applied, and we want to collect
 what remains/what's new on the source repository, so I keep thinking that
 current code is right. Can you be a little more specific on this point?
  2. There is no reason to change the code about ''encoding-errors-
 policy'': that's actually an option that is being read from the
 configuration, so you can put `encoding-errors-policy = replace` in the
 config file with the same meaning.
  3. About `^M` in the changelog, it seem a rather specific glitch that
 should be better handled by a ''before-commit'' hook, rather than doing a
 replace in any case. BTW, instead of using `string.replace(targetstring,
 old, new)` newest Pythons recommends the idiom `targetstring.replace(old,
 new)` which is slightly faster.

-- 
Ticket URL: <http://progetti.arstecnica.it/tailor/ticket/50>
Tailor <http://progetti.arstecnica.it/tailor>
An inter-VCs changeset exchanger
_______________________________________________
Tailor mailing list
[email protected]
http://lists.zooko.com/mailman/listinfo/tailor

Reply via email to