[pacman-dev] [PATCH] meson: add trailing slashes to directory components in configuration defines

Eli Schwartz eschwartz at archlinux.org
Tue Dec 11 04:47:58 UTC 2018


This matches what we currently do in the autotools build configuration,
and ensures that the default pacman-conf definitions for unspecified
values consistently end with the trailing directory slashes.

This has ramifications for thirdparty tools that up to now, have relied
on this slash being there. Those tools should be fixed to prevent
breaking when custom locations are set, but this is no reason not to fix
it on our end as well. An extra trailng slash should never cause harm.

Signed-off-by: Eli Schwartz <eschwartz at archlinux.org>
---
 meson.build | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/meson.build b/meson.build
index 138a1934..0a710653 100644
--- a/meson.build
+++ b/meson.build
@@ -72,10 +72,10 @@ conf.set_quoted('LDCONFIG', LDCONFIG)
 conf.set_quoted('LIB_VERSION', meson.project_version())
 conf.set_quoted('SYSHOOKDIR', join_paths(DATAROOTDIR, 'libalpm/hooks/'))
 conf.set_quoted('CONFFILE', join_paths(SYSCONFDIR, 'pacman.conf'))
-conf.set_quoted('DBPATH', join_paths(LOCALSTATEDIR, 'lib/pacman'))
-conf.set_quoted('GPGDIR', join_paths(SYSCONFDIR, 'pacman.d/gnupg'))
+conf.set_quoted('DBPATH', join_paths(LOCALSTATEDIR, 'lib/pacman/'))
+conf.set_quoted('GPGDIR', join_paths(SYSCONFDIR, 'pacman.d/gnupg/'))
 conf.set_quoted('LOGFILE', join_paths(LOCALSTATEDIR, 'log/pacman.log'))
-conf.set_quoted('CACHEDIR', join_paths(LOCALSTATEDIR, 'cache/pacman/pkg'))
+conf.set_quoted('CACHEDIR', join_paths(LOCALSTATEDIR, 'cache/pacman/pkg/'))
 conf.set_quoted('HOOKDIR', join_paths(SYSCONFDIR, 'pacman.d/hooks/'))
 conf.set_quoted('ROOTDIR', ROOTDIR)
 
-- 
2.20.0


More information about the pacman-dev mailing list