[Buildroot] [PATCH v7 1/1] squeezelite: new package
Vicente Olivert Riera
Vincent.Riera at imgtec.com
Mon Sep 21 08:44:18 UTC 2015
Dear Hiroshi Kawashima,
comments below.
On 09/21/2015 04:47 AM, kei-k at ca2.so-net.ne.jp wrote:
[snip]
> diff --git a/package/squeezelite/squeezelite.mk b/package/squeezelite/squeezelite.mk
> new file mode 100644
> index 0000000..681bb46
> --- /dev/null
> +++ b/package/squeezelite/squeezelite.mk
> @@ -0,0 +1,30 @@
> +################################################################################
> +#
> +# squeezelite
> +#
> +################################################################################
> +
> +SQUEEZELITE_VERSION = v1.8
> +SQUEEZELITE_SITE = https://code.google.com/p/squeezelite
> +SQUEEZELITE_SITE_METHOD = git
> +SQUEEZELITE_LICENSE = GPLv3
> +SQUEEZELITE_LICENSE_FILE = LICENSE.txt
> +SQUEEZELITE_DEPENDENCIES = alsa-lib flac libmad libvorbis faad2 mpg123 libsoxr
> +
> +ifeq ($(BR2_PACKAGE_SQUEEZELITE_ENABLE_RESAMPLE),y)
> + SQUEEZELITE_MAKE_OPTS += -DRESAMPLE
> + ifeq ($(BR2_PACKAGE_SQUEEZELITE_WITH_RESAMPLE_MP),y)
> + SQUEEZELITE_MAKE_OPTS += -DRESAMPLE_MP
> + endif
> +endif
I think the Buildroot style is to not indent this code. Can other dev
tell me if I'm wrong?
> +
> +define SQUEEZELITE_BUILD_CMDS
> + $(TARGET_MAKE_ENV) $(MAKE) OPTS="$(SQUEEZELITE_MAKE_OPTS)" \
You should indent this command with one tab instead of four spaces.
> + CC="$(TARGET_CC)" LD="$(TARGET_LD)" -C $(@D) all
And this line indented with two tabs as is the continuation of the above
command.
> +endef
> +
> +define SQUEEZELITE_INSTALL_TARGET_CMDS
> + $(INSTALL) -D -m 0755 $(@D)/squeezelite $(TARGET_DIR)/usr/bin
Same here. This command should be indented with one tab instead of four
spaces.
Regards,
Vincent.
> +endef
> +
> +$(eval $(generic-package))
>
More information about the buildroot
mailing list