Skip to content

Disable the debug popup on win32/msvc. #4475

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

Conversation

lygstate
Copy link
Contributor

For not popup dialog when crash happend on MSVC/win32
related issue: #4463

JerryScript-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]

@lygstate lygstate force-pushed the disable-win32-crash-popup branch from 0bf32d5 to 0491a46 Compare January 15, 2021 15:35
@lygstate lygstate changed the title Disable the debug popup. Disable the debug popup on win32/msvc. Jan 15, 2021
For not popup dialog when crash happend on MSVC/win32
related issue: jerryscript-project#4463

JerryScript-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
@lygstate lygstate force-pushed the disable-win32-crash-popup branch from 0491a46 to 14722df Compare January 15, 2021 16:06
@akosthekiss
Copy link
Member

I'm not completely sure that console apps should be tweaked just because Windows doesn't let them crash without displaying graphical UI elements. Isn't it easier to simply disable that service on Windows? https://docs.microsoft.com/en-us/powershell/module/windowserrorreporting/disable-windowserrorreporting?view=win10-ps

@rerobika rerobika added the windows Windows specific label Jan 15, 2021
@rerobika
Copy link
Member

rerobika commented Feb 4, 2021

I'd second @akosthekiss's suggestion and I can agree that it must not be implemented in the console app.

@ossy-szeged
Copy link
Contributor

Is it possible to disable these popup by runner script somehow? If we would like to run Windows testing ever on the CI, we will need to solve this issue somehow.

@akosthekiss
Copy link
Member

@ossy-szeged I've suggested (or rather requested feedback on) an approach but haven't heard back since.

@lygstate
Copy link
Contributor Author

lygstate commented Feb 5, 2021

@ossy-szeged I've suggested (or rather requested feedback on) an approach but haven't heard back since.

disable service will affect other applicationsuch as UI

@akosthekiss
Copy link
Member

@lygstate You are free to re-enable the service after the execution of the automated tests.

@lygstate
Copy link
Contributor Author

lygstate commented Feb 5, 2021

@lygstate You are free to re-enable the service after the execution of the automated tests.

Yeap, but it's a budren,

@ossy-szeged
Copy link
Contributor

@lygstate You are free to re-enable the service after the execution of the automated tests.

Yeap, but it's a budren,

+1 for @akosthekiss's suggestion. It's not a burden to modify the test runner script to disable the
service before testing and enable the service after testing. It's obviously a testing feature/issue that's
why we don't want to modify jerry console application, but modify the testing script to reach your goal.

@akosthekiss
Copy link
Member

@lygstate You are free to re-enable the service after the execution of the automated tests.

Yeap, but it's a budren,

Well, it can be scripted, I guess. Then it is less of a burden. And I definitely prefer the "scripting burden" over cluttering code with Windows-specific debug-popup-blocking magic only because Windows effectively blocks automated testing.

@lygstate lygstate closed this Feb 5, 2021
@lygstate
Copy link
Contributor Author

lygstate commented Feb 5, 2021

@lygstate You are free to re-enable the service after the execution of the automated tests.

Yeap, but it's a budren,

Well, it can be scripted, I guess. Then it is less of a burden. And I definitely prefer the "scripting burden" over cluttering code with Windows-specific debug-popup-blocking magic only because Windows effectively blocks automated testing.

No sure if have such a script, there code is also references to other implementation. I'l stick this locally as it works for me

@lygstate lygstate deleted the disable-win32-crash-popup branch February 9, 2021 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
windows Windows specific
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants