Skip to content

Commit 7d82553

Browse files
authored
Fix building for musl (#803)
reverts: #298. There is no gurantee that either HAVE_STRTOD_L or HAVE_LOCALE_H will be defined at compile-time. Try for example building this project :) The locale usage is now defined in POSIX and therefore we can now assume it will be available on Linux, except for ancient systems. https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/locale.h.html Another way of dealing with this would be to have a global mlt_config.h where HAVE_LOCALE_H and HAVE_STRTOD_L are defined to values determined when mlt is installed.
1 parent f3c308b commit 7d82553

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/framework/mlt_property.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
#include <sys/param.h>
3131
#endif
3232

33-
#if (defined(__GLIBC__) && !defined(__APPLE__)) || defined(HAVE_LOCALE_H)
33+
#if (defined(__linux__) && !defined(__APPLE__))
3434
# include <locale.h>
3535
#elif defined(__APPLE__) || (defined(__FreeBSD_version) && __FreeBSD_version >= 900506)
3636
# include <xlocale.h>

0 commit comments

Comments
 (0)