Bug 225686

Summary: net/drawterm: switch to 9front fork, add DragonflyBSD compatibility, remove patches
Product: Ports & Packages Reporter: Piotr Kubaj <pkubaj>
Component: Individual Port(s)Assignee: Tobias Kortkamp <tobik>
Status: Closed Feedback Timeout    
Severity: Affects Only Me CC: winston
Priority: --- Keywords: needs-patch, needs-qa
Version: LatestFlags: bugzilla: maintainer-feedback? (winston)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch
none
patch none

Description Piotr Kubaj freebsd_committer freebsd_triage 2018-02-05 15:27:31 UTC
Created attachment 190342 [details]
patch

This patch switches drawterm port to use 9front's fork. It also adds DragonflyBSD compatibility.

Tested on Poudriere on FreeBSD.
Comment 1 winston 2018-03-04 21:04:10 UTC
Hello, I have contacted Piotr and they're interested in taking over this port. Thanks, Piotr!
Comment 2 Piotr Kubaj freebsd_committer freebsd_triage 2018-03-21 15:40:50 UTC
Created attachment 191706 [details]
patch

* update to a newer version which includes a manpage
* respect CFLAGS
* add -fPIC to CFLAGS
* change maintainer
Comment 3 Tobias Kortkamp freebsd_committer freebsd_triage 2019-01-02 09:13:19 UTC
This needs to be locked to a specific commit.  Using 'tip' isn't appropriate
as it changes over time for every new upstream commit.

It would also be nice to give the distfile a unique filename (probably using
the /?dummy= trick other ports use).

+.if ${OPSYS} == "FreeBSD"
+MAKE_ARGS=	CONF=freebsd
+.else
+MAKE_ARGS=	CONF=dragonfly
+.endif

Since this set after including bsd.port.pre.mk it should be MAKE_ARGS+= or it
overwrites MAKE_ARGS as set by the framework, but all of this can be done
declaratively without the bsd.port.{pre,post}.mk dance with just

MAKE_ARGS=	CONF=${OPSYS:tl}