Summary: | net/opensips: Update to 2.1.1 + port improvements. | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Euan Thoms <euan> | ||||||||||||||
Component: | Individual Port(s) | Assignee: | Kurt Jaeger <pi> | ||||||||||||||
Status: | Closed FIXED | ||||||||||||||||
Severity: | Affects Some People | CC: | franz, pi | ||||||||||||||
Priority: | --- | Keywords: | needs-patch, patch, patch-ready | ||||||||||||||
Version: | Latest | ||||||||||||||||
Hardware: | Any | ||||||||||||||||
OS: | Any | ||||||||||||||||
Attachments: |
|
Portlint passes: [root@portdev /usr/ports/net/opensips]# portlint -AC WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy. 0 fatal errors and 1 warning found. Created attachment 163464 [details]
Poudriere testport log on 10.1 RELEASE amd64
Poudriere testport passes on 10.1 amd64.
Euan, You should be able to use <OPTION>_VARS_ON=EXTRA_MODULES+=proto_tls instead of the .if ${PORT_OPTIONS:MTLS} block. Don't forget to set maintainer-approval to + on attachments for ports you are maintainer of (In reply to Kubilay Kocak from comment #3) Aha! Maybe that's why my ports are not getting committed fast enough. I didn't notice the maintainer approval thingy. *** Bug 204660 has been marked as a duplicate of this bug. *** Created attachment 163494 [details]
Diff patch between current port version and upgrade to v2.1.1
Improved options helpers (to new standards) in Makefile.
Comment on attachment 163494 [details]
Diff patch between current port version and upgrade to v2.1.1
options helpers breaks build
Created attachment 163497 [details]
Diff patch between current port version and upgrade to v2.1.1
Fixed tab indentation on options helpers.
Comment on attachment 163497 [details]
Diff patch between current port version and upgrade to v2.1.1
More breakage by options helpers changes.
I've given up trying to convert the EXTRA_MODULES+= section to the new options helpers syntax.
The following:
>>>>>>>>>>>>>
pre-configure-MYSQL-on:
EXTRA_MODULES+=db_mysql
<<<<<<<<<<<<<
results in:
===> Configuring for opensips-2.1.1
EXTRA_MODULES+=db_mysql
EXTRA_MODULES+=db_mysql: not found
*** Error code 127
Created attachment 163510 [details]
Diff patch between current port version and upgrade to v2.1.1
Reverted options helpers for setting EXTRA_MODULES. Tested with poudriere. Ready to commit now.
Created attachment 163511 [details]
Poudriere testport log on 10.1 RELEASE amd64
Updated poudriere testport log for 10.1 RELEASE amd64.
Testing@work A commit references this bug: Author: pi Date: Wed Nov 25 19:28:01 UTC 2015 New revision: 402407 URL: https://svnweb.freebsd.org/changeset/ports/402407 Log: net/opensips: 2.1.0 -> 2.1.1 Changes: http://opensips.org/pub/opensips/2.1.1/ChangeLog PR: 204773 Submitted by: Euan Thoms <euan@potensol.com> (maintainer) Changes: head/net/opensips/Makefile head/net/opensips/distinfo head/net/opensips/files/Makefile.conf head/net/opensips/files/patch-Makefile head/net/opensips/files/patch-Makefile.defs head/net/opensips/files/patch-io_wait.h head/net/opensips/files/patch-mem__shm_mem.h head/net/opensips/files/patch-modules__proto_tls__proto_tls.c head/net/opensips/files/patch-net__net_tcp.c head/net/opensips/files/patch-net__net_tcp_proc.c head/net/opensips/pkg-plist Committed, thanks! |
Created attachment 163462 [details] Diff patch between current port version and upgrade to v2.1.1 Update to version 2.1.1. Also fixed TLS support by including proto_tls module. Added cachedb_redis module to core of port (since no extra deps are required). Some tidying up and improvements in Makefile and pkg-plist.