Code review comment for lp:~jameinel/bzr/2.1.0b4-485771-win32-nostar-noglob

Revision history for this message
John A Meinel (jameinel) wrote :

This fixes a bug that Alexander noticed.

Basically, we were replacing "\" with "/" for all arguments. Whether they were quoted, or globbed or whatever.

This changes it, so that only arguments with an actual glob get swapped.

I tested it with "bzr add foo\bar" and everything still seems to work fine. (foo\bar gets translated to foo/bar at an appropriate time later during the add process.)

« Back to merge proposal