Re: [NEW PORT: audio/rubberband]

[Tobias Brodel]

necroping,


but it fails in the linking stage, unable to find the `main' function.

so for now i'm just patching that part out, any comments?

please find a tarball attached.

t/

On 01/26/17 18:59, Tobias Brodel wrote:

i've made a few attempts to get the inline assembly built on armv7 with clang,

On 01/18/17 22:55, Tobias Brodel wrote:

hi stuart, thanks for your response.

On 01/17/17 23:10, Stuart Henderson wrote:

Hi, some quick feedback :

Makefile:
- "ONLY_FOR_ARCHS =  amd64 i386", why?

this was due to a build i tried on armv7 which failed with:

    Error: selected processor does not support `fmrx r3,fpscr'

i figured this was inline x86 assembly but your question promted
metolook further. turns out its an issue with floating point
instructionson arm.

i tried copying other ports with `--target=generic' and
`--arch=generic' in CONFIGURE_ARGS with no success. then i tried
CFLAGS+='-mfloat-abi=hard' which the base gcc must not support?
pulling in ports' gcc got me a bit further:

    {standard input}: Assembler messages:

    {standard input}:19: Error: unknown pseudo-op: `.vsave'


unsure how to proceed, uncertain what OpenBSD support
for hardware floating point is like on armv7.

{standard input}:18: Error: selected processor does not support `fstmfdd sp!,{d8,d9}' {standard input}:24: Error: selected processor does not support `fcpyd d9,d0' {standard input}:25: Error: selected processor does not support `fcpyd d8,d1' {standard input}:37: Error: selected processor does not support `fcpyd d0,d9' {standard input}:38: Error: selected processor does not support `fcpyd d1,d8' {standard input}:46: Error: selected processor does not support `fldmfdd ip!,{d8-d9}'

updated tarball attached, simply removed the offending
lines of assembly, runs _far_ slower on armv7 but
produces expected results. some research suggests that
our older binutils in base could be the culprit.

perhaps when clang/llvm get enabled on armv7 we can
lose this patch.

tested on armv7, macppc and amd64

- "#GPLv2 only" please add a space, "# GPLv2 only"
- "COMMENT = Library for ..." lower-case first letter -> "library for"

pkg/DESCR:
- don't list WWW, it comes automatically from HOMEPAGE in Makefile

pkg/PLIST, Makefile:
- the linux-style shared library handling needs modifying.

the attached tarball should have fixed these issues.

are things shaping up?

cheers,
toby/

rubberband.tar.gz
Description: application/gzip