Re: [help wanted] productivity/monica initial setup needs some handling

On 6/7/22 12:15, Omar Polo wrote:
> aisha <openbsd.ports@aisha.cc> wrote:
>> attached patch which updates README and also installs it
>>
>> diff --git a/productivity/monica/Makefile b/productivity/monica/Makefile
>> index 2b0a16b7b99..845ba8aa1fc 100644
>> --- a/productivity/monica/Makefile
>> +++ b/productivity/monica/Makefile
>> @@ -3,6 +3,7 @@ COMMENT = self hosted personal CRM system
>> CATEGORIES = productivity
>>
>> VERSION = 3.7.0
>> +REVISION = 0
>> PKGNAME = monica-${VERSION}
>>
>> MASTER_SITES = https://github.com/monicahq/monica/releases/download/v${VERSION}/
>> diff --git a/productivity/monica/pkg/PLIST b/productivity/monica/pkg/PLIST
>> index d9600c56c9e..46becc63ca6 100644
>> --- a/productivity/monica/pkg/PLIST
>> +++ b/productivity/monica/pkg/PLIST
>> @@ -717,7 +717,7 @@ www/monica/app/ViewHelpers/ContactHelper.php
>> www/monica/artisan
>> www/monica/bootstrap/
>> www/monica/bootstrap/app.php
>> -@mode 775
>> +@mode 770
> not regarding the issue specifically discussed in this thread, but this
> bit caught my eye. We're installing some php files with the +x bit set,
> and they don't really seem to need it, so maybe something like the
> following should go together with this diff? (not run tested)
good catch :D
On my setup (running for a quite a while) the files haven't changed,
timestamp and data are both still the same from Feb 6.
I think this change is fine.
> Index: pkg/PLIST
> ===================================================================
> RCS file: /home/cvs/ports/productivity/monica/pkg/PLIST,v
> retrieving revision 1.1.1.1
> diff -u -p -r1.1.1.1 PLIST
> --- pkg/PLIST 21 May 2022 00:57:51 -0000 1.1.1.1
> +++ pkg/PLIST 7 Jun 2022 16:07:41 -0000
> @@ -717,15 +717,15 @@ www/monica/app/ViewHelpers/ContactHelper
> www/monica/artisan
> www/monica/bootstrap/
> www/monica/bootstrap/app.php
> -@mode 775
> +@mode 770
> @owner www
> @group www
> www/monica/bootstrap/cache/
> -www/monica/bootstrap/cache/packages.php
> -www/monica/bootstrap/cache/services.php
> @mode
> @owner
> @group
> +www/monica/bootstrap/cache/packages.php
> +www/monica/bootstrap/cache/services.php
> www/monica/composer.json
> www/monica/composer.lock
> www/monica/config/
>
> or maybe they're changed at runtime since they're in cache/?
>
>> @owner www
>> @group www
>> www/monica/bootstrap/cache/
>> @@ -20335,3 +20335,5 @@ www/monica/vendor/xantios/mimey/tests/src/MimeMappingGeneratorTest.php
>> www/monica/vendor/xantios/mimey/tests/src/MimeTypesTest.php
>> www/monica/webpack.mix.js
>> www/monica/yarn.lock
>> +@cwd ${LOCALBASE}
>> +share/doc/pkg-readmes/${PKGSTEM}
>> diff --git a/productivity/monica/pkg/README b/productivity/monica/pkg/README
>> index f99d1e4a095..40b1308f129 100644
>> --- a/productivity/monica/pkg/README
>> +++ b/productivity/monica/pkg/README
>> @@ -2,10 +2,23 @@
>> | Running ${PKGSTEM} on OpenBSD
>> +-------------------------------------------------------------------------------
>>
>> -Configuration file
>> -==================
>> +Configuration and setup
>> +=======================
>>
>> -The configuration file for monica is ${LOCALSTATEDIR}/www/monica/.env .
>> +The configuration file for monica is ${LOCALSTATEDIR}/www/monica/.env which
>> +should be configure according to the upstream documentation at
>> +https://github.com/monicahq/monica/blob/main/docs/installation/providers/generic.md
>> +
>> +The initial setup should be done as follows:
>> +
>> +$ cd ${LOCALSTATEDIR}/www/monica && \
>> + su -s /bin/ksh www -c "${MODPHP_BIN} artisan setup:production -v"
>> +
>> +Monica sets up absolute paths during the above setup, which can cause issues
>> +when using a web server, such as httpd(8) or nginx, which run in a chroot
>> +inside /var/www/. To work around this issue, create a relative symlink such as:
>> +
>> +$ ln -s .. /var/www/var/www
> nit: I'd use # instead of $ as prompt in both examples.
>
>> Exposing a webserver
>> ====================