Summary: | www/nextcloud: Fix APCu optionality, add non-ASCII support | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | fsbruva | ||||||
Component: | Individual Port(s) | Assignee: | Bernard Spil <brnrd> | ||||||
Status: | Closed FIXED | ||||||||
Severity: | Affects Only Me | Flags: | bugzilla:
maintainer-feedback?
(brnrd) |
||||||
Priority: | --- | ||||||||
Version: | Latest | ||||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
Attachments: |
|
Created attachment 197557 [details]
Correct APCu behavior, add intl option
Corrected alpha order w/in patch.
Hi Matt, Thanks for the heads-up. Copy-paste mistake. As far as I know, this is not going to work for the APCU=ON as the OPT_RUN_DEPENDS only works above bsd.port.pre include. I'll test that though. make -VRUN_DEPENDS would show. A commit references this bug: Author: brnrd Date: Fri Sep 28 13:52:13 UTC 2018 New revision: 480860 URL: https://svnweb.freebsd.org/changeset/ports/480860 Log: www/nextcloud: Fix APCU option, add INTL PR: 231769 Submitted by: Matt <fsbruva yahoo com> Changes: head/www/nextcloud/Makefile Again, thanks! Keep 'm coming Matt! Yep - I never tested with APCu=ON, sorry for that. Out of curiosity, why didn't https://svnweb.freebsd.org/changeset/ports/480860 bump PORTREVISION? (In reply to Matt from comment #5) Good question! The APCu change does not lead to a different package with the default options, so this does not warrant a bump. Looking at the change again, the APCu option doesn't change the package, yet the intl option does change the dependencies. That should does warrant a bump. |
Created attachment 197556 [details] Correct APCu behavior, add intl option The latest version of the port forces APCu to be installed, disregarding the selected value for the APCu option. Fix this so deselecting the option prevents APCu from being installed. Additionally, add international (ICU) support for improved sorting.