Skip to content

Conversation

kittaakos
Copy link
Contributor

@kittaakos kittaakos commented Mar 22, 2023

Motivation

Change description

Closes #1501

readonly_1501.mp4

Other information

Reviewer checklist

  • PR addresses a single concern.
  • The PR has no duplicates (please search among the Pull Requests before creating one)
  • PR title and description are properly filled.
  • Docs have been added / updated (for bug fixes / features)

@kittaakos kittaakos added topic: code Related to content of the project itself topic: theia Related to the Theia IDE framework labels Mar 22, 2023
@kittaakos kittaakos self-assigned this Mar 22, 2023
Akos Kitta added 4 commits April 11, 2023 10:44
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
Closes arduino#1501
Upstream: eclipse-theia/theia#12354

Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know Akos is already aware, but I'll add a note here for transparency:

There was a hypothesis this update might have allowed us to avoid the GLibC dependency that is causing builds of the IDE on a machine with a newer Ubuntu version (e.g., the ubuntu-latest GitHub Actions runner has Ubuntu 22.04) to fail to start on machines with an older version of GLibC, as discussed at #1983 and #259.

I tried starting the build of this PR (created on the ubuntu-latest runner) on a Ubuntu 18.04 machine and encountered the same problem as discussed in #1983:

Error: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_arduinlmTF8J/resources/app/node_modules/nsfw/build/Release/nsfw.node)
    at process.func [as dlopen] (node:electron/js2c/asar_bundle:5:1800)
    at Object.Module._extensions..node (node:internal/modules/cjs/loader:1170:18)
    at Object.func [as .node] (node:electron/js2c/asar_bundle:5:1800)
    at Module.load (node:internal/modules/cjs/loader:982:32)
    at Module._load (node:internal/modules/cjs/loader:823:12)
    at Function.c._load (node:electron/js2c/asar_bundle:5:13331)
    at Module.require (node:internal/modules/cjs/loader:1006:19)
    at require (node:internal/modules/cjs/helpers:93:18)
    at Object.<anonymous> (/tmp/.mount_arduinlmTF8J/resources/app/node_modules/nsfw/js/src/index.js:4:14)
    at Module._compile (node:internal/modules/cjs/loader:1110:14) {
  code: 'ERR_DLOPEN_FAILED'
}

@kittaakos
Copy link
Contributor Author

Replaced by #2027. Closing as invalid.

@kittaakos kittaakos closed this Apr 20, 2023
@kittaakos kittaakos added the conclusion: invalid Issue/PR not valid label Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: invalid Issue/PR not valid topic: code Related to content of the project itself topic: theia Related to the Theia IDE framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Saving sketch fails silently when files are read-only
2 participants