rc script issues #1
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Hey,
I am so glad that I found your repo! I am fiddling with continuwuity quite a bit building it under FreeBSD, however with 0.5.8 I gave up..
I am running 15.0-p8 on latest with rocksdb-10.10.1 and installed the 0.5.8-generic variant. However starting thrws some rc-script issues:
If I start it directly via
# su -m continuwuity -c '/usr/local/bin/continuwuity -c /usr/local/etc/continuwuity/continuwuity.toml'it starts.My knowlege for rc-scripting is totally not existing (tried to solve it though but gave up..) - do you have a hint where to start (and whats gooing in the wrong direction?
Greetings
Sven
Found it:
daemon.log shows a number of
May 4 21:06:23 host continuwuity[25924]: Error: TOML parse error at line 1, column 15 | 1 | database_path=/var/db/continuwuity | ^^^^^^^^^^^^^^^^^^^^ string values must be quoted, expected literal stringentries. I removed the
-O database_path="'\\\"'"${continuwuity_data_dir}"'\\\"'"and-O log_colors=false -cfrom the rc-script. The data dir is mentioned in the continuwuity.toml and the color bug doesn't bother me (I am the only user on the server). To be on the safe side I escaped the -c /path/to/config.toml, my command_args line looks like that now:command_args="-S -p ${pidfile} -T continuwuity ${procname} -c \"/usr/local/etc/continuwuity/continuwuity.toml\""and it works, continuwuity starts :)
The original error from the
servicecommand is odd, that sounds like it came from the shell, but I can't imagine why that would be returned unless$continuwuity_data_dirwas explicitly empty or had a quote in it or something. I'll have to see if providing a later value incontinuwuity.tomlconflicts with it somehow.After some conversations in
#bsd, I think the right move here is to remove the$continuwuity_data_dirRC variable and leave it up to the user (or a patch) to setdatabase_pathcorrectly. Otherwise we're stepping on the official documentation's toes, and splitting the config file in an arcane way.Setting the permissions and owner correctly (my original concern in this area) should be doable via the packing list, and a message can be shown warning the user about it.