On 12/26/2012 08:18 PM, Saul Wold wrote:
On 12/24/2012 12:34 AM, Constantin Musca wrote:
fix-includedir-ln.patch: added
- fix infinite link recursion by specifying the absolute path
- fix make parallelism issues

libfm_fix_for_automake-1.12.patch: removed
- problem not reprodicible anymore

Signed-off-by: Constantin Musca <[email protected]>
---
.../libfm-0.1.17/libfm_fix_for_automake-1.12.patch | 48 ----------------------
  .../configure_fix.patch                            |  0
If this patch is not being used any more, then it should be removed.

.../libfm/libfm-1.1.0/fix-includedir-ln.patch | 45 ++++++++++++++++++++
What is the intent of this, it seems to be part of the cause of the pcmanfm failure.

See below

Sau!

  .../libfm/{libfm_0.1.17.bb => libfm_1.1.0.bb}      |  8 ++--
  4 files changed, 49 insertions(+), 52 deletions(-)
delete mode 100644 meta/recipes-support/libfm/libfm-0.1.17/libfm_fix_for_automake-1.12.patch rename meta/recipes-support/libfm/{libfm-0.1.17 => libfm-1.1.0}/configure_fix.patch (100%) create mode 100644 meta/recipes-support/libfm/libfm-1.1.0/fix-includedir-ln.patch rename meta/recipes-support/libfm/{libfm_0.1.17.bb => libfm_1.1.0.bb} (75%)

diff --git a/meta/recipes-support/libfm/libfm-0.1.17/libfm_fix_for_automake-1.12.patch b/meta/recipes-support/libfm/libfm-0.1.17/libfm_fix_for_automake-1.12.patch
deleted file mode 100644
index 3efb4dd..0000000
--- a/meta/recipes-support/libfm/libfm-0.1.17/libfm_fix_for_automake-1.12.patch
+++ /dev/null
@@ -1,48 +0,0 @@
-Upstream-Status: Pending
-
-This patch fixes following kind of issues with automake 1.12
-| automake: warnings are treated as errors
-| data/ui/Makefile.am:21: warning: '%'-style pattern rules are a GNU make extension
-...
-| /srv/home/nitin/builds/build-gcc47/tmp/sysroots/x86_64-linux/usr/share/automake-1.12/am/ltlibrary.am: warning: 'libfm-gtk.la': linking libtool libraries using a non-POSIX -| /srv/home/nitin/builds/build-gcc47/tmp/sysroots/x86_64-linux/usr/share/automake-1.12/am/ltlibrary.am: archiver requires 'AM_PROG_AR' in 'configure.ac'
-...
-| src/Makefile.am:21: warning: compiling 'glib-compat.c' with per-target flags requires 'AM_PROG_CC_C_O' in 'configure.ac'
-
-Signed-Off-By: Nitin A Kamble <[email protected]>
-2012/05/03
-
-Index: libfm-0.1.17/configure.ac
-===================================================================
---- libfm-0.1.17.orig/configure.ac
-+++ libfm-0.1.17/configure.ac
-@@ -4,11 +4,17 @@
- AC_PREREQ([2.63])
- AC_INIT([libfm], [0.1.17], [http://pcmanfm.sorceforge.net/])
- AM_INIT_AUTOMAKE([-Wall -Werror foreign])
-+
-+# automake 1.12 seems to require this, but automake 1.11 doesn't recognize it
-+m4_pattern_allow([AM_PROG_AR])
-+AM_PROG_AR
-+
- AC_CONFIG_MACRO_DIR(m4)
- AC_CONFIG_HEADERS([config.h])
-
- # Checks for programs.
- AC_PROG_CC
-+AM_PROG_CC_C_O
- AM_PROG_LIBTOOL
-
- # Checks for libraries.
-Index: libfm-0.1.17/data/ui/Makefile.am
-===================================================================
---- libfm-0.1.17.orig/data/ui/Makefile.am
-+++ libfm-0.1.17/data/ui/Makefile.am
-@@ -18,6 +18,6 @@ EXTRA_DIST = \
-     $(NULL)
-
- # Purge GtkBuilder UI files
--%.ui: %.glade
-+.glade.ui:
-     cp $< $@
-     $(top_builddir)/src/xml-purge $@
diff --git a/meta/recipes-support/libfm/libfm-0.1.17/configure_fix.patch b/meta/recipes-support/libfm/libfm-1.1.0/configure_fix.patch
similarity index 100%
rename from meta/recipes-support/libfm/libfm-0.1.17/configure_fix.patch
rename to meta/recipes-support/libfm/libfm-1.1.0/configure_fix.patch
diff --git a/meta/recipes-support/libfm/libfm-1.1.0/fix-includedir-ln.patch b/meta/recipes-support/libfm/libfm-1.1.0/fix-includedir-ln.patch
new file mode 100644
index 0000000..3f06278
--- /dev/null
+++ b/meta/recipes-support/libfm/libfm-1.1.0/fix-includedir-ln.patch
@@ -0,0 +1,45 @@
+Fix infinite link recursion by specifying the absolute path
+Fix make parallelism issues
+
+Upstream-Status: Pending
+Signed-off-by: Constantin Musca <[email protected]>
+
+Index: libfm-1.1.0/src/Makefile.in
+===================================================================
+--- libfm-1.1.0.orig/src/Makefile.in
++++ libfm-1.1.0/src/Makefile.in
+@@ -2523,8 +2523,6 @@ info: info-recursive
+ info-am:
+
+ install-data-am: install-data-local install-giomodulesLTLIBRARIES \
+-    install-libfmgtkincludeHEADERS install-libfmincludeHEADERS \
+-    install-pkgincludeHEADERS
+
+ install-dvi: install-dvi-recursive
+
+@@ -2617,7 +2615,9 @@ gtk/fm-gtk-marshal.h: gtk/fm-gtk-marshal
+ gtk/fm-gtk-marshal.c: gtk/fm-gtk-marshal.list
+ glib-genmarshal --body --prefix=fm_marshal $? > [email protected] && mv [email protected] $@
+
+-install-data-local:
++install-data-local: install-libfmgtkincludeHEADERS install-libfmincludeHEADERS \
++    install-pkgincludeHEADERS
++
+     @if test -e "$(DESTDIR)$(includedir)/@PACKAGE@"; then \
+         echo rm -rf "$(DESTDIR)$(includedir)/@PACKAGE@"; \
+         rm -rf "$(DESTDIR)$(includedir)/@PACKAGE@"; \
+Index: libfm-1.1.0/src/Makefile.am
+===================================================================
+--- libfm-1.1.0.orig/src/Makefile.am
++++ libfm-1.1.0/src/Makefile.am
+@@ -453,8 +453,8 @@ install-data-local:
+         echo rm -rf "$(DESTDIR)$(includedir)/@PACKAGE@"; \
+         rm -rf "$(DESTDIR)$(includedir)/@PACKAGE@"; \
+     fi
+-    test -z "$(includedir)" || $(MKDIR_P) "$(DESTDIR)$(includedir)"
+-    @LN_S@ @PACKAGE@-@FMLIBVER@ "$(DESTDIR)$(includedir)/@PACKAGE@"
++    rm -rf $(DESTDIR)$(includedir)/@PACKAGE@
++ @LN_S@ $(DESTDIR)$(includedir)/@PACKAGE@-@FMLIBVER@ $(DESTDIR)$(includedir)/@PACKAGE@
+
this does not look right and I think produces the following:

tmp/sysroots/qemuarm/usr/include/libfm:
total 4
lrwxrwxrwx 1 sgw sgw 98 Dec 25 22:55 libfm-1.0 -> /intel/builds/world/tmp/work/armv5te-poky-linux-gnueabi/libfm/1.1.0-r0/image/usr/include/libfm-1.0

Notice linking back into the image dir, that's not correct.


+ uninstall-local:
+     @if test -L "$(DESTDIR)$(includedir)/@PACKAGE@"; then \
diff --git a/meta/recipes-support/libfm/libfm_0.1.17.bb b/meta/recipes-support/libfm/libfm_1.1.0.bb
similarity index 75%
rename from meta/recipes-support/libfm/libfm_0.1.17.bb
rename to meta/recipes-support/libfm/libfm_1.1.0.bb
index 2cac361..7282871 100644
--- a/meta/recipes-support/libfm/libfm_0.1.17.bb
+++ b/meta/recipes-support/libfm/libfm_1.1.0.bb
@@ -10,14 +10,14 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
  SECTION = "x11/libs"
  DEPENDS = "glib-2.0 pango gtk+ menu-cache intltool-native"

-PR = "r4"
+PR = "r0"

  SRC_URI = "${SOURCEFORGE_MIRROR}/pcmanfm/libfm-${PV}.tar.gz \
-           file://libfm_fix_for_automake-1.12.patch \
+           file://fix-includedir-ln.patch \
             "

-SRC_URI[md5sum] = "a97e03d973e6ac727f28d0934d6c9ad5"
-SRC_URI[sha256sum] = "1740681cff4cd4c5a2eaa9805d8898269cfb6a49a0bda0acb242def15bc7131b"
+SRC_URI[md5sum] = "a5bc8b8291cf810c659bfb3af378b5de"
+SRC_URI[sha256sum] = "b9426e588670b53570b808c49abd1d103863614dd3622559b8c3ef1392fe0b3d"

  inherit autotools pkgconfig


I managed to reproduce the problem after several builds. The libfm dir is created before the link is made and that's why the link is incorrect. I have sent patch v2 which fixes the problem by creating a relative symbolic link after do_install.

Cheers,
Constantin

_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to