As for (2), I think we should fix it, especially as:

• the fix is trivial, just don't rename the file, and
• the web-based setup seems more documented online and apparently is what the developers of tt-rss support, and
• that web-based setup correctly flagged the missing iconv module, so it's a good thing :) It was also easier to setup and init the database. I rarely need to use the psql or mysql CLIs so not having to deal with them is a plus :)

Is there a reason the file is being renamed in the first place? That seems a bit gratuitous. If the changed name is really required, I would suggest to copy/link instead of renaming it, so both dist or sample files are present (and updated).

The file was moved by 197045. It was suitable as a use case for the @sample plist documented here.

If I had to guess why the config.php was crated it prevents someone from entering the web based installer on first install that you otherwise didn't want to make changes and to enforce the file permissions of 0400. This is only a guess having a config.php has existed since the ports creation. I'd rather keep it this way.

However, supporting the web installer might still be possible by an addition to the pkg-message mentioning:

• if you want to use the web installer:
  • rm config.php
  • ln -s config.php.sample config.php-dist (maybe do this by default in the port as you mentioned).
  • Goto http://.../install/

However, depending on the config of the web server this might leave config.php readable by group/others. Maybe add a patch for touch/chmod the file before writing it?

I'm not sure the rename is worth it, just to make the @sample usage simpler. There's even support for -dist, so let's use that.

Don't worry too much about permissions either. The web UI helpfully prints out the full config (except it wasn't able to print anything in my case) telling folks to copy-paste that into their config. There's a button that one can use to _try_ and have the webserver write it out, but of course that _should_ fail most of the time for strictly configured webservers.

Oh wait, if we use @sample, then config.php-dist will be copied to config.php (but it has bogus values) so the whole web config will fall on its nose. So I think the best way forward is to not use @sample at all and instead ship config.php-dist as a regular file (that get's updated with a new version).

Then the users can copy that to config.php or let the web UI spit out a config.php that they can write down using sudo or whatever.

Tested this with uninstall/reinstalls confirming the permissions were what we expect.
Ran thru the web installer without errors.
Switched options to confirm the proper depends were being added.
Reviewed ttrssd was be commented properly with the new OPT_VARS= method.

Reworded ttrssd.

In D26061#579331, @uqs wrote:

Looks good, do you want me to go ahead and commit this on your behalf?

Yes, thanks!