emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#65152: closed ([PATCH] services: Modify cgit-configuration fields)


From: GNU bug Tracking System
Subject: bug#65152: closed ([PATCH] services: Modify cgit-configuration fields)
Date: Sat, 09 Sep 2023 10:33:02 +0000

Your message dated Sat, 09 Sep 2023 12:32:13 +0200
with message-id <87sf7nzlg2.fsf_-_@gnu.org>
and subject line Re: bug#65152: [PATCH] services: Modify cgit-configuration 
fields
has caused the debbugs.gnu.org bug report #65152,
regarding [PATCH] services: Modify cgit-configuration fields
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
65152: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=65152
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH] services: Modify cgit-configuration fields Date: Tue, 8 Aug 2023 18:37:10 +0200
* gnu/services/cgit.scm (cgit-configuration)[root-readme]: Accept
'file-object' instead of only 'string'
---
 gnu/services/cgit.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/services/cgit.scm b/gnu/services/cgit.scm
index e33cb9e7db..684ab73f76 100644
--- a/gnu/services/cgit.scm
+++ b/gnu/services/cgit.scm
@@ -581,7 +581,7 @@ (define-configuration cgit-configuration
    (string "a fast webinterface for the git dscm")
    "Text printed below the heading on the repository index page.")
   (root-readme
-   (string "")
+   (file-object "")
    "The content of the file specified with this option will be included
 verbatim below the \"about\" link on the repository index page.")
   (root-title
-- 
2.41.0




--- End Message ---
--- Begin Message --- Subject: Re: bug#65152: [PATCH] services: Modify cgit-configuration fields Date: Sat, 09 Sep 2023 12:32:13 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)
Hi,

Thomas Albers Raviola <thomas@thomaslabs.org> skribis:

> in this context `file-object' is defined to be either a `file-like'
> object or a `string'. As such this wouldn't break backward compatibility
> but include a superset of possible values.
>
> Also the original string is not used as the contents of the readme file
> but as a file path.

Oh right, my bad, I had misunderstood that.  So it’s all good, applied!

Eventually we should probably replace ‘file-object’ by ‘file-like’,
unless there’s a clear use case for passing out-of-band file names (a
use case would be passwords, but probably not ‘README’ files).

Thanks!

Ludo’.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]