Skip to content
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

Update postgresql #15863

Merged
merged 5 commits into from
Jun 7, 2016
Merged

Update postgresql #15863

merged 5 commits into from
Jun 7, 2016

Conversation

matthiasbeyer
Copy link
Contributor

Motivation for this change

As discussed in #15856 (comment)

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @edolstra, @vcunat and @paraseba to be potential reviewers

@groxxda
Copy link
Contributor

groxxda commented May 31, 2016

CC: @ocharles

@ocharles
Copy link
Contributor

I would be happy with all these changes.

@alanorth
Copy link

PostgreSQL 9.3.13 has been out for a few weeks. I came to submit a pull request but saw this one. This is what I have built locally (on Mac OS X):

diff --git a/pkgs/servers/sql/postgresql/default.nix b/pkgs/servers/sql/postgresql/default.nix
index bd7b8fd..df44d78 100644
--- a/pkgs/servers/sql/postgresql/default.nix
+++ b/pkgs/servers/sql/postgresql/default.nix
@@ -86,9 +86,9 @@ in {
   };

   postgresql93 = common {
-    version = "9.3.11";
+    version = "9.3.13";
     psqlSchema = "9.3";
-    sha256 = "08ba951nfiy516flaw352shj1zslxg4ryx3w5k0adls1r682l8ix";
+    sha256 = "0ax7cb1lknzz4105spi8k71n71rb4jp00d2kscasifgxkg9f2i2m";
   };

   postgresql94 = common {

I don't use any other PostgreSQL versions so I haven't tested anything other than 9.3.x.

@paraseba
Copy link
Contributor

paraseba commented Jun 1, 2016

This looks good to me.

I want to use the opportunity to ask a question, anybody here has opinions on the best way to fix #15512

Postgis extension is broken after the multiple outputs patch. Regrouping everything under a single output would fix it, but it's probably not something we want to do? Should we create a new package that contains both postgres and any extensions (the way the NixOs module does)?

@joachifm joachifm merged commit bdc494a into NixOS:master Jun 7, 2016
@matthiasbeyer matthiasbeyer deleted the update-postgresql branch June 7, 2016 13:23
@matthiasbeyer matthiasbeyer restored the update-postgresql branch June 10, 2016 13:32
@matthiasbeyer matthiasbeyer deleted the update-postgresql branch February 8, 2017 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants