> + turn on mergedlibs & LTO for Windows - any objections ? (Michael)
Jesús already replied about the LTO bit, thanks for reviewing. As for --enable-mergelibs, it doesn't look good. Apparently that has never been tried for Windows? Or at least not recently? The option has a help message which says it works for Linux only. OK, it mentioned Android, too, which was wrong information, so maybe the lack of mention of Windows is also just a mistake, and mergelibs has worked for Windows at some stage? (We do build one single "merged" library for each Android app (and one single "merged" executable for each iOS app), but not using the "mergelibs" mechanism.) Anyway, I tried building on Windows with --enable-mergelibs=all, did not succeed. Build stops when linking the chartcontrollerlo library, no imerged.lib found. And indeed I don't see any *merge*.lib anywhere in workdir or instdir. This once again shows how harmful it is to have tons of configuration options. They don't get tested and bit-rot. Each binary option doubles the number of possible configurations. If some change in how LO is built is good, then we should just do it, unconditionally (on some or all platforms, depending on case), not make it into an option that will bit-rot. --tml _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
