Hi,
I have submitted a patch for review:
https://gerrit.libreoffice.org/2492
To pull it, you can do:
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/92/2492/1
Fiddle with hard-coded neon config.h to support USE_NONBLOCKING_CONNECT
...see the condition in neon's src/neon_socket.c under which
USE_NONBLOCKING_CONNECT would be enabled. This should not have any immediate
effect in LibreOffice, as we do not call ne_set_connect_timeout. But it might
be beneficial to selectively do so in light of rhbz#915743 "thread
deadlock/slow join in insert->hyperlink in impress," to work around neon's
disability to abort potentially blocking calls.
The enabled features are hopefully POSIX-y enough to allow for plain #ifdef UNX.
If not, we need to adapt this to more fine grained conditions.
Change-Id: I6bf0a89c1ae73d7183f515e780fff7fe0ae2fed0
---
M neon/configs/config.h
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/neon/configs/config.h b/neon/configs/config.h
index 85de093..db68aa2 100644
--- a/neon/configs/config.h
+++ b/neon/configs/config.h
@@ -53,10 +53,14 @@
/* #undef HAVE_EXPAT */
/* Define to 1 if you have the `fcntl' function. */
-/* #undef HAVE_FCNTL */
+#ifdef UNX
+#define HAVE_FCNTL 1
+#endif
/* Define to 1 if you have the <fcntl.h> header file. */
-/* #undef HAVE_FCNTL_H */
+#ifdef UNX
+#define HAVE_FCNTL_H 1
+#endif
/* Define to 1 if you have the `fstat64' function. */
#define HAVE_FSTAT64
@@ -71,7 +75,9 @@
/* #undef HAVE_GETNAMEINFO */
/* Define to 1 if you have the `getsockopt' function. */
-/* #undef HAVE_GETSOCKOPT */
+#ifdef UNX
+#define HAVE_GETSOCKOPT 1
+#endif
/* Define if GnuTLS support is enabled */
/* #undef HAVE_GNUTLS */
@@ -234,7 +240,9 @@
#endif
/* Define to 1 if the system has the type `socklen_t'. */
-/* #undef HAVE_SOCKLEN_T */
+#ifdef UNX
+#define HAVE_SOCKLEN_T 1
+#endif
/* Define to 1 if you have the `SSL_SESSION_cmp' function. */
/* #undef HAVE_SSL_SESSION_CMP */
--
To view, visit https://gerrit.libreoffice.org/2492
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6bf0a89c1ae73d7183f515e780fff7fe0ae2fed0
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Stephan Bergmann <sbergman@redhat.com>
Context
- [PATCH] Fiddle with hard-coded neon config.h to support USE_NONBLOCK... · Stephan Bergmann (via Code Review)
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.