<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>