]> granicus.if.org Git - fribidi/commitdiff
Fix parallel build
authorKhaled Hosny <khaledhosny@eglug.org>
Sun, 4 Feb 2018 12:32:00 +0000 (14:32 +0200)
committerKhaled Hosny <khaledhosny@eglug.org>
Sun, 4 Feb 2018 12:42:22 +0000 (14:42 +0200)
Parallel build was failing while building the man pages. doc/Makefile.am
tries cleverly to handle the fact that c2man produces all the man pages
at once, following the approach described in:
https://www.gnu.org/software/automake/manual/html_node/Multiple-Outputs.html

However this is failing because c2man is not generating all the
requested man pages, so the rule ends being run multiple times instead
of once and parallel builds are broken. One of the requested pages is
for a function that doesn’t exist, fribidi_log2vis_get_embedding_levels_ex,
so I removed it from fribidi.def.

Two functions were not documented, so I added minimal documentation for
them.

But the last two are for variables not functions and c2man does not like
that it seems so I added a hack to drop these two as well without
modifying fribidi.def.

Fixes https://github.com/fribidi/fribidi/issues/45

doc/Makefile.am
lib/fribidi-common.h
lib/fribidi.def

index fd1d1600c1fa3324c222431f5f154bbe0b2d3fb5..d9d5c021e1f54669183458fce63b441226ab049c 100644 (file)
@@ -27,7 +27,8 @@ include $(top_srcdir)/lib/Headers.mk
 headers += $(libfribidi_la_headers)
 includepath += -I$(top_builddir)/lib -I$(top_srcdir)/lib
 includevpath += :$(top_builddir)/lib:$(top_srcdir)/lib
-inst_symbols += $(libfribidi_la_symbols)
+# The las two lines are not functions, we don't want them here.
+inst_symbols += $(shell head -n -2 $(top_srcdir)/lib/fribidi.def)
 
 dist_man_MANS += $(inst_symbols:=.3)
 dist_noinst_MANS += $(noinst_symbols:=.3)
index c9391450a4e42200a345b0ddc937b85f38666eac..a7a54003ae374de4374a8e9dad31e4557e62e499 100644 (file)
 
 
 #define fribidi_debug_status FRIBIDI_NAMESPACE(debug_status)
+/* fribidi_debug_status - get current debug state
+ *
+ */
 FRIBIDI_ENTRY int fribidi_debug_status (
   void
 );
+
 #define fribidi_set_debug FRIBIDI_NAMESPACE(set_debug)
+/* fribidi_set_debug - set debug state
+ *
+ */
 FRIBIDI_ENTRY int
 fribidi_set_debug (
   int state            /* new state to set */
index d78009057f0380e041f514571f11a360b95fe715..0d79a49cd7585c8ebd3c577631704081d19c8834 100644 (file)
@@ -16,7 +16,6 @@ fribidi_get_type_internal
 fribidi_join_arabic
 fribidi_log2vis
 fribidi_log2vis_get_embedding_levels
-fribidi_log2vis_get_embedding_levels_ex
 fribidi_mirroring_status
 fribidi_remove_bidi_marks
 fribidi_reorder_line