Bug 242586

Summary: [MAINTAINER] www/rss-bridge: update to 2019-12-01
Product: Ports & Packages Reporter: Oleg Sidorkin <osidorkin>
Component: Individual Port(s)Assignee: Dmitry Marakasov <amdmi3>
Status: Closed FIXED    
Severity: Affects Only Me Keywords: buildisok
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch to update www/rss-bridge to 2019-12-01 release (obtained by svn diff)
none
Improved patch none

Description Oleg Sidorkin 2019-12-11 21:24:55 UTC
Created attachment 209877 [details]
patch to update www/rss-bridge to 2019-12-01 release (obtained by svn diff)

Attached patch contains update of www/rss-bridge port to 2019-12-01 release
Comment 1 Automation User 2019-12-11 21:38:29 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/102429496
Comment 2 Dmitry Marakasov freebsd_committer freebsd_triage 2019-12-27 23:36:58 UTC
Created attachment 210269 [details]
Improved patch

How about this improved patch?

Use options helpers, simplify the port and fix version handling.
Comment 3 Oleg Sidorkin 2019-12-28 09:15:38 UTC
(In reply to Dmitry Marakasov from comment #2)
Thanks for contributing.

I'm ok with this patch (however it's better to drop %%SQLITE%% prefix from pkg-plist entry also).
Comment 4 Dmitry Marakasov freebsd_committer freebsd_triage 2019-12-28 17:36:42 UTC
(In reply to Oleg Sidorkin from comment #3)
No, this is intentional. %%SQLITE%% controls whether SQLiteCache.php gets into the package depending on the option, so the behavior of not installing SQLiteCache.php when SQLITE is disabled is retained. My patch just removes needless duplication of that logic with COPY_SKIP_OPT.
Comment 5 commit-hook freebsd_committer freebsd_triage 2019-12-28 17:38:13 UTC
A commit references this bug:

Author: amdmi3
Date: Sat Dec 28 17:37:17 UTC 2019
New revision: 521226
URL: https://svnweb.freebsd.org/changeset/ports/521226

Log:
  - Update to 2019-12-01
  - Simplify the port

  PR:		242586
  Submitted by:	osidorkin@gmail.com (maintainer)

Changes:
  head/www/rss-bridge/Makefile
  head/www/rss-bridge/distinfo
  head/www/rss-bridge/pkg-plist