Open Bug 1644765 Opened 5 years ago Updated 5 years ago

manifest_relpath should be consistent in the use of path separators: \ or /

Categories

(Testing :: Mozbase, defect, P3)

defect

Tracking

(Not tracked)

People

(Reporter: robwu, Unassigned)

References

Details

In bug 1644223, tests did not run as expected on Windows because of inconsistent use of path separators with ancestor_manifest.

I fixed this, and touched a line in manifestparser.py that assigns manifest_relpath, which raised a question about the need for path normalization. I looked into it, and found that path separators are not consistently used with manifest_relpath. Often / is expected, but sometimes \ is assigned. This should be fixed (to avoid bugs, and to avoid normalization / post-processing in various places).

I did an exhaustive enumeration of all manifest_relpath occurrences and their use of path separators at:
https://phabricator.services.mozilla.com/D78921#2405289

Severity: -- → S3
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.