From 2cea85db388d34337676466ef7f50c22e685c7d0 Mon Sep 17 00:00:00 2001 From: "Carsten Haitzler (Rasterman)" Date: Wed, 15 Nov 2017 11:48:25 +0900 Subject: [PATCH] Revert series of patches from cedric (due to dependencies and conflicts) Revert "elementary: currently double declare elm_init/shutdown." This reverts commit 44bb0c18480f5094fcd0c8be93de87be5c1d78c5. Revert "elementary: fix efl_ui_multibutton installed headers." This reverts commit 32a213dc722be2bfec5fb2b513dd556cfb7e9f33. Revert "elementary: introduce Efl_Ui.h." This reverts commit df3d3f7334a79f1ab661b31787002f50af59b3f3. Revert "ecore: do not display error message on cancel." This reverts commit 99654b7cd29b418e0308be350c8d26208c0905a8. Revert "efl: and don't forget to install the new dependencies." This reverts commit 814ffb9b6bd50d498bfd98f4b8a75f1cad3cc0cf. Revert "ecore: remove EFL_OBJECT_BETA as Efl_Core.h is for Efl new inerfaces." This reverts commit 619d0f3cff023414feb8f2aaeebf468806c50b44. Revert "ecore: move EAPI_MAIN from elementary to ecore." This reverts commit e5d84da864214b9f772808040f22e1614889a25f. as such commit e5d84da864214b9f772808040f22e1614889a25f starts the breaking. enlightenment, terminologya and other apps can't compile against that efl anymore. 619d0f3cff023414feb8f2aaeebf468806c50b44 then makes this even worse with even more header errors and undefined types. on top of this df3d3f7334a79f1ab661b31787002f50af59b3f3 then starts making elementary_test segfault when it runs. it wont even start up. asu such of these 7 commits in the first 4 (that are then relied on later) 3 of these first 4 cause serious breakage. this simply is a complete lack of testing changes, so i've rolled fl back to before these things so it builds and works again and you can build against it. PLEASE test these things. this looks ot me to be obviously a lack of any testing... :( --- Makefile.am | 5 +- configure.ac | 1 - pc/.gitignore | 1 - pc/efl-ui.pc.in | 20 -- src/Makefile_Elementary.am | 4 +- src/lib/ecore/Efl_Core.h | 4 +- src/lib/ecore/efl_general.h | 17 -- src/lib/elementary/Efl_Ui.h | 253 ------------------ src/lib/elementary/Elementary.h | 4 + src/lib/elementary/efl_ui_multibuttonentry.h | 1 + src/lib/elementary/elm_general.h | 4 +- .../ecore/system/upower/ecore_system_upower.c | 9 +- 12 files changed, 16 insertions(+), 307 deletions(-) delete mode 100644 pc/efl-ui.pc.in delete mode 100644 src/lib/elementary/Efl_Ui.h diff --git a/Makefile.am b/Makefile.am index 8fb921c734..1106b60b01 100644 --- a/Makefile.am +++ b/Makefile.am @@ -136,9 +136,7 @@ pc/emile.pc \ pc/eet.pc \ pc/evas.pc \ pc/ecore.pc \ -pc/efl-core.pc \ pc/ecore-con.pc \ -pc/efl-net.pc \ pc/ecore-ipc.pc \ pc/ecore-file.pc \ pc/ecore-input.pc \ @@ -159,8 +157,7 @@ pc/emotion.pc \ pc/ethumb.pc \ pc/ethumb_client.pc \ pc/elocation.pc \ -pc/elementary.pc \ -pc/efl-ui.pc +pc/elementary.pc if HAVE_CXX11 pkgconfig_DATA += \ diff --git a/configure.ac b/configure.ac index 003190ba26..ba9b78318e 100644 --- a/configure.ac +++ b/configure.ac @@ -5606,7 +5606,6 @@ pc/efl-cxx.pc pc/efl-wl.pc pc/efl-core.pc pc/efl-net.pc -pc/efl-ui.pc pc/evas-fb.pc pc/evas-eglfs.pc pc/evas-opengl-x11.pc diff --git a/pc/.gitignore b/pc/.gitignore index 5657ab6a68..5e87ab1e64 100644 --- a/pc/.gitignore +++ b/pc/.gitignore @@ -76,4 +76,3 @@ /efl-wl.pc /efl-core.pc /efl-net.pc -/efl-ui.pc \ No newline at end of file diff --git a/pc/efl-ui.pc.in b/pc/efl-ui.pc.in deleted file mode 100644 index 04c9dcf472..0000000000 --- a/pc/efl-ui.pc.in +++ /dev/null @@ -1,20 +0,0 @@ -prefix=@prefix@ -exec_prefix=@exec_prefix@ -libdir=@libdir@ -includedir=@includedir@ -datarootdir=@datarootdir@ -datadir=@datadir@/elementary -pkgdatadir=@datadir@/elementary -vmaj=@VMAJ@ -vmin=@VMIN@ -vmic=@VMIC@ -themes=${pkgdatadir}/themes -eoincludedir=@datadir@/eolian/include -eolian_flags=-I${pc_sysrootdir}${eoincludedir}/elementary-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/emap-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/elocation-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/efreet-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/edje-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/eio-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/eldbus-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/ecore-con-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/ecore-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/efl-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/eo-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/eet-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/emile-@VMAJ@ -I${pc_sysrootdir}${eoincludedir}/eina-@VMAJ@ - -Name: efl_ui -Description: EFL user interface abstraction library -Requires.private: @requirements_pc_elementary@ -Version: @VERSION@ -Libs: -L${libdir} -lelementary -lefl -leina -lpthread -leet -levas -lecore -lecore_evas -lecore_file -lecore_input -ledje -leo -lethumb_client -lemotion -lecore_imf -lecore_con -leldbus -lefreet -lefreet_mime -lefreet_trash -leio @requirements_public_libs_elementary@ @requirements_public_libs_eina@ @requirements_libs_eina@ @requirements_libs_elm@ -Cflags: -I${includedir}/elementary-@VMAJ@ -I${includedir}/emap-@VMAJ@ -I${includedir}/elocation-@VMAJ@ -I${includedir}/efreet-@VMAJ@ -I${includedir}/edje-@VMAJ@ -I${includedir}/eio-@VMAJ@ -I${includedir}/eldbus-@VMAJ@ -I${includedir}/ecore-@VMAJ@ -I${includedir}/efl-@VMAJ@ -I${includedir}/eo-@VMAJ@ -I${includedir}/eet-@VMAJ@ -I${includedir}/emile-@VMAJ@ -I${includedir}/eina-@VMAJ@ diff --git a/src/Makefile_Elementary.am b/src/Makefile_Elementary.am index 95d1cf8976..4c79dc8007 100644 --- a/src/Makefile_Elementary.am +++ b/src/Makefile_Elementary.am @@ -324,9 +324,7 @@ includesunstable_HEADERS = \ lib/elementary/elm_code_indent.h \ lib/elementary/elm_code_file.h \ lib/elementary/elm_code_parse.h \ - lib/elementary/elm_code_syntax.h \ - lib/elementary/efl_ui_multibuttonentry.h \ - lib/elementary/Efl_Ui.h + lib/elementary/elm_code_syntax.h includesunstabledir = $(includedir)/elementary-@VMAJ@ nodist_includesunstable_HEADERS = \ diff --git a/src/lib/ecore/Efl_Core.h b/src/lib/ecore/Efl_Core.h index 8981d4862d..585f847b55 100644 --- a/src/lib/ecore/Efl_Core.h +++ b/src/lib/ecore/Efl_Core.h @@ -64,7 +64,9 @@ extern "C" { extern EAPI double _efl_startup_time; #include "Ecore_Common.h" -#include "Ecore_Eo.h" +#ifdef EFL_OBJECT_BETA +# include "Ecore_Eo.h" +#endif #ifdef __cplusplus } diff --git a/src/lib/ecore/efl_general.h b/src/lib/ecore/efl_general.h index 3a79e46d33..c91f61fce2 100644 --- a/src/lib/ecore/efl_general.h +++ b/src/lib/ecore/efl_general.h @@ -13,23 +13,6 @@ #undef __EFL_NET #undef EFL_MAIN #undef EFL_MAIN_EX -#undef EAPI_MAIN - -#ifdef _WIN32 -// There is no support for quicklaunch on windows, so no needs -// to export the efl_main symbol -# define EAPI_MAIN -#else -# ifdef __GNUC__ -# if __GNUC__ >= 4 -# define EAPI_MAIN __attribute__ ((visibility("default"))) -# else -# define EAPI_MAIN -# endif -# else -# define EAPI_MAIN -# endif -#endif /* ! _WIN32 */ #ifdef EFL_VERSION_MICRO # define _EFL_VERSION_MICRO EFL_VERSION_MICRO diff --git a/src/lib/elementary/Efl_Ui.h b/src/lib/elementary/Efl_Ui.h deleted file mode 100644 index 75c3e3cbae..0000000000 --- a/src/lib/elementary/Efl_Ui.h +++ /dev/null @@ -1,253 +0,0 @@ -#ifndef _EFL_CORE_H -#define _EFL_CORE_H - -#include -#include - -#ifdef _MSC_VER -# include -#endif - -/* Standard headers for standard system calls etc. */ -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#ifdef ELM_DIRENT_H -#include -#endif -#include -#include - -#ifdef ELM_UNIX -#include -#ifdef ELM_LIBINTL_H -#include -#endif -#include -#include -#include -#endif - -#ifdef ELM_ALLOCA_H -#include -#endif - -#ifdef ELM_WIN32 -#include -#ifndef alloca -#define alloca _alloca -#endif -#endif - -#include -#include -#include - -#ifdef EAPI -# undef EAPI -#endif - -#ifdef _WIN32 -# ifdef EFL_ECORE_BUILD -# ifdef DLL_EXPORT -# define EAPI __declspec(dllexport) -# else -# define EAPI -# endif /* ! DLL_EXPORT */ -# else -# define EAPI __declspec(dllimport) -# endif /* ! EFL_ECORE_BUILD */ -#else -# ifdef __GNUC__ -# if __GNUC__ >= 4 -# define EAPI __attribute__ ((visibility("default"))) -# else -# define EAPI -# endif -# else -# define EAPI -# endif -#endif /* ! _WIN32 */ - -#ifdef _WIN32 -# define WIN32_LEAN_AND_MEAN -# include -# undef WIN32_LEAN_AND_MEAN -#elif defined (__FreeBSD__) || defined (__OpenBSD__) -# include -# include -#elif defined (__ANDROID__) -# include -#elif defined (__sun) -# include -#else -# include -# if !defined (EXOTIC_NO_SIGNAL) -# include -# endif -#endif - -#include - -#define __EFL_UI_IS_REQUIRED - -#include - -#include -#include -#include -#include -#include -#include -#include -#include -#include - -#ifdef ELM_ELOCATION -#include -#endif - -#ifdef ELM_EMAP -#include -#endif - -#ifdef __cplusplus -extern "C" { -#endif - -extern EAPI double _efl_startup_time; - -// EO types. Defined for legacy-only builds as legacy uses typedef of EO types. -#include "efl_ui.eot.h" - -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include - -# include -# include -# include -# include -# include -# include - -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include -# include - -/** - * Initialize Elementary - * - * @param[in] argc System's argument count value - * @param[in] argv System's pointer to array of argument strings - * @return The init counter value. - * - * This function initializes Elementary and increments a counter of - * the number of calls to it. It returns the new counter's value. - * - * @warning This call is exported only for use by the @c ELM_MAIN() - * macro. There is no need to use this if you use this macro (which - * is highly advisable). An elm_main() should contain the entry - * point code for your application, having the same prototype as - * elm_init(), and @b not being static (putting the @c EAPI_MAIN symbol - * in front of its type declaration is advisable). The @c - * ELM_MAIN() call should be placed just after it. - * - * Example: - * @dontinclude bg_example_01.c - * @skip static void - * @until ELM_MAIN - * - * See the full @ref bg_example_01_c "example". - * - * @see elm_shutdown(). - * @ingroup Elm_General - */ -EAPI int elm_init(int argc, char **argv); - -/** - * Shut down Elementary - * - * @return The init counter value. - * - * This should be called at the end of your application, just - * before it ceases to do any more processing. This will clean up - * any permanent resources your application may have allocated via - * Elementary that would otherwise persist. - * - * @see elm_init() for an example - * - * @note elm_shutdown() will iterate main loop until all ecore_evas are freed. - * There is a possibility to call your ecore callbacks(timer, animator, event, - * job, and etc.) in elm_shutdown() - * - * @ingroup Elm_General - */ -EAPI int elm_shutdown(void); - -#ifdef __cplusplus -} -#endif - -#undef EAPI -#define EAPI - -#endif - -// We are including efl_general.h again, just in case Efl_Core.h was already included before this -// and the __EFL_*_IS_REQUIRED changed since then. -#include diff --git a/src/lib/elementary/Elementary.h b/src/lib/elementary/Elementary.h index bf41fdea47..3913084bf3 100644 --- a/src/lib/elementary/Elementary.h +++ b/src/lib/elementary/Elementary.h @@ -85,6 +85,7 @@ #endif #ifdef _WIN32 +# define EAPI_MAIN # ifdef ELEMENTARY_BUILD # ifdef DLL_EXPORT # define EAPI __declspec(dllexport) @@ -98,11 +99,14 @@ # ifdef __GNUC__ # if __GNUC__ >= 4 # define EAPI __attribute__ ((visibility("default"))) +# define EAPI_MAIN __attribute__ ((visibility("default"))) # else # define EAPI +# define EAPI_MAIN # endif # else # define EAPI +# define EAPI_MAIN # endif #endif /* ! _WIN32 */ diff --git a/src/lib/elementary/efl_ui_multibuttonentry.h b/src/lib/elementary/efl_ui_multibuttonentry.h index 3d9a75912d..3c76aa76c7 100644 --- a/src/lib/elementary/efl_ui_multibuttonentry.h +++ b/src/lib/elementary/efl_ui_multibuttonentry.h @@ -57,6 +57,7 @@ * @{ */ +#include "efl_ui_multibuttonentry_common.h" #ifdef EFL_EO_API_SUPPORT #include "efl_ui_multibuttonentry_eo.h" #endif diff --git a/src/lib/elementary/elm_general.h b/src/lib/elementary/elm_general.h index 7564f1217c..d718594f8a 100644 --- a/src/lib/elementary/elm_general.h +++ b/src/lib/elementary/elm_general.h @@ -96,7 +96,9 @@ extern EAPI double _elm_startup_time; #define ELM_MAIN() int main(int argc, char **argv) { int ret__; _elm_startup_time = ecore_time_unix_get(); ret__ = elm_quicklaunch_fallback(argc, argv); elm_shutdown(); return ret__; } #endif -#include "Efl_Ui.h" +#define __EFL_UI_IS_REQUIRED + +#include "Efl_Core.h" /**************************************************************************/ /* General calls */ diff --git a/src/modules/ecore/system/upower/ecore_system_upower.c b/src/modules/ecore/system/upower/ecore_system_upower.c index 8a73ac44ac..abc621ff7a 100644 --- a/src/modules/ecore/system/upower/ecore_system_upower.c +++ b/src/modules/ecore/system/upower/ecore_system_upower.c @@ -149,8 +149,7 @@ _on_low_battery_get_cb(void *data EINA_UNUSED, const Eldbus_Message *msg, _eldbus_pending = eina_list_remove(_eldbus_pending, pending); if (eldbus_message_error_get(msg, &errname, &errmsg)) { - if (strcmp(errname, "org.enlightenment.DBus.Canceled")) - ERR("Message error %s - %s", errname, errmsg); + ERR("Message error %s - %s", errname, errmsg); return; } if (!eldbus_message_arguments_get(msg, "v", &variant)) @@ -198,8 +197,7 @@ _on_battery_get_cb(void *data EINA_UNUSED, const Eldbus_Message *msg, _eldbus_pending = eina_list_remove(_eldbus_pending, pending); if (eldbus_message_error_get(msg, &errname, &errmsg)) { - if (strcmp(errname, "org.enlightenment.DBus.Canceled")) - ERR("Message error %s - %s", errname, errmsg); + ERR("Message error %s - %s", errname, errmsg); return; } if (!eldbus_message_arguments_get(msg, "v", &variant)) @@ -287,8 +285,7 @@ _daemon_version_get_cb(void *data EINA_UNUSED, const Eldbus_Message *msg, _eldbus_pending = eina_list_remove(_eldbus_pending, pending); if (eldbus_message_error_get(msg, &errname, &errmsg)) { - if (strcmp(errname, "org.enlightenment.DBus.Canceled")) - ERR("Message error %s - %s", errname, errmsg); + ERR("Message error %s - %s", errname, errmsg); return; } if (!eldbus_message_arguments_get(msg, "v", &variant))