tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: update databases/sqlitebrowser

> On Jan 30, 2019, at 4:13 PM, Leonardo Taccari <> wrote:
> Brook Milligan writes:
>> [...]
>> As suggested in my previous message, I tested a solution on my Mac with conditional PLIST lines (see attached patch).  The installation succeeded.
>> Is this the right solution to commit?
>> [...]
> Before using OPSYS it is needed to include mk/, otherwise
> it looks good!
> (This is only cosmetic but I would rename the PLIST_VAR(S) to just
> `unix' (I think this is consistent with other packages, e.g.
> qt5-qtbase) and add a comment before defining it to point to check
> CMakeLists.txt for that (i.e. so for future updates possible
> non-Apple users can double check if the PLIST needs to be updated
> or not))

Thanks for the comment.  I have attached an updated version of the patch and will commit it after a bit unless I hear comments to the contrary.

I appreciate you looking at this so quickly.


Attachment: sqlitebrowser.patch
Description: Binary data

Home | Main Index | Thread Index | Old Index