-
Notifications
You must be signed in to change notification settings - Fork 302
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
PostgreSQL: bump, adjust, etc #2120
Conversation
Tests won't run as it cannot find the required so files. Somebody have an LD command for it, but i can't seems to find it right now. |
|
||
TEST() | ||
{ | ||
make check |
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.
I haven't checked yet, but if the shared libs are built in e.g. dev-db/postgresql/work-9.3.6/sources/postgresql-10.1/src/lib/
then this should work:
- make check
+ LIBRARY_PATH="$sourceDir/src/lib${LIBRARY_PATH:+:$LIBRARY_PATH}" make check
video. | ||
* It has native programming interfaces for C/C++, Java, .Net, Perl, Python, \ | ||
Ruby, Tcl, ODBC, among others, and exceptional documentation." | ||
HOMEPAGE="http://www.postgresql.org/" |
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.
http -> https
libpgfeutils | ||
fixPkgconfig | ||
|
||
mkdir -p $developLibDir/postgresql/pgxs |
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.
- mkdir -p $developLibDir/postgresql/pgxs
mv $libDir/postgresql/pgxs $developLibDir/postgresql/
(You don't need to mkdir pgxs in this case since the mv in the next command will create that directory.)
@extrowerk any update here? |
Also it might be nice to keep postgres 9 in addition to 10, as they're incompatible. |
anything here worth keeping/merging with the other pending pull request? |
Actually, I didn't notice the USE_DSM_MMAP; we should probably use that instead of the stubbed sys/shm. I'll take a look at that. Otherwise that one probably supersedes this, despite the fact this uses a newer postgres. |
Enabled icu and xml, xslt cause i thought it could be nice.
My patch (the latest) maybe have to be fixed before merge. Any constructive idea will be greatly appreciated.