We should not be adding required dependencies, so WebCrypto should be disabled when GnuTLS is not found:
-- WTF_USE_TILED_BACKING_STORE OFF
-- ENABLE_PLUGIN_PROCESS_GTK2 .............. ON
-- checking for module 'libsecret-1'
-- package 'libsecret-1' not found
-- Could NOT find LIBSECRET (missing: LIBSECRET_INCLUDE_DIRS LIBSECRET_LIBRARIES)
CMake Error at /home/guijemont/.local/share/cmake-3.0/Modules/FindPackageHandleStandardArgs.cmake:136 (message):
Could NOT find GnuTLS: Found unsuitable version "2.12.14", but required is
at least "3.0.0" (found /usr/lib/x86_64-linux-gnu/libgnutls.so)
Call Stack (most recent call first):
cmake only complains if you explicitly enable SUBTLE_CRYPTO and it fails to find the required dependencies. I understand that's the correct behavior.
It is build-webkit that sets ENABLE_SUBTLE_CRYPTO by default, if you call cmake directly instead it works fine.
So maybe we should disable subtle-crypto by default in build-webkit?
Created attachment 242286 [details]
Comment on attachment 242286 [details]
The patch looks good to me, r+ anyone?
If this if provided by the JHBuild, I don't think it's necessary. If it's not provided by the JHBuild, perhaps it should be?
Actually Tools/gtk/install-dependencies is already installing
libgnutls-dev, so maybe there's nothing to fix here?
WebCrypto is already disabled when we use cmake directly, so I don't
think distributors/end users are really required to install gnutls.
(In reply to comment #5)
> Actually Tools/gtk/install-dependencies is already installing
> libgnutls-dev, so maybe there's nothing to fix here?
> WebCrypto is already disabled when we use cmake directly, so I don't
> think distributors/end users are really required to install gnutls.
Probably it should be one of those features that's enabled only when the dependency is found. I think Safari is shipping it so we can probably start as well.
Created attachment 242754 [details]
Ok, here it goes
Comment on attachment 242754 [details]
Clearing flags on attachment: 242754
Committed r176928: <http://trac.webkit.org/changeset/176928>
All reviewed patches have been landed. Closing bug.