<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-06-26 11:26 GMT+02:00 James Cowgill <span dir="ltr"><<a href="mailto:jcowgill@debian.org" target="_blank">jcowgill@debian.org</a>></span>:<span class="gmail-"><br><br></span></div><div class="gmail_quote"><span class="gmail-">Hi James,<br><br></span></div><div class="gmail_quote"><span class="gmail-">thank you for reviewing this package.<br></span></div><div class="gmail_quote"><span class="gmail-"><br></span><span class="gmail-"></span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
>     here is one last minute package ;)<br>
>     Can someone review and upload this package to NEW please?<br>
>     Would be great if this would slip in before freeze ;)<br>
><br>
> Can someone upload avldrums.lv2 package now for me please?<br>
<br>
</span>debian/rules:<br>
> ifeq ($(DEB_HOST_ARCH_CPU),amd64)<br>
> export OPTIMIZATIONS += -msse -msse2 -mfpmath=sse<br>
> endif<br>
<br>
You should be able to remove this bit. Those 3 options are already<br>
enabled by default on amd64.<br></blockquote><div><br></div><div>Done!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> override_dh_auto_clean:<br>
><br>
<br>
Why?<br></blockquote><div><br></div><div>This helped me avoid build problem described on the end of your message ...<br></div><div>Removed now.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> override_dh_auto_configure:<br>
>       dh_auto_configure -- \<br>
>               --prefix=/usr<br>
<br>
dh_auto_configure is a no-op for "Makefile only" packages.<br></blockquote><div><br></div><div>Fixed! <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
><br>
> override_dh_auto_build:<br>
>       $(MAKE) \<br>
>               PREFIX=/usr<br>
<br>
You should use "dh_auto_build -- PREFIX=/usr" so parallel builds work<br></blockquote><div><br></div><div>Done!<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> override_dh_auto_install:<br>
>       $(MAKE) install \<br>
>               DESTDIR=$(CURDIR)/debian/tmp \<br>
>               PREFIX=/usr<br>
<br>
And "dh_auto_install" here.<br></blockquote><div><br></div><div>Done!<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
debian/watch file seems to be broken?<br></blockquote><div><br></div><div>Fixed!<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
the "fluidsynth" and "robtk" directories look like external modules and<br>
should be de-bundled.<br></blockquote><div><br></div><div>I have discussed this with upstream before ...<br></div><div>here is answer ... (  same might go for robtk? )<br>------------------<br></div><div><br><span class="gmail-im">> Is there some way use rather system-wide fluidsynth lib?<br>
<br>
</span>No (same with Ardour btw). It's a stripped down and patched dedicated<br>
version and statically linked with hidden symbols.<br>
<br>
Pro Audio Plugins should be self-contained and not rely on any system<br>
libs except for libc (and libX11/xcb which needs to match the X11 server<br>
ABI).<br>
<br>
There are too many things that can go wrong otherwise (calf is the prime<br>
example). e.g. update fftw or mix some random pango/cairo/pixman or even<br>
just freetype version and you'll throw thread safety out of the window.<br>
Any lib that uses a lock to maintain some global cache will void<br>
realtime safety as well.  default fluidsynth falls in that category as<br>
well:  .sf2 files are cached globally and a lock is used.<br>
<br>
Remember, plugins are loaded in the same memory space as the host, which<br>
may also use those libs. They're not standalone process separated apps.<br>
Ideally plugins are self-contained and don't share any libs with the host.<br>
<br>
If 2 unrelated plugins were to use the same shared libfludsynth various<br>
edge cases can happen since the plugins don't know about each other.<br>
<br>
Also related is exception handling in this case (setjmp/longjmp). At<br>
least for rolling distros (e.g. debian/testing) where a library can be<br>
compiled with a different compiler version than the plugin and host.<br>--------------<br>
<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I see the -data package just contains some soundfonts. Would it be<br>
sensible to put these in the "public" soundfont directory<br>
(/usr/share/sounds/sf2) or are they only usable with avldrums?<br></blockquote><div> <br></div><div>Probably they are usable just with avldrums, but I can ask upstream.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Building in git:<br>
> Version: debian/0.2.2-1-9 -> debian/0-9 debian/2-9 debian/2-9 debian/1-9<br>
> expr: non-integer argument<br>
> <a href="http://git2lv2.mk:37" rel="noreferrer" target="_blank">git2lv2.mk:37</a>: *** "Cannot extract required LV2 minor-version parameter".  Stop.<br>
<br>
While this only happens when using git, it is very frustrating. I think<br>
you should override the "avldrums_VERSION" variable to fix it.<br></blockquote><div><br></div><div>Can you elaborate here? I am not sure how to fix this :( <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I haven't done an indepth copyright check, but I can't seen any obvious<br>
problems.<br></blockquote><div><br></div><div>Ok ...<br><br></div><div>best regards<br><br></div><div>mira <br></div></div><br></div></div>