Bug 230362 - net/csocks: Update to 1.7
Summary: net/csocks: Update to 1.7
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: Tobias Kortkamp
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-04 20:13 UTC by Nathan
Modified: 2018-08-18 12:10 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (raffaele.delorenzo)


Attachments
Update (6.18 KB, patch)
2018-08-04 20:13 UTC, Nathan
no flags Details | Diff
Poudriere Log 10.4-Release (17.15 KB, text/plain)
2018-08-04 20:14 UTC, Nathan
no flags Details
Update (6.15 KB, patch)
2018-08-04 20:24 UTC, Nathan
no flags Details | Diff
Update (6.16 KB, patch)
2018-08-04 20:28 UTC, Nathan
no flags Details | Diff
Poudriere Log 11.2-Stable (17.64 KB, text/plain)
2018-08-04 22:08 UTC, Nathan
no flags Details
Update (1.78 KB, patch)
2018-08-04 22:38 UTC, Nathan
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nathan 2018-08-04 20:13:05 UTC
Created attachment 195856 [details]
Update

Changes:
3 July 2013 Version 1.7:
  - Solved bug in UPDv6 management. Protocol was not set properly.

10 May 2011 Version 1.6:
	- Optimize Hash Table rules
	- Added IPv6 Compatibility on new Linux Header.

Portlint OK
Comment 1 Nathan 2018-08-04 20:14:19 UTC
Created attachment 195857 [details]
Poudriere Log 10.4-Release
Comment 2 Nathan 2018-08-04 20:24:21 UTC
Created attachment 195858 [details]
Update
Comment 3 Nathan 2018-08-04 20:28:56 UTC
Created attachment 195859 [details]
Update
Comment 4 Nathan 2018-08-04 22:08:57 UTC
Created attachment 195860 [details]
Poudriere Log 11.2-Stable
Comment 5 Tobias Kortkamp freebsd_committer freebsd_triage 2018-08-04 22:26:28 UTC
Comment on attachment 195859 [details]
Update

+USES=		compiler:gcc-c++11-lib

Does it not build with Clang anymore?  Why does it require GCC now?
Forcing GCC usage in a port should be a last resort.

-	${INSTALL_LIB} ${WRKDIR}/src/libcsocks.so.1 ${STAGEDIR}${PREFIX}/lib
-	${LN} -s libcsocks.so.1 ${STAGEDIR}${PREFIX}/lib/libcsocks.so
+	${INSTALL_LIB} ${WRKDIR}/src/libcsocks.so.1 ${STAGEDIR}${PREFIX}/lib/libcsocks.so.1
+	${INSTALL_LIB} ${WRKDIR}/src/libcsocks.so.1 ${STAGEDIR}${PREFIX}/lib/libcsocks.so

This is a bogus change.  You're now installing libcsocks.so.1 twice
(so the package will take twice the space).  libcsocks.so should be a symlink.
Please revert this.

--- net/csocks/files/patch-csocks.c	(nonexistent)
+++ net/csocks/files/patch-csocks.c	(working copy)

net/csocks/files/patch-csocks.c looks fishy.  Why is it needed and why
does it remove a bunch of NULL checks?
Comment 6 Nathan 2018-08-04 22:38:31 UTC
Created attachment 195868 [details]
Update

Compiler was warning, during manual build as I was messing with it, that NULL pointers was always zero, like it wasn't needed; but anyway reverted what was requested
Comment 7 commit-hook freebsd_committer freebsd_triage 2018-08-18 12:06:22 UTC
A commit references this bug:

Author: tobik
Date: Sat Aug 18 12:05:40 UTC 2018
New revision: 477498
URL: https://svnweb.freebsd.org/changeset/ports/477498

Log:
  net/csocks: Update to 1.7

  - Add LICENSE

  PR:		230362
  Submitted by:	Nathan <ndowens@yahoo.com> (based on)
  Approved by:	maintainer timeout (2 weeks)

Changes:
  head/net/csocks/Makefile
  head/net/csocks/distinfo
  head/net/csocks/files/patch-Makefile
  head/net/csocks/pkg-descr
Comment 8 Tobias Kortkamp freebsd_committer freebsd_triage 2018-08-18 12:06:47 UTC
Committed with some changes. Thanks!
Comment 9 commit-hook freebsd_committer freebsd_triage 2018-08-18 12:10:28 UTC
A commit references this bug:

Author: tobik
Date: Sat Aug 18 12:10:13 UTC 2018
New revision: 477499
URL: https://svnweb.freebsd.org/changeset/ports/477499

Log:
  net/csocks: Remove bits that probably shouldn't be there

  PR:		230362

Changes:
  head/net/csocks/Makefile
  head/net/csocks/files/patch-Makefile