Bug 163733 – [EFL] REGRESSION(r207616): Build is broken due to missing libgeoclue-dev package

• Component: New Bugs
• Status: RESOLVED FIXED

[Csaba Osztrogonác]

I tried an EFL WebKit build on a fresh new Ubuntu 16.04 
machine and got the following build error:

In file included from ../../Source/WebCore/platform/geoclue/GeolocationProviderGeoclue1.cpp:27:0:
../../Source/WebCore/platform/geoclue/GeolocationProviderGeoclue.h:32:36: fatal error: geoclue/geoclue-master.h: No such file or directory
compilation terminated.

It seems https://trac.webkit.org/changeset/204445 is the culprit.

geoclue/geoclue-master.h is in libgeoclue-dev package, 
but after r204445, only libgeoclue-2-dev is installed.

But the question is what is the proper fix?
- adding geoclue-2.0 to <efl|gtk>/install-dependencies
  --> isn't enough, because USE(GEOCLUE2) isn't exposed to the build
- Source/cmake/OptionsEfl.cmake should use SET_AND_EXPOSE_TO_BUILD similar to GTK 
- fixing the geoclue 1 build
- removing the geoclue path
- whatever else

Additionally I got the following warnings which should be fixed or suppressed:
DerivedSources/WebCore/Geoclue2Interface.c: In function 'geoclue_manager_complete_get_client':
DerivedSources/WebCore/Geoclue2Interface.c:673:21: warning: unused parameter 'object' [-Wunused-parameter]
DerivedSources/WebCore/Geoclue2Interface.c: In function 'geoclue_manager_complete_add_agent':
DerivedSources/WebCore/Geoclue2Interface.c:693:21: warning: unused parameter 'object' [-Wunused-parameter]
DerivedSources/WebCore/Geoclue2Interface.c: In function 'geoclue_client_complete_start':
DerivedSources/WebCore/Geoclue2Interface.c:2373:20: warning: unused parameter 'object' [-Wunused-parameter]
DerivedSources/WebCore/Geoclue2Interface.c: In function 'geoclue_client_complete_stop':
DerivedSources/WebCore/Geoclue2Interface.c:2391:20: warning: unused parameter 'object' [-Wunused-parameter]


note: I just reported this bug, but have no time and interest 
in fixing it myself. I'm going to upload my WIP patch, feel 
free to pick it up if it is important for somebody.

[Csaba Osztrogonác]

Created attachment 292201 [details]
Patch

WIP patch, it fixes the geoclue 2 build.

[Gyuyoung Kim]

Comment on attachment 292201 [details]
Patch

LGTM.

[Carlos Garcia Campos]

(In reply to comment #0)
> I tried an EFL WebKit build on a fresh new Ubuntu 16.04 
> machine and got the following build error:
> 
> In file included from
> ../../Source/WebCore/platform/geoclue/GeolocationProviderGeoclue1.cpp:27:0:
> ../../Source/WebCore/platform/geoclue/GeolocationProviderGeoclue.h:32:36:
> fatal error: geoclue/geoclue-master.h: No such file or directory
> compilation terminated.
> 
> It seems https://trac.webkit.org/changeset/204445 is the culprit.
> 
> geoclue/geoclue-master.h is in libgeoclue-dev package, 
> but after r204445, only libgeoclue-2-dev is installed.
> 
> But the question is what is the proper fix?
> - adding geoclue-2.0 to <efl|gtk>/install-dependencies
>   --> isn't enough, because USE(GEOCLUE2) isn't exposed to the build
> - Source/cmake/OptionsEfl.cmake should use SET_AND_EXPOSE_TO_BUILD similar
> to GTK 
> - fixing the geoclue 1 build
> - removing the geoclue path
> - whatever else
> 
> Additionally I got the following warnings which should be fixed or
> suppressed:
> DerivedSources/WebCore/Geoclue2Interface.c: In function
> 'geoclue_manager_complete_get_client':
> DerivedSources/WebCore/Geoclue2Interface.c:673:21: warning: unused parameter
> 'object' [-Wunused-parameter]
> DerivedSources/WebCore/Geoclue2Interface.c: In function
> 'geoclue_manager_complete_add_agent':
> DerivedSources/WebCore/Geoclue2Interface.c:693:21: warning: unused parameter
> 'object' [-Wunused-parameter]
> DerivedSources/WebCore/Geoclue2Interface.c: In function
> 'geoclue_client_complete_start':
> DerivedSources/WebCore/Geoclue2Interface.c:2373:20: warning: unused
> parameter 'object' [-Wunused-parameter]
> DerivedSources/WebCore/Geoclue2Interface.c: In function
> 'geoclue_client_complete_stop':
> DerivedSources/WebCore/Geoclue2Interface.c:2391:20: warning: unused
> parameter 'object' [-Wunused-parameter]

This is generated code, Michael suppressed those warnings in GTK+ build in r206381 with:

set_source_files_properties(${DERIVED_SOURCES_WEBCORE_DIR}/Geoclue2Interface.c PROPERTIES COMPILE_FLAGS -Wno-unused-parameter)

> 
> note: I just reported this bug, but have no time and interest 
> in fixing it myself. I'm going to upload my WIP patch, feel 
> free to pick it up if it is important for somebody.

[Csaba Osztrogonác]

Comment on attachment 292201 [details]
Patch

Clearing flags on attachment: 292201

Committed r207668: <http://trac.webkit.org/changeset/207668>

[Csaba Osztrogonác]

All reviewed patches have been landed.  Closing bug.