squeezelite: new package
PATCH v14 1/1 squeezelite: new package
---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generate Makefile.
- change BR2_PACKAGE_SQUEEZELITE_FF -> BR2_PACKAGE_SQUEEZELITE_FFMPEG.
- improve prompt message for configuration options.
- change installation destination path to fullpath.
- remove useless help string.
v12 -> v13
- change SQUEEZELITE_SITE to github (mine), cloned from original google code site (original site is unstable to fetch).
- change to use $(TARGET_CONFIGURE_OPTS) on SQUEEZELITE_BUILD_CMDS to be suitable for buildroot make system.
- to accomplish above, change to generate buildroot suitable Makefile from original squeezelite's Makefile (using 'override' directive for CFLAGS, LDFLAGS).
v11 -> v12
- remove -DRESAMPLE_MP, because libsoxr is compiled without openMP support in buildroot, so meaningless for now.
- add -DLINKALL to resolve symbols in optional libraries (libsoxr, etc).
- add more configuration options : ffmpeg, DSD, Visualizer. lirc support is not included for now, because more work will be necessary to use it (eg: modifying package/lirc-tools to add LIRC_TOOLS_INSTALL_STAGING = YES).
--- package/Config.in | 1 + package/squeezelite/0001-Makefile-override.patch | 42 +++++++++++++++++++++ package/squeezelite/Config.in | 38 +++++++++++++++++++ package/squeezelite/squeezelite.mk | 43 ++++++++++++++++++++++ 4 files changed, 124 insertions(+), 0 deletions(-) create mode 100644 package/squeezelite/0001-Makefile-override.patch create mode 100644 package/squeezelite/Config.in create mode 100644 package/squeezelite/squeezelite.mk
package/Config.in +1 -0
@@ -33,6 +33,7 @@ menu "Audio and video applications" source "package/opus-tools/Config.in" source "package/pulseaudio/Config.in" source "package/sox/Config.in" + source "package/squeezelite/Config.in" source "package/tidsp-binaries/Config.in" source "package/tovid/Config.in" source "package/tstools/Config.in"
package/squeezelite/0001-Makefile-override.patch new file +42 -0
42 lines not on this page · raw message
package/squeezelite/Config.in new file +38 -0
@@ -0,0 +1,38 @@ +config BR2_PACKAGE_SQUEEZELITE + bool "squeezelite" + depends on BR2_USE_WCHAR # flac + depends on BR2_TOOLCHAIN_HAS_THREADS # alsa-lib + depends on BR2_USE_MMU # mpg123 + select BR2_PACKAGE_ALSA_LIB + select BR2_PACKAGE_FLAC + select BR2_PACKAGE_LIBMAD + select BR2_PACKAGE_LIBVORBIS + select BR2_PACKAGE_FAAD2 + select BR2_PACKAGE_MPG123 + help + Logitech Media Server client + + https://code.google.com/p/squeezelite/ + +if BR2_PACKAGE_SQUEEZELITE + +config BR2_PACKAGE_SQUEEZELITE_FFMPEG + bool "Enable WMA and ALAC decoding" + default y + select BR2_PACKAGE_FFMPEG + +config BR2_PACKAGE_SQUEEZELITE_DSD + bool "Enable DSD decoding" + +config BR2_PACKAGE_SQUEEZELITE_RESAMPLE + bool "Enable resampling support" + select BR2_PACKAGE_LIBSOXR + +config BR2_PACKAGE_SQUEEZELITE_VISEXPORT + bool "Enable visualiser support" + +endif + +comment "squeezelite needs a toolchain w/ wchar, threads" + depends on BR2_USE_MMU + depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
package/squeezelite/squeezelite.mk new file +43 -0
43 lines not on this page · raw message
Signature · 4 lines
1.7.1
============================================================
Hiroshi KawashimaPATCH v14 1/1 squeezelite: new package
Dear all.
I will appreciate if you point out anything I should improve for following patch.
Thanks,
Kawashima
Quoted 207 lines
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.
- changeBR2_PACKAGE_SQUEEZELITE_FF->BR2_PACKAGE_SQUEEZELITE_FFMPEG.
- improve prompt message for configuration options.
- change installation destination path to fullpath.
- remove useless help string.v12 -> v13
- changeSQUEEZELITE_SITEto github (mine), cloned from original google code site (original site is unstable to fetch).
- change to use $(TARGET_CONFIGURE_OPTS) onSQUEEZELITE_BUILD_CMDSto be suitable for buildroot make system.
- to accomplish above, change to generate buildroot suitableMakefilefrom original squeezelite'sMakefile(using 'override' directive for CFLAGS, LDFLAGS).v11 -> v12
- remove -DRESAMPLE_MP, because libsoxr is compiled without openMP support in buildroot, so meaningless for now.
- add -DLINKALL to resolve symbols in optional libraries (libsoxr, etc).
- add more configuration options : ffmpeg, DSD, Visualizer. lirc support is not included for now, because more work will be necessary to use it (eg: modifying package/lirc-tools to addLIRC_TOOLS_INSTALL_STAGING= YES).--- package/Config.in | 1 + package/squeezelite/0001-Makefile-override.patch | 42 ++++++++++++++++++++(quoted diff) +2
-0+ package/squeezelite/Config.in | 38 +++++++++++++++++++ package/squeezelite/squeezelite.mk | 43 ++++++++++++++++++++ ++ 4 files changed, 124 insertions(+), 0 deletions(-) create mode 100644 package/squeezelite/0001-Makefile-override.patch create mode 100644 package/squeezelite/Config.in create mode 100644 package/squeezelite/squeezelite.mkpackage/Config.in +1
-0@@ -33,6 +33,7 @@ menu "Audio and video applications" source "package/opus-tools/Config.in" source "package/pulseaudio/Config.in" source "package/sox/Config.in" + source "package/squeezelite/Config.in" source "package/tidsp-binaries/Config.in" source "package/tovid/Config.in" source "package/tstools/Config.in"package/squee renamed from package/squeezelite/0001-Makefile-override.patch +0
-0
zelite/0001-Makefile-override.patch
new file mode 100644
index 0000000..5897e0bpackage/squeezelite/0001-Makefile-override.patch new file +11
-0@@ -0,0 +1,42 @@ +This patch is applied to squeezelite Makefile to add override +directive for CFLAGS and LDFLAGS assignment. +This enables adding flags to CFLAGS and LDFLAGS in Makefile. + +Signed-off-by: Hiroshi Kawashima <[email protected]> + +diff -ruN a/Makefile b/Makefile +--- a/Makefile 2015-02-01 23:56:21.000000000 +0900 ++++ b/Makefile 2015-09-29 20:06:52.268069187 +0900 +@@ -1,4 +1,6 @@ + # Cross compile support - create a Makefile which defines these three variables and then includes this
Makefile... ++override CFLAGS += $(OPTS)
++override LDFLAGS += -lasound -lpthread -lm -lrt
+ CFLAGS ?= -Wall -fPIC -O2 $(OPTS)
+ LDFLAGS ?= -lasound -lpthread -lm -lrt
+ EXECUTABLE ?= squeezelite
+@@ -52,20 +54,20 @@
+
+ # add optional link options
+ ifneq (,$(findstring $(OPT_LINKALL), $(CFLAGS)))
+- LDFLAGS += $(LINKALL)
++ override LDFLAGS += $(LINKALL)
+ ifneq (,$(findstring $(OPT_FF), $(CFLAGS)))
+- LDFLAGS += $(LINKALL_FF)
++ override LDFLAGS += $(LINKALL_FF)
+ endif
+ ifneq (,$(findstring $(OPT_RESAMPLE), $(CFLAGS)))
+- LDFLAGS += $(LINKALL_RESAMPLE)
++ override LDFLAGS += $(LINKALL_RESAMPLE)
+ endif
+ ifneq (,$(findstring $(OPT_IR), $(CFLAGS)))
+- LDFLAGS += $(LINKALL_IR)
++ override LDFLAGS += $(LINKALL_IR)
+ endif
+ else
+ # if not LINKALL and linux addLINK_LINUX
+ ifeq ($(UNAME), Linux)
+- LDFLAGS += $(LINK_LINUX)
++ override LDFLAGS += $(LINK_LINUX)
+ endif
+ endif
+package/squeezelite/Config.in new file +38
-0@@ -0,0 +1,38 @@ +config BR2_PACKAGE_SQUEEZELITE + bool "squeezelite" + depends on BR2_USE_WCHAR # flac + depends on BR2_TOOLCHAIN_HAS_THREADS # alsa-lib + depends on BR2_USE_MMU # mpg123 + select BR2_PACKAGE_ALSA_LIB + select BR2_PACKAGE_FLAC + select BR2_PACKAGE_LIBMAD + select BR2_PACKAGE_LIBVORBIS + select BR2_PACKAGE_FAAD2 + select BR2_PACKAGE_MPG123 + help + Logitech Media Server client + + https://code.google.com/p/squeezelite/ + +if BR2_PACKAGE_SQUEEZELITE + +config BR2_PACKAGE_SQUEEZELITE_FFMPEG + bool "Enable WMA and ALAC decoding" + default y + select BR2_PACKAGE_FFMPEG + +config BR2_PACKAGE_SQUEEZELITE_DSD + bool "Enable DSD decoding" + +config BR2_PACKAGE_SQUEEZELITE_RESAMPLE + bool "Enable resampling support" + select BR2_PACKAGE_LIBSOXR + +config BR2_PACKAGE_SQUEEZELITE_VISEXPORT + bool "Enable visualiser support" + +endif + +comment "squeezelite needs a toolchain w/ wchar, threads" + depends on BR2_USE_MMU + depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADSpackage/squeezelite/squeeze renamed from package/squeezelite/squeezelite.mk +0
-0lite.mk
new file mode 100644
index 0000000..cf9e151package/squeezelite/squeezelite.mk new file +1
-0@@ -0,0 +1,43 @@ +################################################################################
package/squeezelite/squeezelite.mk new file +4
-0+# +# squeezelite +# +################################################################################
package/squeezelite/squeezelite.mk new file +38
-0+ +SQUEEZELITE_VERSION = v1.8 +SQUEEZELITE_SITE = https://github.com/robadenshi/squeezelite +SQUEEZELITE_SITE_METHOD = git +SQUEEZELITE_LICENSE = GPLv3 +SQUEEZELITE_LICENSE_FILE = LICENSE.txt +SQUEEZELITE_DEPENDENCIES = alsa-lib flac libmad libvorbis faad2 mpg123 +SQUEEZELITE_MAKE_OPTS = -DLINKALL + +ifeq ($(BR2_PACKAGE_SQUEEZELITE_FFMPEG),y) +SQUEEZELITE_DEPENDENCIES += ffmpeg +SQUEEZELITE_MAKE_OPTS += -DFFMPEG +endif + +ifeq ($(BR2_PACKAGE_SQUEEZELITE_DSD),y) +SQUEEZELITE_MAKE_OPTS += -DDSD +endif + +ifeq ($(BR2_PACKAGE_SQUEEZELITE_RESAMPLE),y) +SQUEEZELITE_DEPENDENCIES += libsoxr +SQUEEZELITE_MAKE_OPTS += -DRESAMPLE +endif + +ifeq ($(BR2_PACKAGE_SQUEEZELITE_VISEXPORT),y) +SQUEEZELITE_MAKE_OPTS += -DVISEXPORT +endif + +define SQUEEZELITE_BUILD_CMDS + $(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) \ + OPTS="$(SQUEEZELITE_MAKE_OPTS)" -C $(@D) all +endef + +define SQUEEZELITE_INSTALL_TARGET_CMDS + $(INSTALL) -D -m 0755 $(@D)/squeezelite \ + $(TARGET_DIR)/usr/bin/squeezelite +endef + +$(eval $(generic-package))Signature · 8 lines
1.7.1 ============================================================ Hiroshi Kawashima _______________________________________________ buildroot mailing list buildroot at busybox.net http://lists.busybox.net/mailman/listinfo/buildroot
============================================================
Hiroshi Kawashima
PATCH v14 1/1 squeezelite: new package
Dear Hiroshi Kawashima,
Quoted 13 lines · On 09/30/2015 03:34 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear all.
I will appreciate if you point out anything I should improve for following patch.
Thanks,
KawashimaQuoted 5 lines
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.
[snip]
Quoted 3 lines
Quoted 3 lines
v12 -> v13
- changeSQUEEZELITE_SITEto github (mine), cloned from original google code site (original site is unstable to fetch).
So you own a clone of that project on github, and the package you want to add in Buildroot clones from it. Great! Then, why not...
[snip]
Quoted 3 lines
Quoted 3 lines
- to accomplish above, change to generate buildroot suitable
Makefilefrom original squeezelite'sMakefile(using 'override' directive for CFLAGS, LDFLAGS).
...fix the Makefile upstream (your github repo) instead of adding a patch in Buildroot?
Regards,
Vincent.
[snip]
PATCH v14 1/1 squeezelite: new package
Dear, Vincent.
Thank you for your comment.
Quoted 2 lines
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?
Since this modification is buildroot specific, modification should be embeded in buildroot environment (as patch file), I should untouch original source trees as possible as I can, that is my understanding.
But anyway, I will obey your (buildroot community) preference.
I will have 2 choice.
1) Use current method (patching to original Makefile).
2) Prepare modified Makefile.buildroot and add to my github repo.
Which is preferred ?
Please advise me.
Regards,
Kawashima
Quoted 32 lines · Vicente Olivert Riera
Dear Hiroshi Kawashima,
Quoted 13 lines · On 09/30/2015 03:34 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear all.
I will appreciate if you point out anything I should improve for following patch.
Thanks,
KawashimaQuoted 5 lines
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.[snip]
Quoted 3 lines
Quoted 3 lines
v12 -> v13
- changeSQUEEZELITE_SITEto github (mine), cloned from original google code site (original site is unstable to fetch).So you own a clone of that project on github, and the package you want to add in Buildroot clones from it. Great! Then, why not...
[snip]
Quoted 3 lines
Quoted 3 lines
- to accomplish above, change to generate buildroot suitable
Makefilefrom original squeezelite'sMakefile(using 'override' directive for CFLAGS, LDFLAGS).
Quoted 8 lines
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Regards,
Vincent.
[snip]
============================================================
Hiroshi Kawashima
PATCH v14 1/1 squeezelite: new package
Dear, Vincent, all.
I deeply appreciate if you suggest me which is preferred (or policy ?) of buildroot community.
Regards,
Kawashima
Quoted 65 lines · kei-k at ca2.so-net.ne.jp writes:
Dear, Vincent.
Thank you for your comment.
Quoted 2 lines
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Since this modification is buildroot specific, modification should be embeded in buildroot environment (as patch file), I should untouch original source trees as possible as I can, that is my understanding.
But anyway, I will obey your (buildroot community) preference.
I will have 2 choice.
1) Use current method (patching to originalMakefile).
2) Prepare modified Makefile.buildroot and add to my github repo.Which is preferred ?
Please advise me.Regards,
KawashimaQuoted 19 lines · Vicente Olivert Riera
Dear Hiroshi Kawashima,
Quoted 13 lines · On 09/30/2015 03:34 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear all.
I will appreciate if you point out anything I should improve for following patch.
Thanks,
KawashimaQuoted 5 lines
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.[snip]
Quoted 10 lines
Quoted 3 lines
Quoted 3 lines
v12 -> v13
- changeSQUEEZELITE_SITEto github (mine), cloned from original google code site (original site is unstable to fetch).So you own a clone of that project on github, and the package you want to add in Buildroot clones from it. Great! Then, why not...
[snip]
Quoted 1 line
Quoted 1 line
- to accomplish above, change to generate buildroot suitable
Makefilefrom
Quoted 1 line
Quoted 1 line
Quoted 1 line
original squeezelite's
Makefile(using 'override' directive for CFLAGS,
Quoted 9 lines
Quoted 1 line
Quoted 1 line
LDFLAGS).
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Regards,
Vincent.
Quoted 8 lines
Quoted 1 line
[snip]
============================================================
Hiroshi Kawashima
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
============================================================
Hiroshi Kawashima
PATCH v14 1/1 squeezelite: new package
Dear Hiroshi Kawashima,
Quoted 4 lines · On 10/01/2015 12:39 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear, Vincent, all.
I deeply appreciate if you suggest me which is preferred (or policy ?) of buildroot community.
I don't know the policy in this case, and I don't even know if we have one. What I was suggesting is my personal opinion. I really think is better to have a flexible and cross-compile friendly Makefile upstream (since you are in control of the upstream repository) rather than fixing it only in Buildroot. If you fix it upstream, other projects could benefit from that, not only Buildroot. But, again, this is only my personal opinion. Better to listen the opinion of other developers as well, specially the maintainers Thomas and Peter.
Regards,
Vincent.
Quoted 80 lines
Regards,
KawashimaQuoted 65 lines · kei-k at ca2.so-net.ne.jp writes:
Dear, Vincent.
Thank you for your comment.
Quoted 2 lines
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Since this modification is buildroot specific, modification should be embeded in buildroot environment (as patch file), I should untouch original source trees as possible as I can, that is my understanding.
But anyway, I will obey your (buildroot community) preference.
I will have 2 choice.
1) Use current method (patching to originalMakefile).
2) Prepare modified Makefile.buildroot and add to my github repo.Which is preferred ?
Please advise me.Regards,
KawashimaQuoted 19 lines · Vicente Olivert Riera
Dear Hiroshi Kawashima,
Quoted 13 lines · On 09/30/2015 03:34 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear all.
I will appreciate if you point out anything I should improve for following patch.
Thanks,
KawashimaQuoted 5 lines
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.[snip]
Quoted 10 lines
Quoted 3 lines
Quoted 3 lines
v12 -> v13
- changeSQUEEZELITE_SITEto github (mine), cloned from original google code site (original site is unstable to fetch).So you own a clone of that project on github, and the package you want to add in Buildroot clones from it. Great! Then, why not...
[snip]
Quoted 1 line
Quoted 1 line
- to accomplish above, change to generate buildroot suitable
Makefilefrom
Quoted 1 line
Quoted 1 line
Quoted 1 line
original squeezelite's
Makefile(using 'override' directive for CFLAGS,
Quoted 9 lines
Quoted 1 line
Quoted 1 line
LDFLAGS).
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Regards,
Vincent.
Quoted 8 lines
Quoted 1 line
[snip]
============================================================
Hiroshi Kawashima
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot============================================================
Hiroshi Kawashima
PATCH v14 1/1 squeezelite: new package
Dear maintainer, Thomas, Peter.
(Thank you Vincent, for your suggestion.)
I deeply appreciate if you suggest me which method is preferred.
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
I will have 2 choice.
1) Use current method (patching to originalMakefile).
2) Prepare modified Makefile.buildroot and add to my github repo.
Regards,
Kawashima
Quoted 75 lines · Vicente Olivert Riera
Dear Hiroshi Kawashima,
Quoted 4 lines · On 10/01/2015 12:39 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear, Vincent, all.
I deeply appreciate if you suggest me which is preferred (or policy ?) of buildroot community.
I don't know the policy in this case, and I don't even know if we have one. What I was suggesting is my personal opinion. I really think is better to have a flexible and cross-compile friendly
Makefileupstream (since you are in control of the upstream repository) rather than fixing it only in Buildroot. If you fix it upstream, other projects could benefit from that, not only Buildroot. But, again, this is only my personal opinion. Better to listen the opinion of other developers as well, specially the maintainers Thomas and Peter.Regards,
Vincent.
Quoted 54 lines
Regards,
KawashimaQuoted 50 lines · kei-k at ca2.so-net.ne.jp writes:
Dear, Vincent.
Thank you for your comment.
Quoted 2 lines
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Since this modification is buildroot specific, modification should be embeded in buildroot environment (as patch file), I should untouch original source trees as possible as I can, that is my understanding.
But anyway, I will obey your (buildroot community) preference.
I will have 2 choice.
1) Use current method (patching to originalMakefile).
2) Prepare modified Makefile.buildroot and add to my github repo.Which is preferred ?
Please advise me.Regards,
KawashimaQuoted 26 lines · Vicente Olivert Riera
Dear Hiroshi Kawashima,
Quoted 13 lines · On 09/30/2015 03:34 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear all.
I will appreciate if you point out anything I should improve for following patch.
Thanks,
KawashimaQuoted 5 lines
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.[snip]
Quoted 3 lines
Quoted 3 lines
v12 -> v13
- changeSQUEEZELITE_SITEto github (mine), cloned from original google code site (original site is unstable to fetch).So you own a clone of that project on github, and the package you want to add in Buildroot clones from it. Great! Then, why not...
Quoted 29 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
[snip]
Quoted 1 line
Quoted 1 line
- to accomplish above, change to generate buildroot suitable
Makefilefr
Quoted 2 lines
Quoted 2 lines
om
Quoted 1 line
Quoted 1 line
Quoted 1 line
original squeezelite's
Makefile(using 'override' directive for CFLAGS
Quoted 21 lines
Quoted 10 lines
,
Quoted 9 lines
Quoted 1 line
Quoted 1 line
LDFLAGS).
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Regards,
Vincent.
Quoted 8 lines
Quoted 1 line
[snip]
============================================================
Hiroshi Kawashima
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot============================================================
Hiroshi Kawashima
============================================================
Hiroshi Kawashima
PATCH v14 1/1 squeezelite: new package
Dear maintainer.
Sorry bothering you, but if no further objection, should I just wait your commit ?
Or anything I should do ?
Please advise me.
Kawashima
Quoted 118 lines · kei-k at ca2.so-net.ne.jp writes:
Dear maintainer, Thomas, Peter.
(Thank you Vincent, for your suggestion.)I deeply appreciate if you suggest me which method is preferred.
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
I will have 2 choice.
1) Use current method (patching to originalMakefile).
2) Prepare modified Makefile.buildroot and add to my github repo.Regards,
KawashimaQuoted 17 lines · Vicente Olivert Riera
Dear Hiroshi Kawashima,
Quoted 4 lines · On 10/01/2015 12:39 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear, Vincent, all.
I deeply appreciate if you suggest me which is preferred (or policy ?) of buildroot community.
I don't know the policy in this case, and I don't even know if we have one. What I was suggesting is my personal opinion. I really think is better to have a flexible and cross-compile friendly
Makefileupstream (since you are in control of the upstream repository) rather than fixing it only in Buildroot. If you fix it upstream, other projects could benefit from that, not only Buildroot. But, again, this is only my personal opinion. Better to listen the opinion of other developers as well, specially the maintainers Thomas and Peter.Quoted 61 lines
Regards,
Vincent.
Quoted 57 lines
Regards,
KawashimaQuoted 53 lines · kei-k at ca2.so-net.ne.jp writes:
Dear, Vincent.
Thank you for your comment.
Quoted 2 lines
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Since this modification is buildroot specific, modification should be embeded in buildroot environment (as patch file), I should untouch original source trees as possible as I can, that is my understanding.
But anyway, I will obey your (buildroot community) preference.
I will have 2 choice.
1) Use current method (patching to originalMakefile).
2) Prepare modified Makefile.buildroot and add to my github repo.Which is preferred ?
Please advise me.Regards,
KawashimaQuoted 29 lines · Vicente Olivert Riera
Dear Hiroshi Kawashima,
Quoted 13 lines · On 09/30/2015 03:34 PM, kei-k at ca2.so-net.ne.jp wrote:
Dear all.
I will appreciate if you point out anything I should improve for following patch.
Thanks,
KawashimaQuoted 5 lines
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.[snip]
Quoted 3 lines
Quoted 3 lines
v12 -> v13
- changeSQUEEZELITE_SITEto github (mine), cloned from original google code site (original site is unstable to fetch).So you own a clone of that project on github, and the package you want to add in Buildroot clones from it. Great! Then, why not...
[snip]
Quoted 1 line
Quoted 1 line
- to accomplish above, change to generate buildroot suitable
Makefilef
Quoted 3 lines
r
Quoted 2 lines
Quoted 2 lines
om
Quoted 1 line
Quoted 1 line
Quoted 1 line
original squeezelite's
Makefile(using 'override' directive for CFLAG
Quoted 23 lines
S
Quoted 21 lines
Quoted 10 lines
,
Quoted 9 lines
Quoted 1 line
Quoted 1 line
LDFLAGS).
...fix the
Makefileupstream (your github repo) instead of adding a patch in Buildroot?Regards,
Vincent.
Quoted 8 lines
Quoted 1 line
[snip]
============================================================
Hiroshi Kawashima
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot============================================================
Hiroshi Kawashima
Quoted 6 lines
============================================================
Hiroshi Kawashima
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
============================================================
Hiroshi Kawashima
PATCH v14 1/1 squeezelite: new package
In reply to kei-k at ca2.so-net.ne.jpHello,
Quoted 9 lines · On Tue, 29 Sep 2015 20:08:13 +0900, kei-k at ca2.so-net.ne.jp wrote:
Signed-off-by Hiroshi Kawashima <[email protected]>---
Changelog
v13 -> v14
- change to use patch file insted of using sed to generateMakefile.
- changeBR2_PACKAGE_SQUEEZELITE_FF->BR2_PACKAGE_SQUEEZELITE_FFMPEG.
- improve prompt message for configuration options.
- change installation destination path to fullpath.
- remove useless help string.
I have applied your patch, after making a few additional changes:
[Thomas:
- replicate the ffmpeg "depends on" related to the broken NIOS2 toolchains to the BR2_PACKAGE_SQUEEZELITE_FFMPEG option.
- use the github helper function.
- add hash file.]
Thanks for your contribution!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com