<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-08-23 15:37 GMT+02:00 James Cowgill <span dir="ltr"><<a href="mailto:jcowgill@debian.org" target="_blank">jcowgill@debian.org</a>></span>:<span class=""></span><br><span class=""></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
On 23/08/17 13:04, Jaromír Mikeš wrote:</span><br></blockquote><div><br></div><div>Hi James,<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> Vee One Suite has new package padthv1<br>
> <a href="http://www.rncbc.org/drupal/node/1846" rel="noreferrer" target="_blank">http://www.rncbc.org/drupal/<wbr>node/1846</a><br>
> <a href="https://padthv1.sourceforge.io/" rel="noreferrer" target="_blank">https://padthv1.sourceforge.<wbr>io/</a><br>
><br>
> I need DD to upload it to "NEW" ... can somebody do it for me?<br>
<br>
</span>debian/control:<br>
- automake is already pulled in by debhelper (via dh-autoreconf).<br>
- Sorting your build dependencies would be nice.<br>
- The Homepage redirects to <a href="https://padthv1.sourceforge.io/" rel="noreferrer" target="_blank">https://padthv1.sourceforge.<wbr>io/</a> so perhaps<br>
that should be used?<br>
- The word "standalone" is not a noun so it does not make sense to use<br>
it on its own at the end of the short description.<br>
- In the padthv1-lv2 short description, "lv2-plugin" should probably be<br>
"LV2 plugin" or something similar.<br>
<br>
debian/copyright:<br>
- Format field should be https now.<br>
- Source field should use https.<br>
- The short name for public domain works is "public-domain".<br>
- You should include the actual public domain waiver statement in your<br>
debian/copyright file (ie add the line "This work is in public domain."<br>
to the public domain section).<br>
- Your "BDS" license looks like the ISC license to me.<br>
<br>
lintian:<br>
W: padthv1: appstream-metadata-in-legacy-<wbr>location<br>
usr/share/appdata/padthv1.<wbr>appdata.xml<br>
<br>
Apparently the path was recently changed (it was the first time I knew<br>
about it).<br>
<br>
W: padthv1-common:<br>
package-has-unnecessary-<wbr>activation-of-ldconfig-trigger<br>
<br>
You can probably ignore this one since it's a debhelper bug and running<br>
ldconfig an extra time usually isn't that much of an issue. If you want<br>
you could fix it by using "dh_makeshlibs --no-scripts" since there are<br>
no public libraries in any package built by this source.<br>
<br>
Other:<br>
The git repository already contains a bogus "debian/0.8.4-1" tag.<br>
<br></blockquote><div><br></div><div>Thank you for reviewing.<br></div><div>All above done!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On a related note, I see you've patched upstream to move the lv2 plugins<br>
back into /usr/lib/lv2. I do not know much about lv2 plugins, but is it<br>
a good idea to install them to a multiarch path at some point? How much<br>
work would that be to do for every lv2 plugin in the archive? Would<br>
doing that cause any big issues?<br></blockquote><div><br></div><div>This is interesting topic which I already started here on list quite long time ago.<br></div><div>I would need to search archive to find this tread.<br></div><div>Generally problem is ( as I understand it ) that plugins hosts doesn't search for plugins in multiarch path :(<br></div><div>Otherwise it would great to have them multiarch.<br><br></div><div>mira <br></div></div><br></div></div>