Add support for per-folder browse.path.

[sean-mcmanus]

Fix for #1073

[sean-mcmanus]

Are there any multi-root CMake Tools automated tests? I didn't see any.

[bobbrow]

They are under test/extension-tests/multi-root-UI. There is a config in launch.json that will run them. The tests are not currently written to handle Windows very well. You should run them on linux or macOS.

[sean-mcmanus]

They are under test/extension-tests/multi-root-UI. There is a config in launch.json that will run them. The tests are not currently written to handle Windows very well. You should run them on linux or macOS.

Ah, thanks...not sure how I didn't see that.

[sean-mcmanus]

They are under test/extension-tests/multi-root-UI. There is a config in launch.json that will run them. The tests are not currently written to handle Windows very well. You should run them on linux or macOS.

Ah, thanks...not sure how I didn't see that.

I added some browsePath checks to the test.

[sean-mcmanus]

@bobbrow Why is the continuous-integration/travis-ci stuck?

[bobbrow]

The travis CI is actually running, but the results are not being reported back to GitHub for some reason. I've been checking here for results: https://travis-ci.org/github/microsoft/vscode-cmake-tools/pull_requests. I also added my feedback to a bug report, but haven't heard anything back from the Travis team. https://travis-ci.community/t/status-checks-randomly-broken/7876

We need to switch this repo over to Azure pipelines anyway, so I haven't investigated much.

[sean-mcmanus]

How did this PR get submitted? It was not intentional.

[bobbrow]

It's ok. I do want you to update the normalize cases to use platformNormalize instead though.

[bobbrow]

And it would probably be better to update the test to have unique target values in codeModel2 just so we don't have to worry about bugs sneaking through.

[sean-mcmanus]

@bobbrow Sure, but I think I should delay that change until we ship our next C/C++ extension version, so 0.27.0 aren't broken...in case there's a reason to ship a CMake Tools update before then.

[bobbrow]

Can you create the PR now and we'll just wait to submit it then? If we don't, we're going to forget since this is a month or more away.

[sean-mcmanus]

Can you create the PR now and we'll just wait to submit it then? If we don't, we're going to forget since this is a month or more away.

#1163