Bug 194853

Summary: [maintainer-update] net-mgmt/ccnet: clean Makefile and add supports to other BSDs
Product: Ports & Packages Reporter: Jingfeng Yan <yan_jingfeng>
Component: Individual Port(s)Assignee: John Marino <marino>
Status: Closed FIXED    
Severity: Affects Many People CC: marino, yan_jingfeng
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
diff from existing port
none
diff from existing port
none
test port log none

Description Jingfeng Yan 2014-11-06 04:07:08 UTC
Based on 193134's input, something in ccnet should be updated to make thing more consistent.

(1) use ${LN} instead of ${INSTALL} for 2nd copy.
(2) __FreeBSD__, __DragonFly__ ... supports

Running poudriere now. Will post result in a while.
Comment 1 Jingfeng Yan 2014-11-06 04:09:43 UTC
Created attachment 149107 [details]
diff from existing port
Comment 2 Jingfeng Yan 2014-11-06 04:11:39 UTC
Created attachment 149108 [details]
diff from existing port
Comment 3 Jingfeng Yan 2014-11-06 04:40:03 UTC
Created attachment 149117 [details]
test port log
Comment 4 John Marino freebsd_committer freebsd_triage 2014-11-06 19:23:43 UTC
FYI, there's no point in supporting __NetBSD__ or __OpenBSD__ until you intend to give the same patches to the upstream source (or pkgsrc).  Only DragonFly and FreeBSD use the ports system.
Comment 5 commit-hook freebsd_committer freebsd_triage 2014-11-06 19:33:08 UTC
A commit references this bug:

Author: marino
Date: Thu Nov  6 19:32:45 UTC 2014
New revision: 372231
URL: https://svnweb.freebsd.org/changeset/ports/372231

Log:
  net-mgmt/ccnet: Fix post-install and support other BSDs

  PR:		194853
  Submitted by:	maintainer (Jingfeng Yan)

Changes:
  head/net-mgmt/ccnet/Makefile
  head/net-mgmt/ccnet/files/patch-lib_utils.c
  head/net-mgmt/ccnet/files/patch-net_common_getgateway.c
Comment 6 John Marino freebsd_committer freebsd_triage 2014-11-06 19:34:55 UTC
I tweaked this to look like net-mgmt/seafile.

In the end, it didn't make sense to spend 6 lines on 2 installations, so I removed the loop completely and just had two install lines.
Comment 7 Jingfeng Yan 2014-11-07 02:12:35 UTC
(In reply to John Marino from comment #6)
> I tweaked this to look like net-mgmt/seafile.
> 
> In the end, it didn't make sense to spend 6 lines on 2 installations, so I
> removed the loop completely and just had two install lines.

Thank you for your supports and efforts.