ping On 2 Feb. 2017 8:46 am, "Tobias Brodel" <[email protected]> wrote:
> ping > > On 26 Jan. 2017 7:02 pm, "Tobias Brodel" <[email protected]> wrote: > >> whoops, wrong tarball. sorry for the noise! >> >> On 01/26/17 18:59, Tobias Brodel wrote: >> >>> 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}:18: Error: selected processor does not support >>>> `fstmfdd sp!,{d8,d9}' >>>> {standard input}:19: Error: unknown pseudo-op: `.vsave' >>>> {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}' >>>> >>>> unsure how to proceed, uncertain what OpenBSD support >>>> for hardware floating point is like on armv7. >>>> >>>> >>> 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/ >>> >> >>
