Skip to content

Commit

Permalink
Don't treat clew as part of an "external" clcc module
Browse files Browse the repository at this point in the history
There is no obvious authoritative upstream for clew anyway, so it causes
philosophical problems for distros. For a while, we used to use a zip archive
from the "clcc" project on SourceForge that included clew.c and
clew.h. (Before that we also just had clew.c and clew.h in our source repo.)

So, drop the external/clcc module and have clew.c and clew.h in the source
repo again. But this time clew is in a module of its own, not in sc.

This re-introduces "No need to have OpenCL optional at configure-time"

This reverts commit 764836c.

Change-Id: I413142f4f9f8399489f9c3e5327132822f07a454
Reviewed-on: https://gerrit.libreoffice.org/13368
Reviewed-by: David Tardon <[email protected]>
Tested-by: David Tardon <[email protected]>
  • Loading branch information
Tor Lillqvist authored and dtardon committed Dec 8, 2014
1 parent 4517c94 commit bf6544c
Show file tree
Hide file tree
Showing 27 changed files with 1,673 additions and 130 deletions.
1 change: 0 additions & 1 deletion Makefile.fetch
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,6 @@ $(WORKDIR)/download: $(BUILDDIR)/config_host.mk $(SRCDIR)/download.lst $(SRCDIR)
$(call fetch_Optional,CAIRO,CAIRO_TARBALL) \
$(call fetch_Optional,CAIRO,PIXMAN_TARBALL) \
$(call fetch_Optional,CDR,CDR_TARBALL) \
$(call fetch_Optional,CLCC,CLCC_TARBALL) \
$(call fetch_Optional,CLUCENE,CLUCENE_TARBALL) \
$(call fetch_Optional,CMIS,CMIS_TARBALL) \
$(call fetch_Optional,COINMP,COINMP_TARBALL) \
Expand Down
2 changes: 1 addition & 1 deletion Repository.mk
Original file line number Diff line number Diff line change
Expand Up @@ -316,7 +316,7 @@ $(eval $(call gb_Helper_register_libraries_for_install,OOOLIBS,ooo, \
chartcore \
chartcontroller \
chartopengl \
$(call gb_Helper_optional,CLCC,clew) \
$(call gb_Helper_optional,OPENCL,clew) \
$(if $(filter $(OS),WNT),,cmdmail) \
cppcanvas \
configmgr \
Expand Down
11 changes: 0 additions & 11 deletions RepositoryExternal.mk
Original file line number Diff line number Diff line change
Expand Up @@ -279,17 +279,6 @@ endef

endif # SYSTEM_GLEW

define gb_LinkTarget__use_clew
$(call gb_LinkTarget_set_include,$(1),\
-I$(call gb_UnpackedTarball_get_dir,clcc)/src \
$$(INCLUDE) \
)
$(call gb_LinkTarget_use_libraries,$(1),\
clew \
)

endef

define gb_LinkTarget__use_iconv
$(call gb_LinkTarget_add_libs,$(1),-liconv)

Expand Down
1 change: 1 addition & 0 deletions RepositoryModule_host.mk
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ $(eval $(call gb_Module_add_moduledirs,libreoffice,\
bridges \
canvas \
chart2 \
$(call gb_Helper_optional,OPENCL,clew) \
cli_ure \
$(call gb_Helper_optional,DESKTOP,codemaker) \
comphelper \
Expand Down
4 changes: 2 additions & 2 deletions external/clcc/Library_clew.mk → clew/Library_clew.mk
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ $(eval $(call gb_Library_add_libs,clew,\
))
endif

$(eval $(call gb_Library_add_generated_cobjects,clew,\
UnpackedTarball/clcc/src/clew \
$(eval $(call gb_Library_add_cobjects,clew,\
clew/source/clew \
))

# vim: set noet sw=4 ts=4:
2 changes: 1 addition & 1 deletion external/clcc/Makefile → clew/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@

module_directory:=$(dir $(realpath $(firstword $(MAKEFILE_LIST))))

include $(module_directory)/../../solenv/gbuild/partial_build.mk
include $(module_directory)/../solenv/gbuild/partial_build.mk

# vim: set noet sw=4 ts=4:
5 changes: 2 additions & 3 deletions external/clcc/Module_clcc.mk → clew/Module_clew.mk
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
#

$(eval $(call gb_Module_Module,clcc))
$(eval $(call gb_Module_Module,clew))

$(eval $(call gb_Module_add_targets,clcc,\
$(eval $(call gb_Module_add_targets,clew,\
Library_clew \
UnpackedTarball_clcc \
))

# vim: set noet sw=4 ts=4:
326 changes: 326 additions & 0 deletions clew/source/clew.c

Large diffs are not rendered by default.

19 changes: 3 additions & 16 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -1096,10 +1096,6 @@ AC_ARG_ENABLE(cairo-canvas,
platforms where Cairo is available.
])

AC_ARG_ENABLE(opencl,
AS_HELP_STRING([--disable-opencl],
[Determines whether to build with OpenCL support.]))

AC_ARG_ENABLE(dbus,
AS_HELP_STRING([--disable-dbus],
[Determines whether to enable features that depend on dbus.
Expand Down Expand Up @@ -10267,21 +10263,12 @@ dnl =================================================
dnl Check whether to build with OpenCL support.
dnl =================================================

AC_MSG_CHECKING([whether to build with the OpenCL support.])
ENABLE_OPENCL=

if test $_os != iOS -a $_os != Android -a "x$enable_opencl" != "xno"; then
AC_MSG_RESULT([yes])
# CLCC in BUILD_TYPE tells that we are building a bundled clcc (just the clew part), OPENCL in
# BUILD_TYPE tells that OpenCL is potentially available on the platform (optional at run-time,
if test $_os != iOS -a $_os != Android; then
# OPENCL in BUILD_TYPE tells that OpenCL is potentially available on the platform (optional at run-time,
# used through clew).
BUILD_TYPE="$BUILD_TYPE CLCC OPENCL"
ENABLE_OPENCL=TRUE
BUILD_TYPE="$BUILD_TYPE OPENCL"
AC_DEFINE(HAVE_FEATURE_OPENCL)
else
AC_MSG_RESULT([no])
fi
AC_SUBST(ENABLE_OPENCL)

dnl ===================================================================
dnl Check whether to enable glTF support
Expand Down
4 changes: 2 additions & 2 deletions cui/Library_cui.mk
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@ $(eval $(call gb_Library_use_sdk_api,cui))
$(eval $(call gb_Library_use_libraries,cui,\
$(call gb_Helper_optional,AVMEDIA,avmedia) \
basegfx \
$(call gb_Helper_optional,OPENCL,\
clew) \
comphelper \
cppu \
cppuhelper \
Expand Down Expand Up @@ -66,8 +68,6 @@ $(eval $(call gb_Library_use_libraries,cui,\

$(eval $(call gb_Library_use_externals,cui,\
boost_headers \
$(call gb_Helper_optional,OPENCL,\
clew) \
icuuc \
icu_headers \
))
Expand Down
2 changes: 0 additions & 2 deletions download.lst
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,6 @@ export BSH_TARBALL := ea570af93c284aa9e5621cd563f54f4d-bsh-2.0b1-src.tar.gz
export CAIRO_TARBALL := f101a9e88b783337b20b2e26dfd26d5f-cairo-1.10.2.tar.gz
export CDR_MD5SUM := b33fd0be3befdd1b37777e08ce058bd9
export CDR_TARBALL := libcdr-0.1.1.tar.bz2
export CLCC_TARBALL := clcc-0.3.0-25-src.zip
export CLCC_MD5SUM := 7ec003cc775d1cd06e789fb054c1d695
export CLUCENE_TARBALL := 48d647fbd8ef8889e5a7f422c1bfda94-clucene-core-2.3.3.4.tar.gz
export CMIS_TARBALL := 22f8a85daf4a012180322e1f52a7563b-libcmis-0.4.1.tar.gz
export COINMP_MD5SUM := 1cce53bf4b40ae29790d2c5c9f8b1129
Expand Down
1 change: 0 additions & 1 deletion external/Module_external.mk
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ $(eval $(call gb_Module_add_moduledirs,external,\
$(call gb_Helper_optional,BSH,beanshell) \
$(call gb_Helper_optional,CAIRO,cairo) \
$(call gb_Helper_optional,CDR,libcdr) \
$(call gb_Helper_optional,CLCC,clcc) \
$(call gb_Helper_optional,CLUCENE,clucene) \
$(call gb_Helper_optional,CMIS,libcmis) \
$(call gb_Helper_optional,COINMP,coinmp) \
Expand Down
25 changes: 0 additions & 25 deletions external/clcc/UnpackedTarball_clcc.mk

This file was deleted.

52 changes: 0 additions & 52 deletions external/clcc/clew-non-static.patch

This file was deleted.

Loading

0 comments on commit bf6544c

Please sign in to comment.