All of lore.kernel.org
 help / color / mirror / Atom feed
From: Saul Wold <sgw@linux.intel.com>
To: Constantin Musca <constantinx.musca@intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 14/15] pcmanfm: upgrade to 1.1.0
Date: Wed, 26 Dec 2012 08:17:03 -0800	[thread overview]
Message-ID: <50DB22FF.70308@linux.intel.com> (raw)
In-Reply-To: <076896b6989230187ea8fe8e7c03a4c28f4bc568.1356337650.git.constantinx.musca@intel.com>

On 12/24/2012 12:34 AM, Constantin Musca wrote:
> pcmanfm_fix_for_automake_1.12.patch: removed
> - problem not reprodicible anymore
>
> owl-window-menu.patch: removed
> - not needed anymore
>
> Signed-off-by: Constantin Musca <constantinx.musca@intel.com>
> ---
>   .../pcmanfm/files/owl-window-menu.patch            | 81 ----------------------
>   .../files/pcmanfm_fix_for_automake_1.12.patch      | 35 ----------
>   .../{pcmanfm_0.9.10.bb => pcmanfm_1.1.0.bb}        | 11 ++-
>   3 files changed, 4 insertions(+), 123 deletions(-)
>   delete mode 100644 meta/recipes-sato/pcmanfm/files/owl-window-menu.patch
>   delete mode 100644 meta/recipes-sato/pcmanfm/files/pcmanfm_fix_for_automake_1.12.patch
>   rename meta/recipes-sato/pcmanfm/{pcmanfm_0.9.10.bb => pcmanfm_1.1.0.bb} (76%)
>

Seems to be an issue with this when I built it for ARM:
> arm-poky-linux-gnueabi-gcc  -march=armv5te  -marm -mthumb-interwork   --sysroot=/intel/builds/world/tmp/sysroots/qemuarm -DHAVE_CONFIG_H -I. -I..  -DPACKAGE_DATA_DIR=\""/usr/share/pcmanfm"\" -DPACKAGE_UI_DIR=\""/usr/share/pcmanfm/ui"\" -DPACKAGE_LOCALE_DIR=\""/usr/share/locale"\" -D_LARGEFILE_SOURCE -D_REENTRANT -D_FILE_OFFSET_BITS=64 -DG_DISABLE_ASSERT -DG_DISABLE_CHECKS  -pthread -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/glib-2.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/lib/glib-2.0/include -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/gio-unix-2.0/ -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/pango-1.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/gtk-2.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/lib/gtk-2.0/include -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/atk-1.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/cairo -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/gdk-pixbuf-2.0 -I/intel/builds/world/
tmp/sysroots/qemuarm/usr/include/pixman-1 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/freetype2 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/libpng15   -DG_DISABLE_CAST_CHECKS -Wall -Werror-implicit-function-declaration  -O2 -pipe -g -feliminate-unused-debug-types -c -o pcmanfm-pref.o `test -f 'pref.c' || echo './'`pref.c
> arm-poky-linux-gnueabi-gcc  -march=armv5te  -marm -mthumb-interwork   --sysroot=/intel/builds/world/tmp/sysroots/qemuarm -DHAVE_CONFIG_H -I. -I..  -DPACKAGE_DATA_DIR=\""/usr/share/pcmanfm"\" -DPACKAGE_UI_DIR=\""/usr/share/pcmanfm/ui"\" -DPACKAGE_LOCALE_DIR=\""/usr/share/locale"\" -D_LARGEFILE_SOURCE -D_REENTRANT -D_FILE_OFFSET_BITS=64 -DG_DISABLE_ASSERT -DG_DISABLE_CHECKS  -pthread -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/glib-2.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/lib/glib-2.0/include -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/gio-unix-2.0/ -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/pango-1.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/gtk-2.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/lib/gtk-2.0/include -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/atk-1.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/cairo -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/gdk-pixbuf-2.0 -I/intel/builds/world/
tmp/sysroots/qemuarm/usr/include/pixman-1 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/freetype2 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/libpng15   -DG_DISABLE_CAST_CHECKS -Wall -Werror-implicit-function-declaration  -O2 -pipe -g -feliminate-unused-debug-types -c -o pcmanfm-utils.o `test -f 'utils.c' || echo './'`utils.c
> tab-page.c:24:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> app-config.c:26:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-app-config.o] Error 1
> make[2]: *** Waiting for unfinished jobs....
> make[2]: *** [pcmanfm-tab-page.o] Error 1
> pref.c:26:22: fatal error: libfm/fm.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-pref.o] Error 1
> volume-manager.c:27:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-volume-manager.o] Error 1
> In file included from utils.c:22:0:
> utils.h:26:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-utils.o] Error 1
> In file included from desktop.c:27:0:
> desktop.h:27:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-desktop.o] Error 1
> In file included from main-win.c:34:0:
> pcmanfm.h:26:22: fatal error: libfm/fm.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-main-win.o] Error 1
> pcmanfm.c:39:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-pcmanfm.o] Error 1
> make[2]: Leaving directory `/intel/builds/world/tmp/work/armv5te-poky-linux-gnueabi/pcmanfm/1.1.0-r0/pcmanfm-1.1.0/src'
> make[1]: *** [all-recursive] Error 1
> make[1]: Leaving directory `/intel/builds/world/tmp/work/armv5te-poky-linux-gnueabi/pcmanfm/1.1.0-r0/pcmanfm-1.1.0'
> make: *** [all] Error 2
> ERROR: oe_runmake failed
> ERROR: Function failed: do_compile (see /intel/builds/world/tmp/work/armv5te-poky-linux-gnueabi/pcmanfm/1.1.0-r0/temp/log.do_compile.5430 for further information)
> sgw@yujin:/intel/builds/world$
> d/tmp/sysroots/qemuarm/usr/include/atk-1.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/cairo -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/gdk-pixbuf-2.0 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/pixman-1 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/freetype2 -I/intel/builds/world/tmp/sysroots/qemuarm/usr/include/libpng15   -DG_DISABLE_CAST_CHECKS -Wall -Werror-implicit-function-declaration  -O2 -pipe -g -feliminate-unused-debug-types -c -o pcmanfm-utils.o `test -f 'utils.c' || echo './'`utils.c
> tab-page.c:24:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> app-config.c:26:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-app-config.o] Error 1
> make[2]: *** Waiting for unfinished jobs....
> make[2]: *** [pcmanfm-tab-page.o] Error 1
> pref.c:26:22: fatal error: libfm/fm.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-pref.o] Error 1
> volume-manager.c:27:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-volume-manager.o] Error 1
> In file included from utils.c:22:0:
> utils.h:26:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-utils.o] Error 1
> In file included from desktop.c:27:0:
> desktop.h:27:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-desktop.o] Error 1
> In file included from main-win.c:34:0:
> pcmanfm.h:26:22: fatal error: libfm/fm.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-main-win.o] Error 1
> pcmanfm.c:39:26: fatal error: libfm/fm-gtk.h: No such file or directory
> compilation terminated.
> make[2]: *** [pcmanfm-pcmanfm.o] Error 1
> make[2]: Leaving directory `/intel/builds/world/tmp/work/armv5te-poky-linux-gnueabi/pcmanfm/1.1.0-r0/pcmanfm-1.1.0/src'
> make[1]: *** [all-recursive] Error 1
> make[1]: Leaving directory `/intel/builds/world/tmp/work/armv5te-poky-linux-gnueabi/pcmanfm/1.1.0-r0/pcmanfm-1.1.0'
> make: *** [all] Error 2
> ERROR: oe_runmake failed
> ERROR: Function failed: do_compile (see /intel/builds/world/tmp/work/armv5te-poky-linux-gnueabi/pcmanfm/1.1.0-r0/temp/log.do_compile.5430 for further information)


I will dig a little later this morning.

Sau!


> diff --git a/meta/recipes-sato/pcmanfm/files/owl-window-menu.patch b/meta/recipes-sato/pcmanfm/files/owl-window-menu.patch
> deleted file mode 100644
> index 88b154e..0000000
> --- a/meta/recipes-sato/pcmanfm/files/owl-window-menu.patch
> +++ /dev/null
> @@ -1,81 +0,0 @@
> -0.9.7 use a new menu bar from gtk_ui_manager, so need a translation here to
> -enable owl menu
> -
> -Signed-off-by: Zhai Edwin <edwin.zhai@intel.com>
> -
> -Upstream-Status: Inappropriate [enable feature]
> -
> -Index: pcmanfm-0.9.9/configure.ac
> -===================================================================
> ---- pcmanfm-0.9.9.orig/configure.ac	2011-08-10 05:28:31.000000000 +0800
> -+++ pcmanfm-0.9.9/configure.ac	2011-10-10 15:47:33.000000000 +0800
> -@@ -70,6 +70,8 @@
> -
> - gtk_modules="gtk+-2.0 >= 2.16.0"
> - PKG_CHECK_MODULES(GTK, [$gtk_modules])
> -+OWL_LIBS="-lowl"
> -+GTK_LIBS="$GTK_LIBS $OWL_LIBS"
> - AC_SUBST(GTK_CFLAGS)
> - AC_SUBST(GTK_LIBS)
> -
> -Index: pcmanfm-0.9.9/src/main-win.c
> -===================================================================
> ---- pcmanfm-0.9.9.orig/src/main-win.c	2011-08-10 05:28:32.000000000 +0800
> -+++ pcmanfm-0.9.9/src/main-win.c	2011-10-10 15:50:48.000000000 +0800
> -@@ -37,6 +37,8 @@
> - #include "pref.h"
> - #include "tab-page.h"
> -
> -+#include "libowl/owlwindowmenu.h"
> -+
> - static void fm_main_win_finalize              (GObject *object);
> - G_DEFINE_TYPE(FmMainWin, fm_main_win, GTK_TYPE_WINDOW);
> -
> -@@ -400,9 +402,27 @@
> -     }
> - }
> -
> -+/* Transfer gtk_ui_manager menubar to gtk menubar */
> -+static GtkWidget * ui_menubar_to_menu(GtkWidget *menubar)
> -+{
> -+    GtkWidget *main_menu;
> -+    GList *iter;
> -+
> -+    main_menu = gtk_menu_new();
> -+
> -+    for ( iter = gtk_container_get_children(GTK_CONTAINER(menubar));
> -+          iter;
> -+          iter = g_list_next(iter) )
> -+    {
> -+        GtkWidget *menu = GTK_WIDGET(iter->data);
> -+        gtk_widget_reparent(menu, main_menu);
> -+    }
> -+    return main_menu;
> -+}
> -+
> - static void fm_main_win_init(FmMainWin *win)
> - {
> --    GtkWidget *vbox, *menubar, *toolitem, *btn;
> -+    GtkWidget *vbox, *menubar, *owl_menubar, *toolitem, *btn;
> -     GtkUIManager* ui;
> -     GtkActionGroup* act_grp;
> -     GtkAction* act;
> -@@ -456,7 +476,6 @@
> -     win->popup = gtk_ui_manager_get_widget(ui, "/popup");
> -     gtk_menu_attach_to_widget(GTK_WIDGET(win->popup), win, NULL);
> -
> --    gtk_box_pack_start( (GtkBox*)vbox, menubar, FALSE, TRUE, 0 );
> -     gtk_box_pack_start( (GtkBox*)vbox, win->toolbar, FALSE, TRUE, 0 );
> -
> -     /* load bookmarks menu */
> -@@ -517,6 +536,10 @@
> -
> -     gtk_container_add( (GtkContainer*)win, vbox );
> -     gtk_widget_show_all(vbox);
> -+
> -+    owl_menubar = ui_menubar_to_menu(menubar);
> -+    gtk_widget_show_all(owl_menubar);
> -+    owl_set_window_menu(GTK_WINDOW(win), GTK_MENU(owl_menubar));
> - }
> -
> -
> diff --git a/meta/recipes-sato/pcmanfm/files/pcmanfm_fix_for_automake_1.12.patch b/meta/recipes-sato/pcmanfm/files/pcmanfm_fix_for_automake_1.12.patch
> deleted file mode 100644
> index 1ad0ff5..0000000
> --- a/meta/recipes-sato/pcmanfm/files/pcmanfm_fix_for_automake_1.12.patch
> +++ /dev/null
> @@ -1,35 +0,0 @@
> -Upstream-Status: Pending
> -
> -Fix these errors:
> -| automake: warnings are treated as errors
> -| data/ui/Makefile.am:18: warning: '%'-style pattern rules are a GNU make extension
> -| src/Makefile.am:10: warning: compiling 'pcmanfm.c' with per-target flags requires 'AM_PROG_CC_C_O' in 'configure.ac'
> -| autoreconf: automake failed with exit status: 1
> -| ERROR: autoreconf execution failed.
> -Signed-Off-By: Nitin A Kamble <nitin.a.kamble@Intel.com>
> -2012/07/09
> -
> -Index: pcmanfm-0.9.10/configure.ac
> -===================================================================
> ---- pcmanfm-0.9.10.orig/configure.ac
> -+++ pcmanfm-0.9.10/configure.ac
> -@@ -13,6 +13,7 @@ AC_PROG_CC
> - AM_PROG_LIBTOOL
> - AM_PROG_CC_STDC
> - AC_HEADER_STDC
> -+AM_PROG_CC_C_O
> -
> - # Checks for header files.
> - AC_HEADER_STDC
> -Index: pcmanfm-0.9.10/data/ui/Makefile.am
> -===================================================================
> ---- pcmanfm-0.9.10.orig/data/ui/Makefile.am
> -+++ pcmanfm-0.9.10/data/ui/Makefile.am
> -@@ -15,6 +15,6 @@ EXTRA_DIST = \
> - 	$(NULL)
> -
> - # Purge GtkBuilder UI files
> --%.ui: %.glade
> -+.glade.ui:
> - 	cp $< $@
> - #	$(top_builddir)/src/xml-purge $@
> diff --git a/meta/recipes-sato/pcmanfm/pcmanfm_0.9.10.bb b/meta/recipes-sato/pcmanfm/pcmanfm_1.1.0.bb
> similarity index 76%
> rename from meta/recipes-sato/pcmanfm/pcmanfm_0.9.10.bb
> rename to meta/recipes-sato/pcmanfm/pcmanfm_1.1.0.bb
> index 1780764..bcfc9db 100644
> --- a/meta/recipes-sato/pcmanfm/pcmanfm_0.9.10.bb
> +++ b/meta/recipes-sato/pcmanfm/pcmanfm_1.1.0.bb
> @@ -11,7 +11,7 @@ SECTION = "x11"
>   DEPENDS = "gtk+ startup-notification libfm"
>   DEPENDS_append_poky = " libowl"
>
> -PR = "r1"
> +PR = "r0"
>
>   COMPATIBLE_HOST = '(x86_64.*|i.86.*|aarch64.*|arm.*|mips.*|powerpc.*|sh.*)-(linux|freebsd.*)'
>
> @@ -20,13 +20,10 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/pcmanfm/pcmanfm-${PV}.tar.gz \
>   	   file://gnome-fs-regular.png \
>   	   file://gnome-mime-text-plain.png \
>   	   file://emblem-symbolic-link.png \
> -	   file://cross-compile-fix.patch \
> -           file://pcmanfm_fix_for_automake_1.12.patch"
> +	   file://cross-compile-fix.patch"
>
> -SRC_URI_append_poky = " file://owl-window-menu.patch"
> -
> -SRC_URI[md5sum] = "d34a3530a6c5dcd674d23021d71c3e95"
> -SRC_URI[sha256sum] = "f133c6f207f719d1fc69fe8bc07b2de6883c6937ffa87448df42e3b1a30e0298"
> +SRC_URI[md5sum] = "af0cff78690e658f3c06ceabf27bc71a"
> +SRC_URI[sha256sum] = "1f6301f330ad648f3322708ec6c0f680a8695a9453932fe19653bab6731e5582"
>
>   inherit autotools pkgconfig
>
>



  reply	other threads:[~2012-12-26 16:32 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-24  8:34 [PATCH 00/15] Package upgrades Constantin Musca
2012-12-24  8:34 ` [PATCH 01/15] ethtool: upgrade to 3.7 Constantin Musca
2012-12-24  8:34 ` [PATCH 02/15] freetype: upgrade to 2.4.11 Constantin Musca
2012-12-24  8:34 ` [PATCH 03/15] gnome-doc-utils: upgrade to 0.20.10 Constantin Musca
2012-12-24  8:34 ` [PATCH 04/15] gsettings-desktop-schemas: add recipe Constantin Musca
2012-12-24  8:34 ` [PATCH 05/15] gthumb: remove recipe Constantin Musca
2012-12-24  9:10   ` Ross Burton
2012-12-24  8:34 ` [PATCH 06/15] json-glib: upgrade to 0.15.2 Constantin Musca
2012-12-24  8:34 ` [PATCH 07/15] libcroco: upgrade to 0.6.8 Constantin Musca
2012-12-24  9:13   ` Ross Burton
2012-12-24 12:40     ` Constantin Musca
2012-12-24  8:34 ` [PATCH 08/15] libfm: upgrade to 1.1.0 Constantin Musca
2012-12-26 18:18   ` Saul Wold
2012-12-27 14:44     ` Constantin Musca
2012-12-24  8:34 ` [PATCH 09/15] libsoup-2.4: upgrade to 2.40.2 Constantin Musca
2012-12-24  8:34 ` [PATCH 10/15] libuser: upgrade to 0.58 Constantin Musca
2012-12-24  8:34 ` [PATCH 11/15] matchbox-panel-2: fix PV Constantin Musca
2012-12-24  8:34 ` [PATCH 12/15] menu-cache: upgrade to 0.4.1 Constantin Musca
2012-12-24  8:34 ` [PATCH 13/15] metacity: upgrade to 2.34.13 Constantin Musca
2012-12-24  9:16   ` Ross Burton
2012-12-24  9:16     ` [OE-core] " Ross Burton
2012-12-24  8:34 ` [PATCH 14/15] pcmanfm: upgrade to 1.1.0 Constantin Musca
2012-12-26 16:17   ` Saul Wold [this message]
2012-12-24  8:34 ` [PATCH 15/15] shared-mime-info: upgrade to 1.0 Constantin Musca

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50DB22FF.70308@linux.intel.com \
    --to=sgw@linux.intel.com \
    --cc=constantinx.musca@intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.