Created attachment 168802 [details]
modified ttrssd file
The script tries to verify that a local database is running. However, I run my database in a separate jail, so I have to delete the ttrssd_prestart check just to get it to think about running. If you really want to verify that the DB is available, then a helper script should probably be created to test this separately using the actual config file instead of assuming a local database.
Have you tried the option DBLOCAL?
It defaults to local, but its purpase should be what you need.
I don't see why that is a compile-time option. The flag shouldn't change any code or dependencies at compile time. If you are going to provide something like that, shouldn't it at least be an rc.conf option?
Also, I installed through the pkg command, so changing this flag using portmaster or the make install process is not intuitive at all.
Would you like me to take a try at getting it work using the rc.conf variable?
Please look at revision 356397:
It has been reported that sometimes the dæmon is launched before the database get available. Your patch is reverting this check.
Yes, my point in comment #2 is that it doesn't make sense that this is a compile time option. This should be runtime option. I agree that my existing file as attached does not help everybody and likely would hurt most users of the default install.
What I am suggesting now is to remove the DBLOCAL use flag and instead modify files/ttrssd.in to take a run.conf configuration option instead.
OK, I see your point. I shall try to replace this option by a flag.
Created attachment 169301 [details]
Replace DBLOCAL option by a flag
Could you please check the proposed patch?
Looks good to me. You may want to update files/pkg-message.in to note the change as well for people upgrading.
Committed, thanks for the report!
A commit references this bug:
Date: Thu Apr 14 19:19:37 UTC 2016
New revision: 413313
Replace the option DBLOCAL by a flag in /etc/rc.conf.
Submitted by: john (at) johnanddalene.net