| Summary: | an internal error occurred during: "Launching XXX Debug." | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Matthias Garzke <matthias_garzke> |
| Component: | Debug | Assignee: | Paul Pazderski <paul-eclipse> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | jacob.luojie, loskutov, paul-eclipse, sarika.sinha |
| Version: | 4.15 | ||
| Target Milestone: | 4.16 M3 | ||
| Hardware: | PC | ||
| OS: | All | ||
| See Also: |
https://bugs.eclipse.org/bugs/show_bug.cgi?id=558489 https://git.eclipse.org/r/161818 https://git.eclipse.org/c/platform/eclipse.platform.debug.git/commit/?id=4296b4127e2e3b8549f5733c4c7d8f2f24a4b891 https://bugs.eclipse.org/bugs/show_bug.cgi?id=562724 |
||
| Whiteboard: | |||
|
Description
Matthias Garzke
fr.ac6.mcu.debug.openocd.processanalyzer.logger.OpenOCDRuntimeProcess.createStreamsProxy(OpenOCDRuntimeProcess.java:268) uses internal API that is changed via bug 558489. Whoever provides fr.ac6.mcu.debug.openocd.processanalyzer should update the code and do not use internal API. @Paul: would it make sense to provide old StreamsProxy(Process process, String encoding) constructor that re-directs to the new one StreamsProxy(Process process, Charset encoding)? (In reply to Andrey Loskutov from comment #1) > @Paul: would it make sense to provide old StreamsProxy(Process process, > String encoding) constructor that re-directs to the new one > StreamsProxy(Process process, Charset encoding)? Yes. Would not hurt us to restore the method. I'll make a patch. New Gerrit change created: https://git.eclipse.org/r/161818 Thanks for the quick response and effort! Is there any action needed to be done by myself? (In reply to Paul Pazderski from comment #2) > (In reply to Andrey Loskutov from comment #1) > > @Paul: would it make sense to provide old StreamsProxy(Process process, > > String encoding) constructor that re-directs to the new one > > StreamsProxy(Process process, Charset encoding)? > > Yes. Would not hurt us to restore the method. I'll make a patch. Have you found the code for the client? If it uses even more internal API? (In reply to Matthias Garzke from comment #4) > Thanks for the quick response and effort! > > Is there any action needed to be done by myself? Find out the code for that tool or find out the authors and ask if the proposed patch makes their code happy again. Google just found this: https://www.openstm32.org/forumthread8134 for me. (In reply to Andrey Loskutov from comment #5) > (In reply to Paul Pazderski from comment #2) > > (In reply to Andrey Loskutov from comment #1) > > > @Paul: would it make sense to provide old StreamsProxy(Process process, > > > String encoding) constructor that re-directs to the new one > > > StreamsProxy(Process process, Charset encoding)? > > > > Yes. Would not hurt us to restore the method. I'll make a patch. > > Have you found the code for the client? If it uses even more internal API? No, that's why I restored all three constructors changed in bug 558489. If it breaks on other internal api changes, well there is a known risk on using internals. The product is probably this [1] but I found no code and apart from the forum thread you linked only this related thread [2] which describes the same error Matthias posted. [1] https://www.ac6-tools.com/content.php/content_SW4MCU/lang_en%5fUS.xphp [2] https://www.openstm32.org/forumthread8137 If this may helps you: http://www.ac6-tools.com/Eclipse-updates/org.openstm32.system-workbench.update-site-v2/ This is the source from where i obtain the workbench. Gerrit change https://git.eclipse.org/r/161818 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.debug.git/commit/?id=4296b4127e2e3b8549f5733c4c7d8f2f24a4b891 *** Bug 563007 has been marked as a duplicate of this bug. *** |