[Buildroot] [PATCH] package/postgresql: fix filesystem naming consistency
Pascal de Bruijn
p.debruijn at unilogic.nl
Thu Nov 28 11:15:42 UTC 2019
Currently the service is called postgresql, but other filesystem
references are called pgsql, which is inconsistent and confusing.
Given that at least Debian uses postgresql in the filesystems
as well I would suggest moving the filesystem reference to align
with the service name as opposed to the other way around.
Signed-off-by: Pascal de Bruijn <p.debruijn at unilogic.nl>
---
package/postgresql/Config.in | 2 +-
package/postgresql/S50postgresql | 8 ++++----
package/postgresql/postgresql.mk | 4 ++--
package/postgresql/postgresql.service | 6 +++---
4 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/package/postgresql/Config.in b/package/postgresql/Config.in
index e548d3c..2f677da 100644
--- a/package/postgresql/Config.in
+++ b/package/postgresql/Config.in
@@ -11,7 +11,7 @@ config BR2_PACKAGE_POSTGRESQL
database system.
Enable the readline package to gain readline support in
- pgsql (the command line interpreter), which offers
+ psql (the command line interpreter), which offers
command history and edit functions.
Enable the zlib package to gain builtin compression for
diff --git a/package/postgresql/S50postgresql b/package/postgresql/S50postgresql
index 1ece4fc..8d7ad34 100644
--- a/package/postgresql/S50postgresql
+++ b/package/postgresql/S50postgresql
@@ -5,20 +5,20 @@
umask 077
-if [ ! -f /var/lib/pgsql/PG_VERSION ]; then
+if [ ! -f /var/lib/postgresql/PG_VERSION ]; then
echo "Initializing postgresql data base..."
- su - postgres -c '/usr/bin/pg_ctl initdb -D /var/lib/pgsql'
+ su - postgres -c '/usr/bin/pg_ctl initdb -D /var/lib/postgresql'
echo "done"
fi
start() {
printf "Starting postgresql: "
- su - postgres -c '/usr/bin/pg_ctl start -w -D /var/lib/pgsql -l logfile'
+ su - postgres -c '/usr/bin/pg_ctl start -w -D /var/lib/postgresql -l logfile'
echo "OK"
}
stop() {
printf "Stopping postgresql: "
- su - postgres -c '/usr/bin/pg_ctl stop -D /var/lib/pgsql -m fast'
+ su - postgres -c '/usr/bin/pg_ctl stop -D /var/lib/postgresql -m fast'
echo "OK"
}
restart() {
diff --git a/package/postgresql/postgresql.mk b/package/postgresql/postgresql.mk
index 858cd69..5a4ffe6 100644
--- a/package/postgresql/postgresql.mk
+++ b/package/postgresql/postgresql.mk
@@ -102,11 +102,11 @@ endif
POSTGRESQL_CONF_ENV += CFLAGS="$(POSTGRESQL_CFLAGS)"
define POSTGRESQL_USERS
- postgres -1 postgres -1 * /var/lib/pgsql /bin/sh - PostgreSQL Server
+ postgres -1 postgres -1 * /var/lib/postgresql /bin/sh - PostgreSQL Server
endef
define POSTGRESQL_INSTALL_TARGET_FIXUP
- $(INSTALL) -dm 0700 $(TARGET_DIR)/var/lib/pgsql
+ $(INSTALL) -dm 0700 $(TARGET_DIR)/var/lib/postgresql
$(RM) -rf $(TARGET_DIR)/usr/lib/postgresql/pgxs
endef
diff --git a/package/postgresql/postgresql.service b/package/postgresql/postgresql.service
index 53e6f84..7698558 100644
--- a/package/postgresql/postgresql.service
+++ b/package/postgresql/postgresql.service
@@ -15,10 +15,10 @@ Group=postgres
StandardOutput=syslog
StandardError=syslog
SyslogIdentifier=postgres
-PIDFile=/var/lib/pgsql/postmaster.pid
+PIDFile=/var/lib/postgresql/postmaster.pid
-ExecStartPre=/bin/sh -c "if [ ! -f /var/lib/pgsql/PG_VERSION ]; then /usr/bin/pg_ctl initdb -D /var/lib/pgsql; fi"
-ExecStart=/usr/bin/postgres -D /var/lib/pgsql
+ExecStartPre=/bin/sh -c "if [ ! -f /var/lib/postgresql/PG_VERSION ]; then /usr/bin/pg_ctl initdb -D /var/lib/postgresql; fi"
+ExecStart=/usr/bin/postgres -D /var/lib/postgresql
ExecReload=/usr/bin/kill -HUP $MAINPID
KillMode=mixed
KillSignal=SIGINT
--
2.7.4
More information about the buildroot
mailing list