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

Update versions of node and workflow items in GitHub Actions #345

Merged
merged 3 commits into from
Dec 10, 2024

Conversation

jonahgraham
Copy link
Contributor

@jonahgraham jonahgraham commented Dec 10, 2024

This change also includes some test changes that are as a result of the changes in versions of tools.

@jonahgraham
Copy link
Contributor Author

Parts of this update is needed because of actions/runner-images#10636 - the main branch is failing because of running build and test on latest ubuntu.

The UART fails to open test was looking for a `0` in the error
message, corresponding to the port number that was being opened.
However, depending on the specific machine run on, the actual
error from the backend may vary. On GitHub actions instead of
referring to the port number it says:

  "error on socket connection\n - AggregateError"

And on my local machine it shows

  "error on socket connection\n - Error: connect ECONNREFUSED 127.0.0.1"

With this change we still require the adapter to report an error
on socket connection, but the specific error details are not
required to match any particular pattern.
@jonahgraham jonahgraham force-pushed the versions branch 3 times, most recently from f8879be to c92ddde Compare December 10, 2024 02:31
…ing port

We were parsing the port number from gdbserver before it had
been fully output. Use a regex now to wait for terminating
newline so we know gdbserver has output full port.
@jonahgraham jonahgraham merged commit 9272451 into eclipse-cdt-cloud:main Dec 10, 2024
4 checks passed
@jonahgraham jonahgraham deleted the versions branch December 10, 2024 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant