Message ID | 20201001173230.829-1-luoyonggang@gmail.com (mailing list archive) |
---|---|
Headers | show |
Series | Fixes curses on msys2/mingw | expand |
On Thu, Oct 01, 2020 at 07:39:33PM +0200, Paolo Bonzini wrote: > On 01/10/20 19:32, Yonggang Luo wrote: > > And also convert related configure script to meson. > > V5-V6 > > Dropping configure: Fixes ncursesw detection under msys2/mingw by convert the= > > m to meson first. > > That need the meson 0.56 upstream to fixes the curses detection. > > Add > > * configure: fixes indent of $meson setup > > > > Yonggang Luo (4): > > configure: fixes indent of $meson setup > > curses: Fixes compiler error that complain don't have langinfo.h on > > msys2/mingw > > curses: Fixes curses compiling errors. > > win32: Simplify gmtime_r detection not depends on if _POSIX_C_SOURCE > > are defined on msys2/mingw > > > > configure | 42 ++++----------------------------------- > > include/sysemu/os-win32.h | 4 ++-- > > ui/curses.c | 14 ++++++------- > > util/oslib-win32.c | 4 ++-- > > 4 files changed, 15 insertions(+), 49 deletions(-) > > > > --=20 > > 2.28.0.windows.1 > > > > > > Looks good (though there is no conversion to meson yet in this version). > > Not trivial, but perhaps Gerd can take it? Sure, series looks good. Can't finish this today due to -ENOTIME and I'm offline next week so this has to wait a bit in the UI patch queue ... take care, Gerd
On Fri, Oct 02, 2020 at 02:47:44PM +0200, Gerd Hoffmann wrote: > On Thu, Oct 01, 2020 at 07:39:33PM +0200, Paolo Bonzini wrote: > > On 01/10/20 19:32, Yonggang Luo wrote: > > > And also convert related configure script to meson. > > > V5-V6 > > > Dropping configure: Fixes ncursesw detection under msys2/mingw by convert the= > > > m to meson first. > > > That need the meson 0.56 upstream to fixes the curses detection. > > > Add > > > * configure: fixes indent of $meson setup > > > > > > Yonggang Luo (4): > > > configure: fixes indent of $meson setup > > > curses: Fixes compiler error that complain don't have langinfo.h on > > > msys2/mingw > > > curses: Fixes curses compiling errors. > > > win32: Simplify gmtime_r detection not depends on if _POSIX_C_SOURCE > > > are defined on msys2/mingw > > > > > > configure | 42 ++++----------------------------------- > > > include/sysemu/os-win32.h | 4 ++-- > > > ui/curses.c | 14 ++++++------- > > > util/oslib-win32.c | 4 ++-- > > > 4 files changed, 15 insertions(+), 49 deletions(-) > > > > > > --=20 > > > 2.28.0.windows.1 > > > > > > > > > > Looks good (though there is no conversion to meson yet in this version). > > > > Not trivial, but perhaps Gerd can take it? > > Sure, series looks good. > > Can't finish this today due to -ENOTIME and I'm offline next week so > this has to wait a bit in the UI patch queue ... Patch 2 needs more work I think, as its missing the corresponding configure change. Regards, Daniel
On Fri, Oct 02, 2020 at 04:38:01PM +0100, Daniel P. Berrangé wrote: > On Fri, Oct 02, 2020 at 02:47:44PM +0200, Gerd Hoffmann wrote: > > On Thu, Oct 01, 2020 at 07:39:33PM +0200, Paolo Bonzini wrote: > > > On 01/10/20 19:32, Yonggang Luo wrote: > > > > And also convert related configure script to meson. > > > > V5-V6 > > > > Dropping configure: Fixes ncursesw detection under msys2/mingw by convert the= > > > > m to meson first. > > > > That need the meson 0.56 upstream to fixes the curses detection. > > > > Add > > > > * configure: fixes indent of $meson setup > > > > > > > > Yonggang Luo (4): > > > > configure: fixes indent of $meson setup > > > > curses: Fixes compiler error that complain don't have langinfo.h on > > > > msys2/mingw > > > > curses: Fixes curses compiling errors. > > > > win32: Simplify gmtime_r detection not depends on if _POSIX_C_SOURCE > > > > are defined on msys2/mingw > > > > > > > > configure | 42 ++++----------------------------------- > > > > include/sysemu/os-win32.h | 4 ++-- > > > > ui/curses.c | 14 ++++++------- > > > > util/oslib-win32.c | 4 ++-- > > > > 4 files changed, 15 insertions(+), 49 deletions(-) > > > > > > > > --=20 > > > > 2.28.0.windows.1 > > > > > > > > > > > > > > Looks good (though there is no conversion to meson yet in this version). > > > > > > Not trivial, but perhaps Gerd can take it? > > > > Sure, series looks good. > > > > Can't finish this today due to -ENOTIME and I'm offline next week so > > this has to wait a bit in the UI patch queue ... > > Patch 2 needs more work I think, as its missing the corresponding > configure change. Also the series must be adapted to latest master, auto-rebase didn't work ... take care, Gerd