Bug 218730 - print/fontforge remove clang36
Summary: print/fontforge remove clang36
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: Martin Wilke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-18 14:19 UTC by Martin Wilke
Modified: 2017-04-23 03:28 UTC (History)
1 user (show)

See Also:
cyberbotx: maintainer-feedback+


Attachments
remove clang36 dependency (830 bytes, patch)
2017-04-18 14:19 UTC, Martin Wilke
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Wilke freebsd_committer freebsd_triage 2017-04-18 14:19:51 UTC
Created attachment 181867 [details]
remove clang36 dependency

Hi,

attached patch removes clang36 dependency, it builds and works fine with the base compiler.
Comment 1 Naram Qashat 2017-04-18 14:25:29 UTC
Can I assume this was tested with i386 as opposed to amd64? That was the reason why the clang dependency was set as it was, due to the bug listed there that dim@ reported upstream. If it has been tested under i386 and it works, then I approve of this.
Comment 3 Martin Wilke freebsd_committer freebsd_triage 2017-04-23 03:13:30 UTC
Hi Naram,

Are you ok with this change?

Thanks Martin
Comment 4 Naram Qashat 2017-04-23 03:19:00 UTC
Comment on attachment 181867 [details]
remove clang36 dependency

Yeah, I'm OK with it.
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-04-23 03:28:23 UTC
A commit references this bug:

Author: miwi
Date: Sun Apr 23 03:28:16 UTC 2017
New revision: 439195
URL: https://svnweb.freebsd.org/changeset/ports/439195

Log:
  - Remove clang36 dependency

  PR:		218730
  Submitted by:	myself
  Approved by:	maintainer

Changes:
  head/print/fontforge/Makefile
Comment 6 Martin Wilke freebsd_committer freebsd_triage 2017-04-23 03:28:53 UTC
Committed. Thank you for your quick respond.