← Back to team overview

ubuntustudio-bugs team mailing list archive

Re: [Bug 1827288] Re: [Needs Packaging] LSP-Plugins for Eoan

 

Hi Steve,

On Mon, May 6, 2019 at 2:24 PM, Steve Langasek 
<steve.langasek@xxxxxxxxxxxxx> wrote:
> The version number in the debian/changelog of the git branch is wrong;
> this needs to be a non-native package using the upstream tarball from
> sourceforge.

Fixed the versioning. The upstream tarball is on Github, no longer on 
sourceforge (their website is a little behind, if that's where you got 
your info).

> + libjack-jackd2-dev | libjack-dev,
> + php7.2-cli | php5-cli,
> + cairo-static | libcairo2-dev,
> 
> why are you using alternative build-dependencies for a new, 
> Ubuntu-only
> package?  You should only list whichever of these build dependencies 
> you
> actually expect to be used (and there is no cairo-static package in
> Ubuntu anywhere, so that is definitely wrong).
> 
> Please use php-cli, not phpN.M-cli, as the build-dependency.
> 

I don't know why, but I can fix. This is mostly a grab from the 
KXStudio repo. As I don't know completely how to clean it, this is why 
I appreciate feedback so I can learn these things and fix them, 
reducing my ignorance through more practice. Learning by doing.

> +Provides:
> + lv2-plugin,
> + ladspa-plugin,
> 
> Why are these Provides: attached to the lsp-plugins package, when 
> there
> are separate lsp-plugins-ladspa and lsp-plugins-lv2 packages?  The
> Provides look like they belong on the latter packages.

Fixed.

> 
> +Package: lsp-plugins-common
> +Architecture: any
> +Provides: lsp-plugins-standalone
> 
> Something is definitely wrong here given that you also have a binary
> package named lsp-plugins-standalone which depends on lsp-plugins-
> common.  In fact, lsp-plugins-standalone appears to be an empty 
> package;
> so you should not list this binary package at all in debian/control.

Fixed by renaming the common package to standalone. All of the other 
plugins depend on these binaries, so it make sense. I guess I was being 
redundant by having an empty package just to install the binaries as 
standalone.


> +FLAGS=-O3 -pipe -ffast-math -mtune=generic -msse -msse2 -mfpmath=sse
> -fPIC -DPIC -fvisibility=hidden -Wno-unused-result
> 
> How have you arrived at these flags?  Compiler flags should a) inherit
> from dpkg-buildflags by default (You are completely overriding them),
> and b) document the reasons for any divergence from the distro 
> defaults.
> But actually, it seems that the only thing you're doing is setting 
> these
> flags in a make variable called 'dpkg_buildflags', so in fact this has
> no effect at all.
> 
> Not that you could tell by looking at the build log, which does not 
> show
> the compiler commands being invoked.  This is significant for
> debuggability of builds in launchpad.  Please figure out how to make
> this upstream's build system appropriately verbose.

I went ahead and removed all of those flags. The build worked fine, so 
I don't understand why they were there to begin with. Again, that was 
copied from KXStudio.

> +override_dh_auto_test:
> +       #Tests only work manually, ok to override per 
> <http://manpages.ubuntu.com/manpages/disco/en/man1/dh_auto_test.1.html>
> +
> 
> what are these tests, and why do they only work manually?  It's
> /allowed/ to override tests, but the goal should be to run them.

Overriding dh_auto_test with the commands one would use to manually run 
the tests seemed to do the trick.

Thanks for the help!

-- 
You received this bug notification because you are a member of Ubuntu
Studio Bugs, which is subscribed to Ubuntu Studio.
Matching subscriptions: UbuntuStudio Bugs
https://bugs.launchpad.net/bugs/1827288

Title:
  [Needs Packaging] LSP-Plugins for Eoan

Status in Linux Studio Plugins:
  In Progress
Status in Ubuntu Studio:
  In Progress

Bug description:
  From the project page:

  LSP (Linux Studio Plugins) is a collection of open-source plugins
  currently compatible with follwing formats:

   * LADSPA - set of plugins for Linux Audio Developer's Simple Plugin API
   * LV2 - set of plugins and UIs for Linux Audio Developer's Simple Plugin API (LADSPA) version 2
   * LinuxVST - set of plugins and UIs for Steinberg's VST 2.4 format ported on GNU/Linux Platform
   * JACK - Standalone versions for JACK Audio connection Kit with UI

  The basic idea is to fill the lack of good and useful plugins under the GNU/Linux platform.
  ----
  I have finished packaging this and would like it included in Ubuntu's universe repositories, targeting eoan.

  Code is at https://launchpad.net/lsp-plugins

To manage notifications about this bug go to:
https://bugs.launchpad.net/lsp-plugins/+bug/1827288/+subscriptions


References