search

JeffGarland / liaw2019-process

Repository for initial drafting of boost.process standards paper
MIT License
5 stars 4 forks source link

[MSVC] Template resolution #60

Open klemens-morgenstern opened 3 years ago

klemens-morgenstern commented 3 years ago

MSVC has an issue in the current version, with the process signatures, where the process_launcher is placed at the end. The is the following code is not working on msvc, but on gcc:

proc::process proc1{target_path, {"--exit-code", "40", "--wait", "100"}, proc::default_process_launcher{}};

MSVC gives a C2440 error.

I do not know if this is an issue with MSVC or with my design, we might need to move the position of the customer initializer. I am doing the latter for some experimentation on process_groups. Either way, this should be mentioned.

Edit: I had a few more issues and it seems that the MSVC concept support is just buggy as of now.

JeffGarland commented 3 years ago

I had a few more issues and it seems that the MSVC concept support is just buggy as of now.

That doesn't surprise me given that the ink isn't even dry on c++20. I don't think we want to change our design. Maybe there's a channel for reporting the bugs?

klemens-morgenstern commented 3 years ago

Given that the official description of MSVC is "partial support" I am quite certain they're aware. Wasn't too sure with the design though.