Removed unused PostgreSQL packages

  • Open
  • quality assurance status badge
Details
5 participants
  • Björn Höfling
  • Leo Famulari
  • Christopher Baines
  • Maxim Cournoyer
  • Mathieu Othacehe
Owner
unassigned
Submitted by
Leo Famulari
Severity
normal
L
L
Leo Famulari wrote on 17 May 2021 21:22
(address . guix-patches@gnu.org)
YKLCh/AdKWL7Jqzi@jasmine.lan
Currently we have packages for PostgreSQL 9.6, 10, 11, and 13.

Versions 9.6 and 11 are not used in Guix. So I propose that we remove
them.
L
L
Leo Famulari wrote on 17 May 2021 21:23
[PATCH 2/2] gnu: Remove PostgreSQL 11.
(address . 48484@debbugs.gnu.org)
72dfebe0d86c5bdd0890a5c88d2081dcbb170cb3.1621279407.git.leo@famulari.name
* gnu/packages/databases.scm (postgresql-11): Remove variable.
(postgresql-10): Inherit from postgresql-13.
---
gnu/packages/databases.scm | 17 ++---------------
1 file changed, 2 insertions(+), 15 deletions(-)

Toggle diff (34 lines)
diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index 5b6504b5c1..96033bc020 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -1098,25 +1098,12 @@ pictures, sounds, or video.")
"18dliq7h2l8irffhyyhdmfwx3si515q6gds3cxdjb9n7m17lbn9w"))
(patches (search-patches "postgresql-disable-resolve_symlinks.patch"))))))
-(define-public postgresql-11
- (package
- (inherit postgresql-13)
- (name "postgresql")
- (version "11.12")
- (source (origin
- (inherit (package-source postgresql-13))
- (uri (string-append "https://ftp.postgresql.org/pub/source/v"
- version "/postgresql-" version ".tar.bz2"))
- (sha256
- (base32
- "016bacpmqxc676ipzc1l8zv1jj44mjz7dv7jhqazg3ibdfqxiyc7"))))))
-
(define-public postgresql-10
(package
- (inherit postgresql-11)
+ (inherit postgresql-13)
(version "10.17")
(source (origin
- (inherit (package-source postgresql-11))
+ (inherit (package-source postgresql-13))
(uri (string-append "https://ftp.postgresql.org/pub/source/v"
version "/postgresql-" version ".tar.bz2"))
(sha256
--
2.31.1
L
L
Leo Famulari wrote on 17 May 2021 21:23
[PATCH 1/2] gnu: Remove PostgreSQL 9.
(address . 48484@debbugs.gnu.org)
a8f6dd76613c9ff6edb2291df2fa9bde2ba323cc.1621279407.git.leo@famulari.name
* gnu/packages/databases.scm (postgresql-9.6): Remove variable.
---
gnu/packages/databases.scm | 12 ------------
1 file changed, 12 deletions(-)

Toggle diff (25 lines)
diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index 1a6d6f13d5..5b6504b5c1 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -1123,18 +1123,6 @@ pictures, sounds, or video.")
(base32
"0v5jahkqm6gkq67s4bac3h7297bscn2ab6y128idi73cc1qq1wjs"))))))
-(define-public postgresql-9.6
- (package
- (inherit postgresql-10)
- (version "9.6.22")
- (source (origin
- (inherit (package-source postgresql-10))
- (uri (string-append "https://ftp.postgresql.org/pub/source/v"
- version "/postgresql-" version ".tar.bz2"))
- (sha256
- (base32
- "0c19kzrj5ib5ygmavf5d6qvxdwrxzzz6jz1r2dl5b815208cscix"))))))
-
(define-public postgresql postgresql-13)
(define-public pgloader
--
2.31.1
B
B
Björn Höfling wrote on 17 May 2021 21:53
Re: [bug#48484] Removed unused PostgreSQL packages
(name . Leo Famulari)(address . leo@famulari.name)(address . 48484@debbugs.gnu.org)
20210517215325.48f2f598@alma-ubu.fritz.box
On Mon, 17 May 2021 15:22:47 -0400
Leo Famulari <leo@famulari.name> wrote:

Toggle quote (5 lines)
> Currently we have packages for PostgreSQL 9.6, 10, 11, and 13.
>
> Versions 9.6 and 11 are not used in Guix. So I propose that we remove
> them.

These versions are still supported [0], 9.6 until 2021-11-11. Although
they have no dependencies, people could still use them. I would keep
them until EOL.

Björn


-----BEGIN PGP SIGNATURE-----

iF0EAREKAB0WIQQiGUP0np8nb5SZM4K/KGy2WT5f/QUCYKLJtQAKCRC/KGy2WT5f
/etIAKCA9NOM1dc3oRYtlPyR9nh6CAT0YwCgmgcD1FTSmEAyr5oB8O/jhmLkosM=
=iEgl
-----END PGP SIGNATURE-----


C
C
Christopher Baines wrote on 18 May 2021 00:36
(name . Leo Famulari)(address . leo@famulari.name)(address . 48484@debbugs.gnu.org)
87tun1asyb.fsf@cbaines.net
Leo Famulari <leo@famulari.name> writes:

Toggle quote (5 lines)
> Currently we have packages for PostgreSQL 9.6, 10, 11, and 13.
>
> Versions 9.6 and 11 are not used in Guix. So I propose that we remove
> them.

In terms of reasons to keep them, the PostgreSQL service configuration
for Guix pushes users to specify a specific postgresql package, so
removing particular versions could break users configuration.

While it is possible to upgrade, I'm not aware of any guidance about the
particulars with doing this for Guix, so I'd be hesitant to remove old
supported versions, until there's something to point at which users can
follow to upgrade.
M
M
Mathieu Othacehe wrote on 4 Jun 2021 15:05
Re: bug#48484: Removed unused PostgreSQL packages
(name . Christopher Baines)(address . mail@cbaines.net)
87zgw5dbl5.fsf_-_@gnu.org
Hello,

Toggle quote (5 lines)
> While it is possible to upgrade, I'm not aware of any guidance about the
> particulars with doing this for Guix, so I'd be hesitant to remove old
> supported versions, until there's something to point at which users can
> follow to upgrade.

I agree with Chris and Björn and think that we can close this one for
now if it's OK for you Leo.

Thanks,

Mathieu
M
M
Maxim Cournoyer wrote on 22 May 2022 05:37
(name . Leo Famulari)(address . leo@famulari.name)
875yly8cgw.fsf_-_@gmail.com
Hello Leo,

Mathieu Othacehe <othacehe@gnu.org> writes:

Toggle quote (10 lines)
> Hello,
>
>> While it is possible to upgrade, I'm not aware of any guidance about the
>> particulars with doing this for Guix, so I'd be hesitant to remove old
>> supported versions, until there's something to point at which users can
>> follow to upgrade.
>
> I agree with Chris and Björn and think that we can close this one for
> now if it's OK for you Leo.

It's been 50 weeks; perhaps the old postgresql are now EOL? In which
case I'd say go ahead with the pruning.

Thanks,

Maxim
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 48484@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 48484
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch