Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

logCollectionRegExps not working? #83

Closed
vadi2 opened this issue Apr 19, 2021 · 5 comments
Closed

logCollectionRegExps not working? #83

vadi2 opened this issue Apr 19, 2021 · 5 comments
Labels
bug Something isn't working

Comments

@vadi2
Copy link
Contributor

vadi2 commented Apr 19, 2021

Using the upgraded run-vcpkg with the logCollectionRegExps option, it didn't seem to pick up the build failures in this log, despite the regex matching the 1st log path:

image

@lukka
Copy link
Owner

lukka commented Apr 19, 2021

@vadi2 thanks for trying this out!

I have both a unit test and a workflow validation test where I have successfully exercised the "log collection" feature.

The validation test is using CMake to trigger an error, it would be nice to use vcpkg as well. How to trigger a vcpkg error with a log? Once we found that out, I can add it to the validation workflow and verify whether and why it is not working.

@lukka
Copy link
Owner

lukka commented Apr 19, 2021

@vadi2 by code inspection, the unit test is leveraging the wrong ToolRunner class, this may certainly be a problem. To be investigated.

@lukka lukka added the bug Something isn't working label Apr 19, 2021
@lukka
Copy link
Owner

lukka commented Apr 22, 2021

it turns out that the wrong ToolRunner used in the unit test is not the problem. It was a wrong usage of RegExp.lastIndex the actual root cause. I also augmented the default regular expression passed onto the logCollectionRegExps input. Let me know next time an error occurs whether this is working!

@vadi2
Copy link
Contributor Author

vadi2 commented Apr 22, 2021

Will do!

@lukka
Copy link
Owner

lukka commented Jun 29, 2021

It should be fixed in run-vcpkg@v7.4 or simply run-vcpkg@v7. Please report back!

@lukka lukka closed this as completed Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants