Clément Lassieur writes: > Hi Christopher, > > Christopher Baines writes: > >> For the default config file representation. This makes it possible to more >> easily change the configuration file, and have dynamic content. In particular, >> I'm looking at adding a pid file location to the config file. >> >> * gnu/services/databases.scm (): New record type. >> (%default-postgres-config): Remove this, it's been replaced by the >> configuration file. >> (): Alter the default for the config file field. >> (postgresql-service): Alter the default value for the config-file parameter. >> --- >> gnu/services/databases.scm | 86 +++++++++++++++++++++++++++++++++++----------- >> 1 file changed, 66 insertions(+), 20 deletions(-) > > Thank you for this work! No problem, I've finally got around to going through some patches I've had sitting around for a while. >> +(define-gexp-compiler (postgresql-config-file-compiler >> + (file ) system target) >> + (match file >> + (($ log-destination hba-file >> + ident-file extra-config) >> + (define (quote string) >> + (if string >> + (list "'" string "'") >> + (list))) > > I don't think it's a good thing to hide one of the most important lisp > functions :-). I don't quite follow. I was trying to use '() rather than (list) if that is what you mean, but I kept getting odd errors from Guile, so I gave up, and ended up going with this.