Work around for VSC Bug - Fix Path Mappings to lowercase the drive letter

[DonJayamanne]

When using path mappings on windows, ensure the drive letter is lower cased.
I.e. when user uses the following setup, ${workspaceFolder} gets translated to a path with an upper case drive letter.

      "pathMappings": [
        {
          "localRoot": "${workspaceFolder}/modules/rfidreader/",
          "remoteRoot": "/app/"
        }
      ],

This is a work around for a VSC bug

[DonJayamanne]

Prescribed Solution:

• Modify the provideAttachDefaults method of src/client/debugger/extension/configuration/resolvers/attach.ts file:
  • To take the localRoot of path mappings and convert drive letter to lower case (on windows)

[karrtikr]

When the pathmappings is not an empty list, localRoot is not being lowercased.

[DonJayamanne]

@kimadeline I can see where this will fail.
We're lower casing ONLY if pathMappings.length === 0.
We need to change it and ensure all paths in path mappings are lower cased.

Thanks @karrtikr

[karrtikr]

Suggestion

• Check src\client\debugger\extension\hooks\childProcessAttachService.ts -> fixPathMappings().
  Use similar logic to resolve localRoot

const systemVariables = new SystemVariables(config.workspaceFolder);
const localRoot = systemVariables.resolveAny(<localRootProvided>);

• Then take the localRoot of path mappings and convert drive letter to lower case (on windows)

[DonJayamanne]

@kimadeline I agree with @karrtikr His suggestion puts the changes in the right place.

[karrtikr]

validated