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

3.0.0-alpha3 escaping fix #8968

Merged
merged 1 commit into from Dec 11, 2023
Merged

3.0.0-alpha3 escaping fix #8968

merged 1 commit into from Dec 11, 2023

Conversation

lyusupov
Copy link
Contributor

@lyusupov lyusupov commented Dec 7, 2023

Description of Change

On a Linux host the alpha3 gives a build error like this:

>: -c: line 1: unexpected EOF while looking for matching `''
>: -c: line 2: syntax error: unexpected end of file

alpha2 was fine.

The issue looks very similar to #8424

The idea of this fix was taken from that PR #8433 .

@CLAassistant
Copy link

CLAassistant commented Dec 7, 2023

CLA assistant check
All committers have signed the CLA.

@lucasssvaz
Copy link
Collaborator

What is the IDE version you are using ? We were having this issue on the CI but it was due to an old IDE version being used. It was fixed in #8961.

@lyusupov
Copy link
Contributor Author

lyusupov commented Dec 7, 2023

What is the IDE version you are using ?

IDE1 ( ARDUINO IDE VERSION: 1.8.13 )

GitHub Actions build log : https://github.com/lyusupov/SoftRF/actions/runs/7123496016/job/19396200382

@me-no-dev
Copy link
Member

@pillo79 could you please comment on this. It is indeed the case that older IDE1 versions have issue with escaping that line.

@VojtechBartoska VojtechBartoska added the Resolution: Awaiting response Waiting for response of author label Dec 8, 2023
@pillo79
Copy link
Contributor

pillo79 commented Dec 11, 2023

Thanks for the heads-up @me-no-dev! We don't support the old IDE anymore so I had not checked that... but it's such an easy fix to make everyone happy, I'm all for it!

@me-no-dev me-no-dev merged commit 7f5db80 into espressif:master Dec 11, 2023
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Awaiting response Waiting for response of author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants