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

tooks/kconfig: Win32 build (GIT8266O-816) #1239

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Sonic-Amiga
Copy link

New MSYS2/MinGW is a pure Windows environment due to ABI differences on 64 bits. Therefore UNIX calls are not available when building for mingw target. Avoid creating UNAME_RELEASE symbol, which is taken from current Linux kernel version. Anyways this isn't important for this SDK.

New MSYS2/MinGW is a pure Windows environment due to ABI differences on 64 bits.
Therefore UNIX calls are not available when building for mingw target. Avoid
creating UNAME_RELEASE symbol, which is taken from current Linux kernel version.
Anyways this isn't important for this SDK.

Signed-off-by: Pavel Fedin <[email protected]>
@CLAassistant
Copy link

CLAassistant commented Jun 9, 2023

CLA assistant check
All committers have signed the CLA.

@Sonic-Amiga
Copy link
Author

Hello everyone! I see this SDK is being actively worked on. I've got myself a project and i like new environment. So i am bulding in a modern version of MSYS2/MinGW64 environment, which requires some tweaks to the SDK. I hope you'll be interested

@github-actions github-actions bot changed the title tooks/kconfig: Win32 build tooks/kconfig: Win32 build (GIT8266O-816) Jun 9, 2023
On native Windows ncurses Esc key is reported immediately on first press.
Adjust on_key_esc() to work correctly in this scenario

Signed-off-by: Pavel Fedin <[email protected]>
@Sonic-Amiga
Copy link
Author

Added one more patch; now kconfig fully works.
Note: except backspace key because that one is an ncurses bug: msys2/MINGW-packages#17494

On Windows rename() does not overwrite existing file but errors out. Handle that
by explicitly erasing what should be erased.

Saving configuration now works.

Signed-off-by: Pavel Fedin <[email protected]>
@Sonic-Amiga
Copy link
Author

One more fix for overwriting the config.

@Sonic-Amiga
Copy link
Author

@donghengqaz @cwespressif hello! Gentle PING

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