GNU bug report logs - #52162
[PATCH] gnu: subversion: Add --enable-plaintext-password-storage flag

Previous Next

To reply to this bug, email your comments to 52162 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Philip Kaludercic <philipk <at> posteo.net>
To: guix-patches <at> gnu.org
Cc: Philip Kaludercic <philipk <at> posteo.net>
Subject: [PATCH] gnu: subversion: Add --enable-plaintext-password-storage flag
Date: Sun, 28 Nov 2021 15:41:10 +0000

* version-control.scm (subversion): Enable plain text passwords while
compiling SVN, undoing the controversial move to disable them by the
maintainers. See also
https://lists.apache.org/thread/b6g2hx2m3s117wcmno08opl874ons3q8.
---
 gnu/packages/version-control.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index 10487c6939..ff3dc61456 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -1969,7 +1969,8 @@ (define-public subversion
     (arguments
      '(#:parallel-tests? #f             ; TODO Seems to cause test failures on
                                         ; i686-linux
-       #:configure-flags '("--enable-static=no")
+       #:configure-flags '("--enable-static=no"
+                           "--enable-plaintext-password-storage")
        #:phases
        (modify-phases %standard-phases
          (add-after 'configure 'patch-libtool-wrapper-ls
-- 
2.34.0

Message #8 received at 52162 <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Philip Kaludercic <philipk <at> posteo.net>
Cc: 52162 <at> debbugs.gnu.org
Subject: Re: [bug#52162] [PATCH] gnu: subversion: Add
 --enable-plaintext-password-storage flag
Date: Sun, 28 Nov 2021 17:08:17 -0500

On Sun, Nov 28, 2021 at 03:41:10PM +0000, Philip Kaludercic wrote:
> * version-control.scm (subversion): Enable plain text passwords while
> compiling SVN, undoing the controversial move to disable them by the
> maintainers. See also
> https://lists.apache.org/thread/b6g2hx2m3s117wcmno08opl874ons3q8.

Thanks for the patch!

> -       #:configure-flags '("--enable-static=no")
> +       #:configure-flags '("--enable-static=no"
> +                           "--enable-plaintext-password-storage")

In general, in Guix, we aim to distribute software as it was released by
upstream. We do make changes to fix bugs, but it's outside our normal
activity for us to change defaults from upstream.

This is partly a pragmatic choice (less work to maintain) and partly a
way in which we choose to be "good citizens" with respect to our role as
distributors. But also, because Guix makes it easier for users to
customize their packages compared to other systems like APT and RPM.

I would like to hear other peoples' opinions before we decide about this
patch.

Message #11 received at 52162 <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: 52162 <at> debbugs.gnu.org
Subject: [PATCH] gnu: subversion: Add --enable-plaintext-password-storage flag
Date: Mon, 29 Nov 2021 09:15:48 +0100

I’ve read the discussion that ends with an email[1] by Nathan 
Hartman expressing support for enabling the feature but disabling 
it by default in configuration.  Hence I think it would be 
reasonable to wait for a new release addressing it.  In the 
meantime users can define their own package variant with this 
configure flag added.

It doesn’t need to be changed in Guix itself, because this only 
affects users of the package, not other packages that have 
subversion as an input.  So it’s enough to add a new leaf package. 
As a compromise I think it would be acceptable to add a separate 
subversion-plaintext (or whatever) package variant, which will be 
superseded by the official subversion package as soon as a new 
release is made.

[1] 
https://mail-archives.apache.org/mod_mbox/subversion-dev/202110.mbox/%3cCAJT2EHr7wBfMdg0df7hEnv9+JedcjRkFCzP+q21=dvcWrPv84Q <at> mail.gmail.com%3e

-- 
Ricardo

Message #14 received at 52162 <at> debbugs.gnu.org (full text, mbox):

From: Philip Kaludercic <philipk <at> posteo.net>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: 50297-done <at> debbugs.gnu.org
Subject: Re: bug#50297: 28.0.50; Aggregate project functions for project.el
Date: Thu, 23 Sep 2021 12:08:30 +0000

Dmitry Gutov <dgutov <at> yandex.ru> writes:

> On 23.09.2021 13:46, Philip Kaludercic wrote:
>> Sorry about that, forgot to byte-compile before preparing the
>> patch. This should fix the issues:
>
> Thanks. You can go ahead and install.
>
> Just add NEWS entries for the commands.

Done and pushed.

-- 
	Philip Kaludercic

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.