Bug 222879 - [patch] misc/raspberrypi-userland: enable armv7 build
Summary: [patch] misc/raspberrypi-userland: enable armv7 build
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: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-09 13:31 UTC by Sylvain Garrigues
Modified: 2017-10-10 06:30 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (mikael)

Attachments
patch (279 bytes, patch)
2017-10-09 13:55 UTC, Mikael Urankar 		no flags Details | Diff
My working patch (237 bytes, patch)
2017-10-09 17:43 UTC, Sylvain Garrigues 		no flags Details | Diff
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 Sylvain Garrigues 2017-10-09 13:31:06 UTC 
Currently, the port fails with "is only for armv6, while you are running armv7".
Comment 1 Mikael Urankar freebsd_committer freebsd_triage 2017-10-09 13:55:44 UTC 
Created attachment 187025 [details]
patch
Comment 2 Sylvain Garrigues 2017-10-09 17:43:33 UTC 
Created attachment 187029 [details]
My working patch

Tested with this patch
Comment 3 Mikael Urankar freebsd_committer freebsd_triage 2017-10-09 19:29:54 UTC 
Comment on attachment 187029 [details]
My working patch

maintainer-approval++
Comment 4 commit-hook freebsd_committer freebsd_triage 2017-10-10 01:47:37 UTC 
A commit references this bug:

Author: linimon
Date: Tue Oct 10 01:46:52 UTC 2017
New revision: 451653
URL: https://svnweb.freebsd.org/changeset/ports/451653

Log:
  Enable on armv7.

  PR:		222879
  Submitted by:	Sylvain Garrigues <sylvain at syvlaingarrigues dot com>

Changes:
  head/misc/raspberrypi-userland/Makefile
Comment 5 Mark Linimon freebsd_committer freebsd_triage 2017-10-10 01:48:35 UTC 
Committed, thanks.

I used the longer form because I was not sure the short form would work.
Comment 6 Sylvain Garrigues 2017-10-10 06:30:42 UTC 
It does work with the regexp (code logic in bsd.port.mk is using a match test) but I’m fine with the longer form, thanks, closing the bug.