Created attachment 170077 [details] freeswitch-1.6.8 svn diff Maintainer update to freeswitch 1.6.8 svn diff attached. The following svn commands were used before the diff was created: svn remove files/patch-src_mod_applications_mod__http__cache_mod__http__cache.c svn remove files/patch-src_mod_event__handlers_mod__rayo_xmpp__streams.c svn move files/patch-modules.conf files/extrapatch-modules.conf Changes included in update: * Updated from 1.6.7 to 1.6.8 * Removed unnecessary patches * explicitly depend on perl5 with USES * Added ALL_MODULES knob to apply the extrapatch to modules.conf to enable all modules that successfully build on FreeBSD and their associated dependencies * modules are now installed stripped * cleaned-up the MASTER_SITES:N functionality for sounds * mod_avmd no longer builds on FreeBSD - Will work with upstream to fix for next release The previous version of the port would enable every module that would build on FreeBSD. This version of the port builds the standard set of modules unless the ALL_MODULES knob is enabled. The previous behaviour is restored when ALL_MODULES is enabled. This allows the standard freeswitch user on FreeBSD to have a much smaller installation footprint and attack surface. Power users can get a tested "batteries included" build by simply enabling the ALL_MODULES knob at build time. Using the standard modules.conf means that lame is not a dependency which means fresh installs of FreeBSD can simply "pkg install freeswitch" in the future and be up and running in minutes. This fails today because of the lame dependency which can not be distributed as a binary due to patent restrictions.
Can you check the files/extrapatch-modules.conf ? It's not in the current freeswitch port, so the patch fails to apply cleanly ?
Created attachment 170126 [details] full contents of extrapatch-modules.conf
(In reply to Kurt Jaeger from comment #1) The original was called patch-modules.conf. I ran this command before making the svn diff: svn move files/patch-modules.conf files/extrapatch-modules.conf I attached the full contents of the file to the ticket for your reference.
(In reply to corsmith from comment #3) Thanks, got it, testbuilding right now.
Testbuilds failed on: http://people.freebsd.org/~pi/logs/net__freeswitch-cur-1462772464.txt http://people.freebsd.org/~pi/logs/net__freeswitch-103-1462772464.txt http://people.freebsd.org/~pi/logs/net__freeswitch-10i-1462772464.txt http://people.freebsd.org/~pi/logs/net__freeswitch-93a-1462772464.txt
Could you please remove dependency on openssl port? Explained in: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=209196
testbuilds with the WITH_OPENSSL_PORT line removed @work.
(In reply to Kurt Jaeger from comment #7) The build will complete but the ssl support will not be enabled. You will notice in the test build log that the ssl library is not found in the configure phase. The openssl port is required because the configure uses pkg-config to determine if the library exists and what the build options are. The system openssl does not install an openssl.pc file so pkg-config fails to detect it. An upstream patch for configure would be required for freeswitch to detect the system openssl.
(In reply to corsmith from comment #8) Testbuilds still fail: http://people.freebsd.org/~pi/logs/net__freeswitch-103-1462772464.txt http://people.freebsd.org/~pi/logs/net__freeswitch-10i-1462772464.txt http://people.freebsd.org/~pi/logs/net__freeswitch-93a-1462772464.txt http://people.freebsd.org/~pi/logs/net__freeswitch-cur-1462772464.txt
Regarding removing openssl port dependency: Please, look at my last comment in: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=209196
So, what's next ? We have a 1.6.8 which does not build -- any ideas on how to get it building in poudriere ?
Created attachment 170402 [details] Update to 1.6.8 Update to 1.6.8 with configurable lame support and removed dependency on openssl port.
Created attachment 170404 [details] svn diff Update to 1.6.8 with configurable lame support and removed dependency on openssl port (corrected previous attachment).
(In reply to numisemis from comment #13) The distinfo changes are not acceptable as they remove all of the other language files. I will re-upload an original version of my diff with the WITH_OPENSSL_PORT option removed.
(In reply to Kurt Jaeger from comment #11) It looks like replies in email form do not make it back into the case in bugzilla. I built the version sent with the svn diff under poudriere. The AMD64/10.3-RELEASE build is failing due to a pkg-plist error: http://people.freebsd.org/~pi/logs/net__freeswitch-103-1462772464.txt Are you sure you are using my copy of the pkg-plist I sent in the patch? The i386/10.3-RELEASE fails as expected (only builds on AMD64) http://people.freebsd.org/~pi/logs/net__freeswitch-10i-1462772464.txt Not sure about AMD64/9.3-RELEASE. I'm guessing it is CLANG vs GCC problem. We already have BROKEN_FreeBSD_9= Fails in configure due to compiler issues http://people.freebsd.org/~pi/logs/net__freeswitch-93a-1462772464.txt
Created attachment 170420 [details] freeswitch-1.6.8 svn diff version 2 freeswitch-1.6.8 svn diff - version 2 Maintainer update to freeswitch 1.6.8 svn diff attached. The following svn commands were used before the diff was created: svn remove files/patch-src_mod_applications_mod__http__cache_mod__http__cache.c svn remove files/patch-src_mod_event__handlers_mod__rayo_xmpp__streams.c svn move files/patch-modules.conf files/extrapatch-modules.conf svn add files/patch-src_mod_applications_mod__avmd_avmd__fast__acosf.c Changes included in update: * Updated from 1.6.7 to 1.6.8 * Removed unnecessary patches * explicitly depend on perl5 with USES * Added ALL_MODULES knob to apply the extrapatch to modules.conf to enable all modules that successfully build on FreeBSD and their associated dependencies * modules are now installed stripped * cleaned-up the MASTER_SITES:N functionality for sounds * mod_avmd patch from numisemis@yahoo.com added The previous version of the port would enable every module that would build on FreeBSD. This version of the port builds the standard set of modules unless the ALL_MODULES knob is enabled. The previous behaviour is restored when ALL_MODULES is enabled. This allows the standard freeswitch user on FreeBSD to have a much smaller installation footprint and attack surface. Power users can get a tested "batteries included" build by simply enabling the ALL_MODULES knob at build time. Using the standard modules.conf means that lame is not a dependency which means fresh installs of FreeBSD can simply "pkg install freeswitch" in the future and be up and running in minutes. This fails today because of the lame dependency which can not be distributed as a binary due to patent restrictions.
testbuilds@work
Testbuilds fail in poudriere: http://people.freebsd.org/~pi/logs/net__freeswitch-93a-1463549967.txt http://people.freebsd.org/~pi/logs/net__freeswitch-103-1463549967.txt http://people.freebsd.org/~pi/logs/net__freeswitch-cur-1463549967.txt
Created attachment 170482 [details] svn diff for patch-configure that allows configure to find lua5.3 Attached is a patch to configure to allow it to find lua5.3. The current default lua version is 5.2 and the build works in this case without the patch.
(In reply to Kurt Jaeger from comment #18) This appears to be caused by a non-standard DEFAULT_VERSIONS=lua=5.3 in the testbuild environment. If lua5.3 support is required then please add the attached additional configure patch to enable.
(In reply to corsmith from comment #20) Done, looks very nice! Thanks!
Committed, thanks!
A commit references this bug: Author: pi Date: Thu May 19 18:54:03 UTC 2016 New revision: 415524 URL: https://svnweb.freebsd.org/changeset/ports/415524 Log: net/freeswitch: 1.6.7 -> 1.6.8 - Removed unnecessary patches - explicitly depend on perl5 with USES - Added ALL_MODULES knob to apply the extrapatch to modules.conf to enable all modules that successfully build on FreeBSD and their associated dependencies - modules are now installed stripped - cleaned-up the MASTER_SITES:N functionality for sounds - mod_avmd no longer builds on FreeBSD - Will work with upstream to fix for next release PR: 209353 Submitted by: corsmith@gmail.com (maintainer) Reviewed by: numisemis@yahoo.com Changes: head/net/freeswitch/Makefile head/net/freeswitch/distinfo head/net/freeswitch/files/extrapatch-modules.conf head/net/freeswitch/files/patch-configure head/net/freeswitch/files/patch-modules.conf head/net/freeswitch/files/patch-src_mod_applications_mod__avmd_avmd__fast__acosf.c head/net/freeswitch/files/patch-src_mod_applications_mod__http__cache_mod__http__cache.c head/net/freeswitch/files/patch-src_mod_event__handlers_mod__rayo_xmpp__streams.c head/net/freeswitch/pkg-plist