Created attachment 197316 [details] mattermost-server-5.3.1.diff Hi, the mattermost port is fairly outdated. Here are the patches.
Created attachment 197317 [details] mattermost-webapp-5.3.1.diff
The server and webapp should be separate PRs
Hello, Nathan please note both packages needs the other side. Oliver, thanks for your contribution. Can you keep the alphabetic order in the MV in server ? Can you also add the poudriere testport output ?
Created attachment 197519 [details] mattermost-webapp-5.3.1_v2.diff Updated mattermost-webapp.diff to fix orphaned files.
Created attachment 197520 [details] mattermost-server-5.3.1.log mattermost-server testport output
Created attachment 197521 [details] mattermost-webapp-5.3.1.log mattermost-webapp testport output
Trying to start mattermostd gives: ------------------------------------ ERROR ------------------------------------------------ The platform binary has been deprecated, please switch to using the new mattermost binary. The platform binary will be removed in a future version. ------------------------------------------------------------------------------------------- Could not start Mattermost, use the mattermost command directly: failed to find mattermost Looks like we just need to change line 47 from copying "platform" to copying "mattermost". That sound right to you?
It's fine, they deprecate the platform to use the proper binary. The rc.d file should be changed
(In reply to loic.blot from comment #8) What change(s) need to be made to the rc script?
Created attachment 198631 [details] mattermost-webapp-5.4.0.diff
Created attachment 198632 [details] mattermost-server-5.4.0.diff
* Update to Mattermost 5.4.0 * changed platform binary to new mattermost binary Poudriere testport is on its way.
Created attachment 198634 [details] mattermost-webapp-5.4.0.diff
Created attachment 198652 [details] mattermost-webapp-5.4.0.log
Created attachment 198653 [details] mattermost-server-5.4.0.log
Please approve the patches if they look OK. Thanks!
if i'm correct you miss to update the rc.d script to use the correct binary. Else it's okay to me
(In reply to loic.blot from comment #17) The do-install target in the server port was changed to copy the "mattermost" binary instead of the "platform" binary, but the target of the copy remains the same, so I think the rc script doesn't need updating for that. If you approve the changes, please set the maintainer-approval flag on the patch file attachments to "+" instead of "?".
hi steve, it's an obsolete flag i forgot to update. But currently it's good for me.
A commit references this bug: Author: swills Date: Wed Oct 31 13:07:59 UTC 2018 New revision: 483573 URL: https://svnweb.freebsd.org/changeset/ports/483573 Log: www/mattermost-webapp www/mattermost-server: Update to 5.4.0 PR: 231559 Submitted by: Oliver B?ttcher <marvin0815@gmail.com> Approved by: loic.blot@unix-experience.fr (maintainer) Changes: head/www/mattermost-server/Makefile head/www/mattermost-server/distinfo head/www/mattermost-server/pkg-plist head/www/mattermost-webapp/Makefile head/www/mattermost-webapp/distinfo head/www/mattermost-webapp/pkg-plist
Committed, thanks!