Bug 249959 - audio/ardour6: update to 6.3.0 and unbreak
Summary: audio/ardour6: update to 6.3.0 and unbreak
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Christoph Moench-Tegeder
URL:
Keywords: easy, patch, patch-ready
Depends on:
Blocks:
 
Reported: 2020-09-28 19:08 UTC by Christoph Moench-Tegeder
Modified: 2020-09-29 20:07 UTC (History)
0 users

See Also:
beerml: maintainer-feedback+


Attachments
update ardor6 to 6.3.0 (2.54 KB, patch)
2020-09-28 19:08 UTC, Christoph Moench-Tegeder
cmt: maintainer-approval?
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christoph Moench-Tegeder freebsd_committer 2020-09-28 19:08:26 UTC
Created attachment 218382 [details]
update ardor6 to 6.3.0

attached updates ardour6 to version 6.3.0. patching wscript makes sure this build system at least recognizes FreeBSD. As build time CPU detection is a problematic idea (think package builder) and detecting CPU features from inside the build jail is far from trivial, I decided to let amd64 have SSE and MMX (those are over 20 years old now and were introduced back in the 32bit era) and don't even try for SSE on i386 (is i386 even a relevant platform for an application like this?).
The whole ss3-vs-not-problem seems to have been uncovered by the recent glib update, which is why we didn't note this problem until now.
Additionally, ardour 6.3 replaces the "mixer-demo" with a much more elaborate mixer system, that accounts for the big change in pkg-plist.

This passes popudriere and at least starts for me.
The problem between cccache, waf and gas (see bug #246915) needs to wait for another day.
Comment 1 Michael Beer 2020-09-29 07:14:08 UTC
(In reply to Christoph Moench-Tegeder from comment #0)

do you say that in addition to what is specified regarding SSE support in the ports makefile, the wscript checks for the architecture once more and even overwrites the makefile settings?
Is this a new feature of the wscript?
As you said there were no problems in the past with this, and when I build locally the only problem I see that the runtime detection for AVX (some new AVX was added) doesn't work for FreeBSD. I didn't have time to look into this further. 
Regarding i386: I don't think supporting any older CPU without SSE makes sense for such an application. That's why I specified SSE support also for the 32-bit builds in the makefile.
Comment 2 Christoph Moench-Tegeder freebsd_committer 2020-09-29 09:21:32 UTC
(In reply to Michael Beer from comment #1)
Right, wscript does some checks on it's own (badly, in our case, as it doesn't know about FreeBSD) and messes with compiler flags and build targets again. See https://github.com/Ardour/ardour/blob/master/wscript#L522 for example.
Comment 3 Michael Beer 2020-09-29 13:23:15 UTC
(In reply to Christoph Moench-Tegeder from comment #2)
OK, thanks for the explanation.
Regarding SSE for 32-bit build I still would prefer including SSE support by default. The standard FPU is more affected by denormalisation issues which happen quite often in audio precessing because of the usage of IIR filter in e.g. equalizers. For SSE you can switch of the special treatment of denormalized floating point values in contrast.
Comment 4 commit-hook freebsd_committer 2020-09-29 20:04:35 UTC
A commit references this bug:

Author: cmt
Date: Tue Sep 29 20:03:40 UTC 2020
New revision: 550616
URL: https://svnweb.freebsd.org/changeset/ports/550616

Log:
  audio/ardour6: update to 6.3.0 and unbreak

  attempt to fill in the blanks (like FreeBSD support) in this port's
  wscript's CPU feature detection. Build time CPU detection is problematic
  in the context of package builds, so for now this assumes that all
  amd64 CPUs have SSE and i386 CPUs don't. Juggling the flags in this
  way also fixes the breakage uncovered by the latest glib update.

  PR:		249959
  Approved by:	Michael Beer (maintainer)

Changes:
  head/audio/ardour6/Makefile
  head/audio/ardour6/distinfo
  head/audio/ardour6/files/
  head/audio/ardour6/files/patch-wscript
  head/audio/ardour6/pkg-plist
Comment 5 Christoph Moench-Tegeder freebsd_committer 2020-09-29 20:07:44 UTC
thanks - great we could get this in before next Quarterly branch. We'll need to re-visit CPU detection soon, both in wscript and for the runtime parts.