When updating our internal IDE (based on Eclipse 2022-03) to ship with <a class="commi

regression: CI build hangs at "loading target platform" about lcdsl HOT 3 CLOSED

ssi-schaefer commented on July 29, 2024

regression: CI build hangs at "loading target platform"

from lcdsl.

Comments (3)

jukzi commented on July 29, 2024

looks like the com.wamas.ide.launching.ui.build.LcDslTargetPlatformSupport.uriMap is completed too late - @szarnekow could you please take a look into the stacktrace.

from lcdsl.

jukzi commented on July 29, 2024

here the code relys on uriMap already initialized while it is currently in the initializeCache():

        at com.google.common.util.concurrent.Futures.getUnchecked(Futures.java:1317)
        at com.wamas.ide.launching.ui.build.LcDslTargetPlatformSupport.stateChanged(LcDslTargetPlatformSupport.java:554) <--use cache
        at org.eclipse.pde.internal.core.PluginModelManager.fireStateChanged(PluginModelManager.java:427)
        at org.eclipse.pde.internal.core.PluginModelManager.initializeTable(PluginModelManager.java:644)
        - locked <0x0000000704146b58> (a org.eclipse.pde.internal.core.PluginModelManager)
        at org.eclipse.pde.internal.core.PluginModelManager.getExternalModels(PluginModelManager.java:1178)
        at com.wamas.ide.launching.ui.build.LcDslTargetPlatformSupport.lambda$11(LcDslTargetPlatformSupport.java:201) <--initializeCache()

from lcdsl.

szarnekow commented on July 29, 2024

I'm not at my machine right now but judging from the stack and from what I remember: It looks like we can skip all handling of events if the future is not done yet. The event processing should only be necessary to update the cached information. The update is not necessary if nothing was cached yet.

from lcdsl.

Recommend Projects