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 jerryscript to master #4

Merged
merged 7 commits into from
Dec 3, 2021

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Dec 1, 2021

This PR updates jerryscript to current master branch. The current 2.4 release was in January so like there will be another before too long and makes sense to base further work on that.

The API has changed slightly (for the better!) and there are lots of fixes and improvements.

Building the snapshot tool on Windows now uses the MinGW compiler rather than the default Visual Studio tools, which aren't part of the Sming installation. (They're a pain to install as well.)

@mikee47
Copy link
Contributor Author

mikee47 commented Dec 1, 2021

Discussion on changes to public API in master and due in next release jerryscript-project/jerryscript#4186

@slaff
Copy link
Owner

slaff commented Dec 2, 2021

@mikee47 Can you add here library CI as suggested in SmingHub/Sming#2434 ?

@mikee47
Copy link
Contributor Author

mikee47 commented Dec 2, 2021

@slaff Repo. doesn't need any changes, follow instructions here https://github.com/SmingHub/Sming/tree/develop/Tools/ci/library

@mikee47
Copy link
Contributor Author

mikee47 commented Dec 2, 2021

(I've done it - forgot I had access!)

@slaff
Copy link
Owner

slaff commented Dec 2, 2021

(I've done it - forgot I had access!)

Thanks a lot! I need CI also for the pull requests but let me see if I can find the needed settings...

@mikee47 mikee47 force-pushed the update/jerryscript-master branch from 441da97 to 64e86dc Compare December 2, 2021 09:29
@mikee47
Copy link
Contributor Author

mikee47 commented Dec 2, 2021

@slaff the Windows ESP32 build overran, got a PR to fix that by splitting out each SOC as usual.

@mikee47
Copy link
Contributor Author

mikee47 commented Dec 2, 2021

@slaff If you want to check the new fix use https://raw.githubusercontent.com/mikee47/Sming/feature/appveyor-byoc/Tools/ci/library/appveyor.txt as the custom yaml

@slaff slaff merged commit ad056fd into slaff:develop Dec 3, 2021
@mikee47 mikee47 deleted the update/jerryscript-master branch November 1, 2022 07:36
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.

2 participants