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

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 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 2017-10-09 19:29:54 UTC
Comment on attachment 187029 [details]
My working patch

maintainer-approval++
Comment 4 commit-hook freebsd_committer 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.