<div dir="ltr">I'll add the license and dependency statements and will modify the patch to use the more elegant approach suggested by Samuel.<div><br></div><div>Thanks for your input, Eric.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 26, 2014 at 9:11 PM, Peter Korsgaard <span dir="ltr"><<a href="mailto:jacmet@uclibc.org" target="_blank">jacmet@uclibc.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>>>>> "Eric" == Eric Limpens <<a href="mailto:limpens@gmail.com">limpens@gmail.com</a>> writes:<br>
<br>
> Signed-off-by: Eric Limpens <<a href="mailto:limpens@gmail.com">limpens@gmail.com</a>><br>
> ---<br>
> package/Config.in | 1 +<br>
> package/ympd/Config.in | 10 ++++++++++<br>
> package/ympd/ympd-0001-CMake-cross-compile-mkdata.patch | 16 ++++++++++++++++<br>
> package/ympd/<a href="http://ympd.mk" target="_blank">ympd.mk</a> | 16 ++++++++++++++++<br>
> 4 files changed, 43 insertions(+)<br>
> create mode 100644 package/ympd/Config.in<br>
> create mode 100644 package/ympd/ympd-0001-CMake-cross-compile-mkdata.patch<br>
> create mode 100644 package/ympd/<a href="http://ympd.mk" target="_blank">ympd.mk</a><br>
<br>
> +++ b/package/ympd/ympd-0001-CMake-cross-compile-mkdata.patch<br>
> @@ -0,0 +1,16 @@<br>
> +CMakeLists.txt: make it cross-compile friendly. Upstream is looking into a proper solution.<br>
> +<br>
> +Signed-Off-by: Eric Limpens <Limpens@Gmail.com><br>
> +<br>
> +--- ympd-a98d760dddff4b0dd595d977c9077f34beff05c5/CMakeLists.txt 2014-05-24 15:25:47.000000000 +0200<br>
> ++++ ympd-a98d760dddff4b0dd595d977c9077f34beff05c5-1/CMakeLists.txt 2014-09-23 20:04:55.491997850 +0200<br>
> +@@ -29,6 +29,7 @@<br>
> + )<br>
> +<br>
> +-add_executable(mkdata htdocs/mkdata.c)<br>
> +-get_target_property(MKDATA_EXE mkdata LOCATION)<br>
> ++#add_executable(mkdata htdocs/mkdata.c)<br>
> ++#get_target_property(MKDATA_EXE mkdata LOCATION)<br>
> ++set(MKDATA_EXE "./mkdata")<br>
<br>
<br>
It seems like you missed Samuels' feedback?<br>
<br>
> +<br>
> + add_custom_command(OUTPUT ${PROJECT_BINARY_DIR}/assets.c<br>
> diff --git a/package/ympd/<a href="http://ympd.mk" target="_blank">ympd.mk</a> b/package/ympd/<a href="http://ympd.mk" target="_blank">ympd.mk</a><br>
> new file mode 100644<br>
> index 0000000..33fd4ec<br>
> --- /dev/null<br>
> +++ b/package/ympd/<a href="http://ympd.mk" target="_blank">ympd.mk</a><br>
> @@ -0,0 +1,16 @@<br>
> +################################################################################<br>
> +#<br>
> +# ympd<br>
> +#<br>
> +################################################################################<br>
> +<br>
> +YMPD_VERSION = a98d760dddff4b0dd595d977c9077f34beff05c5<br>
> +YMPD_SITE = $(call github,notandy,ympd,$(YMPD_VERSION))<br>
<br>
What about license?<br>
<br>
You need to add MPD_DEPENDENCIES = libmpdclient to ensure libmpdclient<br>
gets built before ympd.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Bye, Peter Korsgaard<br>
</font></span></blockquote></div><br></div>