osutils / _win32_pathjoin bytes / strings issues

Bug #1922746 reported by Robert Ladyman
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Breezy
Triaged
High
Unassigned

Bug Description

I've managed to compile brz for windows, however, there remain some problems.

osutils.py code for windows (_win32_pathjoin, _win32_abspath, etc.) fails when called for brz init.

I've tracked this back to a couple of commits (6677.1.2 and 6677.1.1 'b-sprinkling') in bzr/working_tree_4 and dirstate.py although the issue is really the file / directory-handling in osutils and win32utils.

For example, _win32_joinpath() performs an ntpath.join() call but then tries to do a replace('\\', '/') which fails if a byte-string is passed as any of the arguments. The non-win32 version just uses os.path.join (lne 421 of osutils.py) without the attempted replace(). _

Tags: win32

Related branches

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Yeah, I think we need to have a careful look at dirstate and how it should interact with ntpath on Windows. mgz might have some opinions?

Jelmer Vernooij (jelmer)
Changed in brz:
status: New → Triaged
importance: Undecided → High
tags: added: win32
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.