Re: [mpg123-devel] Please test new network code in/out of mpg123 (now with HTTPS support)!
Brought to you by:
sobukus
From: Dave Y. <dav...@gm...> - 2022-05-14 02:13:15
|
On 05/13/22 08:41 AM, Thomas Orgis wrote: > Am Fri, 13 May 2022 07:57:08 -0700 > schrieb Dave Yeo <dav...@gm...>: > >> Unluckily it seems that io.h also declares _setmode() but not _O_BINARY >> or _O_TEXT. > OK, please try the current trunk. I made a more elaborate compile check > now that tests the function as well the symbol. Does > > 1. the check work? > > $ grep -i setmode src/config.h > /* for Win/DOS system with setmode() */ > /* #undef HAVE_SETMODE */ > /* for Win/DOS system with _setmode() */ > /* #undef HAVE__SETMODE */ > > You should have one of those defined. > > 2. the build work? Yes the build works fine now. Still need to workaround for os2me.h, need something like, --- configure.ac (revision 5065) +++ configure.ac (working copy) @@ -1945,6 +1945,7 @@ ;; os2) OS2_LIBS="-lcx -lmmpm2" + CFLAGS+="-idirafter /@unixroot/usr/include/os2tk45" AC_CHECK_HEADERS([os2.h]) # os2me.h depends on os2.h # Yes, that way of coding it is ugly. @@ -1953,6 +1954,7 @@ # We mimick exactly the way how the header will be used. # It seems to be picky... AC_CHECK_HEADERS([os2me.h], [], [], [#define INC L_OS2MM +#undef VERSION #define INCL_DOS #define INCL_VIO #define INCL_KBD as klibc has no multimedia support so need to fall back on the official OS/2 toolkit headers and we don't want those libc headers. The above works for configure but the -idirafter doesn't end up in the makefiles. Currently using C_INCLUDE_PATH, make 'CFLAGS -idirafter /@unixroot/usr/include/os2tk45' also works. Thanks Dave |