omxplayer: bump to version 2ee17b2

PATCH New buildroot.org at fln.lt 3 participants 3 messages Dec 1, 2017

lore

Share & tools

lore

buildroot.org at fln.lt

author

2017-12-01 00:02 UTC #0 lore raw

PATCH 1/1 omxplayer: bump to version 2ee17b2

This patch updates omxplayer package to the latest commit on master.

Latest omxplayer version no longer have broken cross-compile support in Makefile.include file, but still contains some hard-coded include paths that needs to be replaced before building.

Signed-off-by Julius Kriukas <[email protected]>

---
 .../0001-Makefiles-clean-up-the-cruft.patch        | 67 ----------------------
 package/omxplayer/0001-cleanup-Makefile.patch      | 18 ++++++
 package/omxplayer/omxplayer.hash                   |  4 +-
 package/omxplayer/omxplayer.mk                     |  9 +--
 4 files changed, 21 insertions(+), 77 deletions(-)
 delete mode 100644 package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch
 create mode 100644 package/omxplayer/0001-cleanup-Makefile.patch

package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch deleted +0 ~~-67~~

deleted file mode 100644
index 2dc6166975..0000000000
@@ -1,67 +0,0 @@
-From 563dafc1129848419482b540d149d0b8687cac1e Mon Sep 17 00:00:00 2001
-From: "Yann E. MORIN" <[email protected]>
-Date: Sun, 10 Apr 2016 16:22:53 +0200
-Subject: [PATCH] Makefiles: clean up the cruft
-
-Most of the variables that Makefile.include tries (but fails) to set,
-are already available from Buildroot's variables:
-  - AR, AS, CC, CXX, OBJDUMP...
-  - CFLAGS, CXXFLAGS, CPPFLAGS...
-
-This leaves us with a few select variables that define include and
-library paths local to the omxplayer package, plus a few optimisations.
-
-Finally, also remove hard-coded, absolute paths pointing to the host
-system (won't work for cross-compilation, so our paranoid wrapper would
-catch those paths).
-
-Signed-off-by: "Yann E. MORIN" <[email protected]>
----
- Makefile         |  2 +-
- Makefile.include | 20 ++------------------
- 2 files changed, 3 insertions(+), 19 deletions(-)
-
-diff --git a/Makefile b/Makefile
-index bcfadfb..3dc1a03 100644
---- a/Makefile
-+++ b/Makefile
-@@ -4,7 +4,7 @@ CFLAGS+=-std=c++0x -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -DTARGET_POSIX
- 
- LDFLAGS+=-L./ -Lffmpeg_compiled/usr/local/lib/ -lc -lWFC -lGLESv2 -lEGL -lbcm_host -lopenmaxil -lfreetype -lz
- 
--INCLUDES+=-I./ -Ilinux -Iffmpeg_compiled/usr/local/include/ -I /usr/include/dbus-1.0 -I /usr/lib/arm-linux-gnueabihf/dbus-1.0/include
-+INCLUDES+=-I./ -Ilinux -Iffmpeg_compiled/usr/local/include/
- 
- DIST ?= omxplayer-dist
- 
-diff --git a/Makefile.include b/Makefile.include
-index 58e9560..79ca277 100644
---- a/Makefile.include
-+++ b/Makefile.include
-@@ -20,21 +20,5 @@ endif
- 
- JOBS=7
- 
--CFLAGS			:= -isystem$(PREFIX)/include
--CXXFLAGS		:= $(CFLAGS)
--CPPFLAGS		:= $(CFLAGS)
--LDFLAGS			:= -L$(BUILDROOT)/lib
--LD					:= $(TOOLCHAIN)/bin/$(HOST)-ld --sysroot=$(SYSROOT)
--CC					:= $(TOOLCHAIN)/bin/$(HOST)-gcc --sysroot=$(SYSROOT)
--CXX         := $(TOOLCHAIN)/bin/$(HOST)-g++ --sysroot=$(SYSROOT)
--OBJDUMP			:= $(TOOLCHAIN)/bin/$(HOST)-objdump
--RANLIB			:= $(TOOLCHAIN)/bin/$(HOST)-ranlib
--STRIP				:= $(TOOLCHAIN)/bin/$(HOST)-strip
--AR 					:= $(TOOLCHAIN)/bin/$(HOST)-ar
--CXXCP 			:= $(CXX) -E
--PATH 				:= $(PREFIX)/bin:$(BUILDROOT)/output/host/usr/bin:$(PATH)
--
--CFLAGS			+= -pipe -mfloat-abi=$(FLOAT) -mcpu=arm1176jzf-s -fomit-frame-pointer -mabi=aapcs-linux -mtune=arm1176jzf-s -mfpu=vfp -Wno-psabi -mno-apcs-stack-check -g -mstructure-size-boundary=32 -mno-sched-prolog
--LDFLAGS			+= -L$(SDKSTAGE)/lib -L$(SDKSTAGE)/usr/lib -L$(SDKSTAGE)/opt/vc/lib/ -Lpcre/build
--#INCLUDES		+= -isystem$(SDKSTAGE)/usr/include -isystem$(SDKSTAGE)/opt/vc/include -isystem$(SYSROOT)/usr/include -isystem$(SDKSTAGE)/opt/vc/include/interface/vcos/pthreads -isystem$(SDKSTAGE)/usr/include/freetype2
--INCLUDES		+= -isystem$(SDKSTAGE)/opt/vc/include -isystem$(SYSROOT)/usr/include -isystem$(SDKSTAGE)/opt/vc/include/interface/vcos/pthreads -Ipcre/build -Iboost-trunk -Ifreetype2/include
-+CFLAGS			+= -fomit-frame-pointer -Wno-psabi -mno-apcs-stack-check    \
-+			   -mstructure-size-boundary=32 -mno-sched-prolog
--- 
-1.9.1
-

package/omxplayer/0001-cleanup-Makefile.patch new file +18 -0

new file mode 100644
index 0000000000..d91e2e4fe9
@@ -0,0 +1,18 @@
+diff --git a/Makefile b/Makefile
+index 5f4e414..53fa1bc 100644
+--- a/Makefile
++++ b/Makefile
+@@ -1,10 +1,10 @@
+-CFLAGS=-pipe -mfloat-abi=hard -mcpu=arm1176jzf-s -fomit-frame-pointer -mabi=aapcs-linux -mtune=arm1176jzf-s -mfpu=vfp -Wno-psabi -mno-apcs-stack-check -g -mstructure-size-boundary=32 -mno-sched-prolog
++CFLAGS+=-fomit-frame-pointer -Wno-psabi -mno-apcs-stack-check -g -mstructure-size-boundary=32 -mno-sched-prolog
+ CFLAGS+=-std=c++0x -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -DTARGET_POSIX -DTARGET_LINUX -fPIC -DPIC -D_REENTRANT -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -DHAVE_CMAKE_CONFIG -D__VIDEOCORE4__ -U_FORTIFY_SOURCE -Wall -DHAVE_OMXLIB -DUSE_EXTERNAL_FFMPEG  -DHAVE_LIBAVCODEC_AVCODEC_H -DHAVE_LIBAVUTIL_OPT_H -DHAVE_LIBAVUTIL_MEM_H -DHAVE_LIBAVUTIL_AVUTIL_H -DHAVE_LIBAVFORMAT_AVFORMAT_H -DHAVE_LIBAVFILTER_AVFILTER_H -DHAVE_LIBSWRESAMPLE_SWRESAMPLE_H -DOMX -DOMX_SKIP64BIT -ftree-vectorize -DUSE_EXTERNAL_OMX -DTARGET_RASPBERRY_PI -DUSE_EXTERNAL_LIBBCM_HOST
+ 
+ LDFLAGS=-L$(SDKSTAGE)/opt/vc/lib/
+-LDFLAGS+=-L./ -Lffmpeg_compiled/usr/local/lib/ -lc -lbrcmGLESv2 -lbrcmEGL -lbcm_host -lopenmaxil -lfreetype -lz -lasound
++LDFLAGS+=-L./ -lc -lbrcmGLESv2 -lbrcmEGL -lbcm_host -lopenmaxil -lfreetype -lz -lasound
+ 
+-INCLUDES+=-I./ -Ilinux -Iffmpeg_compiled/usr/local/include/ -I /usr/include/dbus-1.0 -I /usr/lib/arm-linux-gnueabihf/dbus-1.0/include -I/usr/include/freetype2 -isystem$(SDKSTAGE)/opt/vc/include -isystem$(SDKSTAGE)/opt/vc/include/interface/vcos/pthreads
++INCLUDES+=-I./ -Ilinux
+ 
+ DIST ?= omxplayer-dist
+ STRIP ?= strip

package/omxplayer/omxplayer.hash +2 -2

index 4ee215b767..3715e48488 100644
@@ -1,3 +1,3 @@
 # Locally computed
-sha1    23eb444757e445c127a7114ae233569cac64f984                          omxplayer-6c90c7503ba4658221774759edf7f2ae816711de.tar.gz
-sha256  93ebfd654314231749936e4fc09f5ec4a361806cdb640d7579a5106921e00299  omxplayer-6c90c7503ba4658221774759edf7f2ae816711de.tar.gz
+sha1    258316ff94a1f1e5e47a5bb1a28d324ab08871a9                          omxplayer-2ee17b22a6149a043a2e402580504f282c615373.tar.gz
+sha256  c16498258ee78c59f9705b621e4b5b7b0673e1687d94681f9dc52451b1ceba48  omxplayer-2ee17b22a6149a043a2e402580504f282c615373.tar.gz

package/omxplayer/omxplayer.mk +1 -8

index 9bb58778c2..6e92250438 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-OMXPLAYER_VERSION = 6c90c7503ba4658221774759edf7f2ae816711de
+OMXPLAYER_VERSION = 2ee17b22a6149a043a2e402580504f282c615373
 OMXPLAYER_SITE = $(call github,popcornmix,omxplayer,$(OMXPLAYER_VERSION))
 OMXPLAYER_LICENSE = GPL-2.0+
 OMXPLAYER_LICENSE_FILES = COPYING

@@ -23,14 +23,7 @@ OMXPLAYER_EXTRA_CFLAGS = \
 # procedure is, well, tainted. Fix this by forcing the real,
 # correct values.
 OMXPLAYER_MAKE_ENV = \
-	USE_BUILDROOT=1 \
-	BUILDROOT=$(TOP_DIR) \
 	SDKSTAGE=$(STAGING_DIR) \
-	TARGETFS=$(TARGET_DIR) \
-	TOOLCHAIN=$(HOST_DIR) \
-	HOST=$(GNU_TARGET_NAME) \
-	SYSROOT=$(STAGING_DIR) \
-	JOBS=$(PARALLEL_JOBS) \
 	$(TARGET_CONFIGURE_OPTS) \
 	STRIP=true \
 	CFLAGS="$(TARGET_CFLAGS) $(OMXPLAYER_EXTRA_CFLAGS)"

Signature · 1 line

2.11.0

Yann E. MORIN

2017-12-01 13:21 UTC #1 lore raw

PATCH 1/1 omxplayer: bump to version 2ee17b2

Julius, All,

On 2017-12-01 02:02 +0200, buildroot.org at fln.lt spake thusly:

Quoted 7 lines

This patch updates omxplayer package to the latest commit on master.
Latest omxplayer version no longer have broken cross-compile support in Makefile.include file, but still contains some hard-coded include paths that needs to be replaced before building.
Signed-off-by Julius Kriukas <[email protected]>

Your From adress does not match your SoB line, above. Please use your real name as the From field when sending patches to Buildroot.

Also, we have a script utils/get-developer which returns the list of person interested in specific packages. You can use that script to see who to add in Cc of your patch:

With committed changes:

    $ git show |./utils/get-developers -
    git send-email --to buildroot at buildroot.org --cc "Yann E. MORIN <[email protected]>"

Or manually:

    $ ./utils/get-developers -p omxplayer
    Yann E. MORIN <[email protected]>

;-)

package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch

Quoted 11 lines

---
 .../0001-Makefiles-clean-up-the-cruft.patch        | 67 ----------------------
 package/omxplayer/0001-cleanup-Makefile.patch      | 18 ++++++
 package/omxplayer/omxplayer.hash                   |  4 +-
 package/omxplayer/omxplayer.mk                     |  9 +--
 4 files changed, 21 insertions(+), 77 deletions(-)
 delete mode 100644 package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch
 create mode 100644 package/omxplayer/0001-cleanup-Makefile.patch

package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch deleted +0 -0

deleted file mode 100644

I like it 100% when my patches get dropped. :-)

package/omxplayer/0001-cleanup-Makefile.patch

Quoted 5 lines

package/omxplayer/0001-cleanup-Makefile.patch new file +0 -0
new file mode 100644
index 0000000000..d91e2e4fe9

However, I like it much less when new patches are added, which do not have a commit log and a SoB line for themselves.

Also, since omxplayer is using git, we want to use git-fiormatted patches.

Please see how the patch you are removing was formatted.

Regards,
Yann E. MORIN.

Signature · 6 lines

.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

Julius Kriukas

2017-12-01 17:27 UTC #2 lore raw

PATCH 1/1 omxplayer: bump to version 2ee17b2

Quoted 53 lines · Yann E. MORIN

Julius, All,
On 2017-12-01 02:02 +0200, buildroot.org at fln.lt spake thusly:
Quoted 7 lines
This patch updates omxplayer package to the latest commit on master.
Latest omxplayer version no longer have broken cross-compile support in Makefile.include file, but still contains some hard-coded include paths that needs to be replaced before building.
Signed-off-by Julius Kriukas <[email protected]>
Your From adress does not match your SoB line, above. Please use your real name as the From field when sending patches to Buildroot.
Also, we have a script utils/get-developer which returns the list of person interested in specific packages. You can use that script to see who to add in Cc of your patch:
With committed changes:
    $ git show |./utils/get-developers -
    git send-email --to buildroot at buildroot.org --cc "Yann E. MORIN <[email protected]>"
Or manually:
    $ ./utils/get-developers -p omxplayer
    Yann E. MORIN <[email protected]>
;-)
package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch
Quoted 11 lines
---
 .../0001-Makefiles-clean-up-the-cruft.patch        | 67 ----------------------
 package/omxplayer/0001-cleanup-Makefile.patch      | 18 ++++++
 package/omxplayer/omxplayer.hash                   |  4 +-
 package/omxplayer/omxplayer.mk                     |  9 +--
 4 files changed, 21 insertions(+), 77 deletions(-)
 delete mode 100644 package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch
 create mode 100644 package/omxplayer/0001-cleanup-Makefile.patch
package/omxplayer/0001-Makefiles-clean-up-the-cruft.patch deleted +0 -0
deleted file mode 100644
I like it 100% when my patches get dropped. :-)
package/omxplayer/0001-cleanup-Makefile.patch
Quoted 5 lines
package/omxplayer/0001-cleanup-Makefile.patch new file +0 -0
new file mode 100644
index 0000000000..d91e2e4fe9
However, I like it much less when new patches are added, which do not have a commit log and a SoB line for themselves.

Quoted 8 lines

Also, since omxplayer is using git, we want to use git-fiormatted patches.
Please see how the patch you are removing was formatted.
Regards,
Yann E. MORIN.

Yann, thanks for the feedback and initial omxplayer buildroot package.

I will fix outstanding issues and resubmit.

Signature · 1 line

Julius Kriukas