Maia Mailgaurd / PHP 7 support

[Janky Jay, III]

Hi Miroslav,

On 12/29/18 6:16 PM, Miroslav Lachman wrote:
> Janketh Jay wrote on 2018/12/30 01:03:
>> Hi All,
>>
>> On 03/12/2018 2:17 pm, Reko Turja via freebsd-ports wrote:
>>> Hello Janky Jay!
>>>
>>> -----Original Message----- Sent: Monday, December 3, 2018 4:17 PM
>>> Subject: Re: Maia Mailgaurd
>>>
>>>> Okay. No sweat. I'll work on getting a port patch going as that seems
>>>> it would be the easiest way to push this out. I'll provide a
>>>> download as
>>>> soon as I have something available for testing.
>>>
>>> Sounds good, thank you for your work on ports!
>>>
>>> -Reko
>>>
>>
>>        So, after a bit of testing (and not being able to update the
>> port for PHP7 due to Smarty issues), below is a simple patch that
>> will keep the current port version of Maia (1.0.4_3) working with
>> PHP7 (testing with PHP 7.2). Please let me know if this does not work
>> or if there is anything else that needs to be added or changed.
>> Hopefully, I'll get the Smarty issue(s) sorted out soon and have a
>> new port submitted.
>
> [...]
>
>>        Also, if you're interested in seeing/testing the new port
>> (from the latest Github commit for technion's maia_mailguard), you
>> can download the patch from the link below [1] and let me know if
>> it's successful for you. Everything seems to work except for the
>> WebUI due to a fatal call to Smarty which I cannot figure out (I'm
>> not a PHP developer). However, once that is sorted, I should be ready
>> to submit the new port so Maia can be updated to the latest version.
>
> I forgot about this in my previous message - I guess there is an old
> version of Smarty not supporting PHP 7. It would be probably better to
> replace it with newer version of Smarty with PHP 7 support but there
> can be some syntax changes in Smarty config / initialisation. I
> remember something about it about a year ago when I was upgrading one
> of our own (very old) project to newer Smarty templates. There are
> some differences between Smarty 2 and Smarty 3. But as I read now on
> their website, the latest Smarty 2 version 2.6.31 supports PHP 7.2 so
> it should be painless to replace old noncompatible version with this one.

    Thanks again for the heads up on the Smarty3 compat being resolved.
It seems that the latest PHP_FLAVOR'd Smarty3 does indeed fix the issue
when properly linked (I hope I did it correctly). I'll be running this
through Poudrier tonight to make sure everything is good and I'll submit
the port update as soon as it's verified. In the meantime, if anyone
wants to test the patch [1] (or at least look at it to make sure I'm
doing it right regarding the PHP flavoring, linking variables, etc... to
make sure it's done correctly), that would be great! Thanks again, Miroslav!

1. https://www.purplehat.org/downloads/maia/maia-1.0.4-g20181202.diff

Regards,
Janky Jay, III


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/freebsd-ports/attachments/20181229/eaa6724d/attachment.sig>