Bug 241381

Summary: Add option to enable OMEMO to net-im/profanity via make config
Product: Ports & Packages Reporter: pvoigt
Component: Individual Port(s)Assignee: Tilman Keskinoz <arved>
Status: Closed FIXED    
Severity: Affects Many People CC: swills
Priority: --- Flags: bugzilla: maintainer-feedback? (arved)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patched Makefile
none
working patch none

Description pvoigt 2019-10-20 21:16:29 UTC 
Created attachment 208470 [details]
Patched Makefile

The current make config options of net-im/profanity do not allow to build with OMEMO enabled. According to the original source code it can be enabled:
https://profanity-im.github.io/guide/070/omemo.html.

As I needed OMEMO support I did the following:

1.) I added "--enable-omemo" to CONFIGURE_ARGS of /usr/ports/net-im/profanity/Makefile

2.) As configure complained about missing libsignal I installed port net/libsignal and rebuilt profanity.

With these two steps net-im/profanity could be built with OMEMO enabled. According to my tests OMEMO support works as expected.

Please find the patch for the Makefile attached.
Comment 2 pvoigt 2019-11-08 15:07:37 UTC 
Looks like net/libsignal does not build under FreeBSD 12.0-RELEASE-p10 amd64.

I have built under FreeBSD 11.3-RELEASE-p4 and net/libsignal-2.3.2.

Thus OMEMO works reliably in chats and multi-user-chats.
Comment 3 pvoigt 2019-11-08 15:50:10 UTC 
I strongly suppose for automatically build required net/libsignal the Makefile needs an additional patch, e.g. something under LIB_DEPENDS= like:

libsignal-protocol-c.so:net/libsignal

Unfortunately, my knowledge of Makefiles is rather limited. The port maintainer, arved@FreeBSD.org, will know this for sure.

@Steve:
If net/libsignal is built manually before net-im/profanity should build under FreeBSD 12.0-RELEASE-p10 amd64 as well.
Comment 4 Tilman Keskinoz freebsd_committer freebsd_triage 2019-11-08 17:41:07 UTC 
Created attachment 208980 [details]
working patch

patch seems to work
Comment 5 pvoigt 2019-11-08 18:21:59 UTC 
@Tilman: Thanks for putting the bricks together in clean Makefile patch. I suppose OMEMO will now be enabled by default and cannot be disabled by "make config". For me this totally fine.
Comment 6 Tilman Keskinoz freebsd_committer freebsd_triage 2019-11-11 17:30:38 UTC 
committed my patch. Thanks for reporting
Comment 7 commit-hook freebsd_committer freebsd_triage 2019-11-11 17:30:56 UTC 
A commit references this bug:

Author: arved
Date: Mon Nov 11 17:30:17 UTC 2019
New revision: 517269
URL: https://svnweb.freebsd.org/changeset/ports/517269

Log:
  Enable OMEMO support
  Bump PORTREVISION

  PR:		241381
  Reported by:	pvoigt@uos.de

Changes:
  head/net-im/profanity/Makefile