-
Notifications
You must be signed in to change notification settings - Fork 0
Add extension_distdir GUC #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -539,6 +539,7 @@ char *ConfigFileName; | |
char *HbaFileName; | ||
char *IdentFileName; | ||
char *external_pid_file; | ||
char *extension_destdir; | ||
|
||
char *application_name; | ||
|
||
|
@@ -4554,6 +4555,17 @@ struct config_string ConfigureNamesString[] = | |
check_canonical_path, NULL, NULL | ||
}, | ||
|
||
{ | ||
{"extension_destdir", PGC_SUSET, FILE_LOCATIONS, | ||
gettext_noop("Path to prepend for extension loading."), | ||
gettext_noop("This directory is prepended to paths when loading extensions (control and SQL files), and to the '$libdir' directive when loading modules that back functions. The location is made configurable to allow build-time testing of extensions that do not have been installed to their proper location yet."), | ||
GUC_SUPERUSER_ONLY | ||
}, | ||
&extension_destdir, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could this be made That would be a real help for container environments, local testing, etc, and would be more consistent with how the library search path works. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I replied to the -hackers thread, but I'd argue that's a completely different feature that requires a fair bit of thought. I would definitely prefer it, but the approach proposed here feels like a decent interim step to me, one that's already proven (in the Debian packaging system) and does most of what we need to solve the immediate challenges, at the expense of a slightly funky directory layout. |
||
"", | ||
NULL, NULL, NULL | ||
}, | ||
|
||
{ | ||
{"ssl_library", PGC_INTERNAL, PRESET_OPTIONS, | ||
gettext_noop("Shows the name of the SSL library."), | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dynamic_library_path
isCLIENT_CONN_OTHER
notFILE_LOCATIONS
. Should this be the same for consistency, and is there a security reason for the dynamic library path choice?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's superuser-only, so I'm not sure it matters. Happy to change it to whatever the consensus determines.