Bug 216191 - multimedia/libvpx: grab and drop architecture whitelist
Summary: multimedia/libvpx: grab and drop architecture whitelist
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: Jan Beich
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-01-17 18:44 UTC by Jan Beich
Modified: 2017-01-22 09:08 UTC (History)
2 users (show)

See Also:
ashish: maintainer-feedback+

Attachments
v0 (4.08 KB, patch)
2017-01-17 18:44 UTC, Jan Beich 		no flags Details | Diff
v0.1 (4.14 KB, patch)
2017-01-17 19:29 UTC, Jan Beich 		jbeich: maintainer-approval? (ashish)
 Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Beich freebsd_committer freebsd_triage 2017-01-17 18:44:37 UTC 
Created attachment 179003 [details]
v0

armv6 was dropped in v1.6.1 per https://chromium.googlesource.com/webm/libvpx/+/d55724fae9cb%5E%21/
Rather than pointlessly populate synthetic platforms let's fallback to
generic-gnu like other downstream builds.

I guess this means taking maintainership. ;)
Comment 1 Jan Beich freebsd_committer freebsd_triage 2017-01-17 19:00:39 UTC 
Comment on attachment 179003 [details]
v0

bsd.cpu.mk has some magic for armv6, so folks should use CPUTYPE=cortexa or similar than passing -march to CFLAGS directly. I'm not sure if we actually support armv8 on armv6 or it requires aarch32 target added in src/.
Comment 2 Jan Beich freebsd_committer freebsd_triage 2017-01-17 19:03:06 UTC 
Comment on attachment 179003 [details]
v0

> -+ASFLAGS+=-I$(BUILD_PFX)$(BUILD_ROOT)/ -I$(SRC_PATH)/ -meabi=5

Oops, removing this was premature.

/usr/bin/ld: ERROR: Source object vpx_dsp/arm/intrapred_neon_asm.asm.S.o has EABI version 0, but target libvpx.so.4.1.0 has EABI version 5
/usr/bin/ld: failed to merge target specific data of file vpx_dsp/arm/intrapred_neon_asm.asm.S.o
/usr/bin/ld: ERROR: Source object vpx_dsp/arm/vpx_convolve_copy_neon_asm.asm.S.o has EABI version 0,but target libvpx.so.4.1.0 has EABI version 5
/usr/bin/ld: failed to merge target specific data of file vpx_dsp/arm/vpx_convolve_copy_neon_asm.asm.S.o
Comment 3 Jan Beich freebsd_committer freebsd_triage 2017-01-17 19:29:40 UTC 
Created attachment 179005 [details]
v0.1
Comment 4 Ashish SHUKLA freebsd_committer freebsd_triage 2017-01-22 05:17:17 UTC 
(In reply to Jan Beich (mail not working) from comment #3)

Howdy!

I'm able to test it on AMD64 platform, where it seems to build as expected. 

Also, thanks for taking the maintainership, you have my approval. Please go ahead, and commit.

Thanks!
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-01-22 09:04:24 UTC 
A commit references this bug:

Author: jbeich
Date: Sun Jan 22 09:03:56 UTC 2017
New revision: 432101
URL: https://svnweb.freebsd.org/changeset/ports/432101

Log:
  multimedia/libvpx: grab and drop architecture whitelist

  PR:		216191
  Approved by:	ashish (previous maintainer)
  MFH:		2017Q1

Changes:
  head/multimedia/libvpx/Makefile
  head/multimedia/libvpx/files/extra-patch-build_make_Makefile
  head/multimedia/libvpx/files/patch-configure
Comment 6 commit-hook freebsd_committer freebsd_triage 2017-01-22 09:07:28 UTC 
A commit references this bug:

Author: jbeich
Date: Sun Jan 22 09:07:04 UTC 2017
New revision: 432102
URL: https://svnweb.freebsd.org/changeset/ports/432102

Log:
  MFH: r432101

  multimedia/libvpx: grab and drop architecture whitelist

  PR:		216191
  Approved by:	ashish (previous maintainer)
  Approved by:	ports-secteam blanket (unbreaks armv6 after r431741)

Changes:
_U  branches/2017Q1/
  branches/2017Q1/multimedia/libvpx/Makefile
  branches/2017Q1/multimedia/libvpx/files/extra-patch-build_make_Makefile
  branches/2017Q1/multimedia/libvpx/files/patch-configure