~abcdw/rde-devel

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
10 2

[PATCH 0/2] feature-laminar

Details
Message ID
<20230914121220.6204-1-ngraves@ngraves.fr>
DKIM signature
missing
Download raw message
These patches add a home feature-laminar, which can help with
continuous integration of projects.

The laminar service in Guix uses a special system account, which doesn't
have access to user projects repositories, which makes it very difficult
to actually use on a project. Instead, this version integrates quite
well with local project development: you can have your laminar jobs
versionned in projects and simply link the job in
$LAMINAR_HOME/cfg/jobs properly.

Nicolas Graves (2):
  rde: Add a laminar home service.
  rde: Add feature-laminar.

 src/contrib/features/ci.scm  | 52 +++++++++++++++++++++
 src/gnu/home-services/ci.scm | 89 ++++++++++++++++++++++++++++++++++++
 2 files changed, 141 insertions(+)
 create mode 100644 src/contrib/features/ci.scm
 create mode 100644 src/gnu/home-services/ci.scm

-- 
2.41.0

[PATCH 1/2] rde: Add a laminar home service.

Details
Message ID
<20230914121220.6204-2-ngraves@ngraves.fr>
In-Reply-To
<20230914121220.6204-1-ngraves@ngraves.fr> (view parent)
DKIM signature
missing
Download raw message
Patch: +89 -0
---
 src/gnu/home-services/ci.scm | 89 ++++++++++++++++++++++++++++++++++++
 1 file changed, 89 insertions(+)
 create mode 100644 src/gnu/home-services/ci.scm

diff --git a/src/gnu/home-services/ci.scm b/src/gnu/home-services/ci.scm
new file mode 100644
index 00000000..9e2ebcd7
--- /dev/null
+++ b/src/gnu/home-services/ci.scm
@@ -0,0 +1,89 @@
;;; rde --- Reproducible development environment.
;;;
;;; Copyright © 2023 Nicolas Graves <ngraves@ngraves.fr>
;;;
;;; This file is part of rde.
;;;
;;; rde is free software; you can redistribute it and/or modify it
;;; under the terms of the GNU General Public License as published by
;;; the Free Software Foundation; either version 3 of the License, or (at
;;; your option) any later version.
;;;
;;; rde is distributed in the hope that it will be useful, but
;;; WITHOUT ANY WARRANTY; without even the implied warranty of
;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
;;; GNU General Public License for more details.
;;;
;;; You should have received a copy of the GNU General Public License
;;; along with rde.  If not, see <http://www.gnu.org/licenses/>.

(define-module (gnu home-services ci)
  #:use-module (guix gexp)
  #:use-module (guix records)
  #:use-module (gnu packages admin)
  #:use-module (gnu packages ci)
  #:use-module (gnu home services)
  #:use-module (gnu services ci)
  #:use-module (gnu home services shepherd)
  #:use-module (ice-9 match)
  #:export (home-laminar-service-type))

;;;; Commentary:
;;;
;;; This module implements a service that to run instances of Laminar, a
;;; continuous integration tool.
;;;
;;; The version in Guix uses a special system account laminar, which doesn't
;;; have access to user projects repositories, which makes it very difficult
;;; to use. Instead, this version integrates quite well with local project
;;; development: you can have your laminar jobs versionned in projects and
;;; simply link the job in $LAMINAR_HOME/cfg/jobs properly.
;;;
;;;; Code:

(define laminar-shepherd-service
  (match-lambda
    (($ (@@ (gnu services ci) <laminar-configuration>)
        laminar home-directory
        bind-http bind-rpc
        title keep-rundirs archive-url
        base-url)
     (list (shepherd-service
            (documentation "Run Laminar.")
            (provision '(laminar))
            ;; (requirement '(networking))  ; considered provided by the system
            (start #~(make-forkexec-constructor
                      (list #$(file-append laminar "/sbin/laminard"))
                      #:environment-variables
                      `(,(string-append "LAMINAR_HOME="
                                        #$home-directory)
                        ,(string-append "LAMINAR_BIND_HTTP="
                                        #$bind-http)
                        ,(string-append "LAMINAR_BIND_RPC="
                                        #$bind-rpc)
                        ,(string-append "LAMINAR_TITLE="
                                        #$title)
                        ,(string-append "LAMINAR_KEEP_RUNDIRS="
                                        #$(number->string
                                           keep-rundirs))
                        ,@(if #$archive-url
                              (list
                               (string-append "LAMINAR_ARCHIVE_URL="
                                              #$archive-url))
                              '())
                        ,@(if #$base-url
                              (list
                               (string-append "LAMINAR_BASE_URL="
                                              #$base-url))
                              '()))))
            (stop #~(make-kill-destructor)))))))

(define home-laminar-service-type
  (service-type
   (name 'home-laminar)
   (extensions
    (list
     (service-extension home-shepherd-service-type laminar-shepherd-service)))
   (default-value (laminar-configuration))
   (description
    "Run the Laminar continuous integration service.")))
-- 
2.41.0

[PATCH 2/2] rde: Add feature-laminar.

Details
Message ID
<20230914121220.6204-3-ngraves@ngraves.fr>
In-Reply-To
<20230914121220.6204-1-ngraves@ngraves.fr> (view parent)
DKIM signature
missing
Download raw message
Patch: +52 -0
---
 src/contrib/features/ci.scm | 52 +++++++++++++++++++++++++++++++++++++
 1 file changed, 52 insertions(+)
 create mode 100644 src/contrib/features/ci.scm

diff --git a/src/contrib/features/ci.scm b/src/contrib/features/ci.scm
new file mode 100644
index 00000000..4a51e5c5
--- /dev/null
+++ b/src/contrib/features/ci.scm
@@ -0,0 +1,52 @@
;;; rde --- Reproducible development environment.
;;;
;;; Copyright © 2023 Nicolas Graves <ngraves@ngraves.fr>
;;;
;;; This file is part of rde.
;;;
;;; rde is free software; you can redistribute it and/or modify it
;;; under the terms of the GNU General Public License as published by
;;; the Free Software Foundation; either version 3 of the License, or (at
;;; your option) any later version.
;;;
;;; rde is distributed in the hope that it will be useful, but
;;; WITHOUT ANY WARRANTY; without even the implied warranty of
;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
;;; GNU General Public License for more details.
;;;
;;; You should have received a copy of the GNU General Public License
;;; along with rde.  If not, see <http://www.gnu.org/licenses/>.

(define-module (contrib features ci)
  #:use-module (rde features)
  #:use-module (gnu packages golang)
  #:use-module (gnu services)
  #:use-module (gnu services base)
  #:use-module (gnu home services)
  #:use-module (gnu home-services ci)
  #:use-module (guix gexp)

  #:export (feature-laminar))

(define* (feature-laminar
          #:key
          (laminar laminar))
  "This feature sets up laminar for local continuous integration. Compared to
the version in Guix, this feature has access to the directories of the current
user, making laminar usable locally. For instance, it can be used to build
guix locally with a post-merge hook. It is in contrib, because most of what
can be done with laminar can be done with simple git hooks or scripts, the
value added being user experience."
  (ensure-pred file-like? laminar)

  (define (get-home-services config)
    (list
     (simple-service
      'laminar-add-laminar-package
      home-profile-service-type
      (list laminar))))

  (feature
   (name 'laminar)
   (values `((laminar . #t)))
   (home-services-getter get-home-services)))
-- 
2.41.0

Re: [PATCH 0/2] feature-laminar

Details
Message ID
<871qf1ne1y.fsf@ngraves.fr>
In-Reply-To
<20230914121220.6204-1-ngraves@ngraves.fr> (view parent)
DKIM signature
missing
Download raw message
On 2023-09-14 14:09, Nicolas Graves wrote:

> These patches add a home feature-laminar, which can help with
> continuous integration of projects.
>
> The laminar service in Guix uses a special system account, which doesn't
> have access to user projects repositories, which makes it very difficult
> to actually use on a project. Instead, this version integrates quite
> well with local project development: you can have your laminar jobs
> versionned in projects and simply link the job in
> $LAMINAR_HOME/cfg/jobs properly.
>
> Nicolas Graves (2):
>   rde: Add a laminar home service.
>   rde: Add feature-laminar.
>
>  src/contrib/features/ci.scm  | 52 +++++++++++++++++++++
>  src/gnu/home-services/ci.scm | 89 ++++++++++++++++++++++++++++++++++++
>  2 files changed, 141 insertions(+)
>  create mode 100644 src/contrib/features/ci.scm
>  create mode 100644 src/gnu/home-services/ci.scm

Seems that I was able to fix my issue with git send-email in
emacs-sway@0.7! Will probably retry the emacs package, but at least I
can properly send cover letters now. 

-- 
Best regards,
Nicolas Graves

[PATCH v2 1/2] rde: Add a laminar home service.

Details
Message ID
<20230914145402.3353-1-ngraves@ngraves.fr>
In-Reply-To
<871qf1ne1y.fsf@ngraves.fr> (view parent)
DKIM signature
missing
Download raw message
Patch: +90 -0
---
 src/gnu/home-services/ci.scm | 90 ++++++++++++++++++++++++++++++++++++
 1 file changed, 90 insertions(+)
 create mode 100644 src/gnu/home-services/ci.scm

diff --git a/src/gnu/home-services/ci.scm b/src/gnu/home-services/ci.scm
new file mode 100644
index 00000000..3d8c86f2
--- /dev/null
+++ b/src/gnu/home-services/ci.scm
@@ -0,0 +1,90 @@
;;; rde --- Reproducible development environment.
;;;
;;; Copyright © 2023 Nicolas Graves <ngraves@ngraves.fr>
;;;
;;; This file is part of rde.
;;;
;;; rde is free software; you can redistribute it and/or modify it
;;; under the terms of the GNU General Public License as published by
;;; the Free Software Foundation; either version 3 of the License, or (at
;;; your option) any later version.
;;;
;;; rde is distributed in the hope that it will be useful, but
;;; WITHOUT ANY WARRANTY; without even the implied warranty of
;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
;;; GNU General Public License for more details.
;;;
;;; You should have received a copy of the GNU General Public License
;;; along with rde.  If not, see <http://www.gnu.org/licenses/>.

(define-module (gnu home-services ci)
  #:use-module (guix gexp)
  #:use-module (guix records)
  #:use-module (gnu packages admin)
  #:use-module (gnu packages ci)
  #:use-module (gnu home services)
  #:use-module (gnu services ci)
  #:use-module (gnu home services shepherd)
  #:use-module (ice-9 match)
  #:export (home-laminar-service-type)
  #:re-export (laminar-configuration))

;;;; Commentary:
;;;
;;; This module implements a service that to run instances of Laminar, a
;;; continuous integration tool.
;;;
;;; The version in Guix uses a special system account laminar, which doesn't
;;; have access to user projects repositories, which makes it very difficult
;;; to use. Instead, this version integrates quite well with local project
;;; development: you can have your laminar jobs versionned in projects and
;;; simply link the job in $LAMINAR_HOME/cfg/jobs properly.
;;;
;;;; Code:

(define laminar-shepherd-service
  (match-lambda
    (($ (@@ (gnu services ci) <laminar-configuration>)
        laminar home-directory
        bind-http bind-rpc
        title keep-rundirs archive-url
        base-url)
     (list (shepherd-service
            (documentation "Run Laminar.")
            (provision '(laminar))
            ;; (requirement '(networking))  ; considered provided by the system
            (start #~(make-forkexec-constructor
                      (list #$(file-append laminar "/sbin/laminard"))
                      #:environment-variables
                      `(,(string-append "LAMINAR_HOME="
                                        #$home-directory)
                        ,(string-append "LAMINAR_BIND_HTTP="
                                        #$bind-http)
                        ,(string-append "LAMINAR_BIND_RPC="
                                        #$bind-rpc)
                        ,(string-append "LAMINAR_TITLE="
                                        #$title)
                        ,(string-append "LAMINAR_KEEP_RUNDIRS="
                                        #$(number->string
                                           keep-rundirs))
                        ,@(if #$archive-url
                              (list
                               (string-append "LAMINAR_ARCHIVE_URL="
                                              #$archive-url))
                              '())
                        ,@(if #$base-url
                              (list
                               (string-append "LAMINAR_BASE_URL="
                                              #$base-url))
                              '()))))
            (stop #~(make-kill-destructor)))))))

(define home-laminar-service-type
  (service-type
   (name 'home-laminar)
   (extensions
    (list
     (service-extension home-shepherd-service-type laminar-shepherd-service)))
   (default-value (laminar-configuration))
   (description
    "Run the Laminar continuous integration service.")))
-- 
2.41.0

[PATCH v2 2/2] rde: Add feature-laminar.

Details
Message ID
<20230914145402.3353-2-ngraves@ngraves.fr>
In-Reply-To
<20230914145402.3353-1-ngraves@ngraves.fr> (view parent)
DKIM signature
missing
Download raw message
Patch: +71 -0
---
 src/contrib/features/ci.scm | 71 +++++++++++++++++++++++++++++++++++++
 1 file changed, 71 insertions(+)
 create mode 100644 src/contrib/features/ci.scm

diff --git a/src/contrib/features/ci.scm b/src/contrib/features/ci.scm
new file mode 100644
index 00000000..e5f4a85a
--- /dev/null
+++ b/src/contrib/features/ci.scm
@@ -0,0 +1,71 @@
;;; rde --- Reproducible development environment.
;;;
;;; Copyright © 2023 Nicolas Graves <ngraves@ngraves.fr>
;;;
;;; This file is part of rde.
;;;
;;; rde is free software; you can redistribute it and/or modify it
;;; under the terms of the GNU General Public License as published by
;;; the Free Software Foundation; either version 3 of the License, or (at
;;; your option) any later version.
;;;
;;; rde is distributed in the hope that it will be useful, but
;;; WITHOUT ANY WARRANTY; without even the implied warranty of
;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
;;; GNU General Public License for more details.
;;;
;;; You should have received a copy of the GNU General Public License
;;; along with rde.  If not, see <http://www.gnu.org/licenses/>.

(define-module (contrib features ci)
  #:use-module (rde features)
  #:use-module (rde features predicates)
  #:use-module (gnu packages ci)
  #:use-module (gnu services)
  #:use-module (gnu services base)
  #:use-module (gnu home services xdg)
  #:use-module (gnu home services)
  #:use-module (gnu home-services ci)
  #:use-module (guix gexp)

  #:export (feature-laminar))

(define* (feature-laminar
          #:key
          (laminar laminar)
          (home-directory #f))
  "This feature sets up laminar for local continuous integration. Compared to
the version in Guix, this feature has access to the directories of the current
user, making laminar usable locally. For instance, it can be used to build
guix locally with a post-merge hook. It is in contrib, because most of what
can be done with laminar can be done with simple git hooks or scripts, the
value added being user experience."
  (ensure-pred file-like? laminar)
  (ensure-pred maybe-string? home-directory)

  (define (get-home-services config)
    (when (not home-directory)
      (require-value 'home-directory config)
      (require-value 'xdg-base-directories-configuration config))
    (list
     (service home-laminar-service-type
              (laminar-configuration
               (home-directory
                (if home-directory
                    home-directory
                    (string-append
                     (get-value 'home-directory config)
                     (string-drop
                      (home-xdg-base-directories-configuration-state-home
                       (get-value 'xdg-base-directories-configuration config))
                      (string-length "$HOME"))
                     "/laminar")))))
     (simple-service
      'laminar-add-laminar-package
      home-profile-service-type
      (list laminar))))

  (feature
   (name 'laminar)
   (values `((laminar . #t)))
   (home-services-getter get-home-services)))
-- 
2.41.0

Re: [PATCH 0/2] feature-laminar

Details
Message ID
<87bke3hdxu.fsf@trop.in>
In-Reply-To
<871qf1ne1y.fsf@ngraves.fr> (view parent)
DKIM signature
missing
Download raw message
On 2023-09-14 14:18, Nicolas Graves wrote:

> On 2023-09-14 14:09, Nicolas Graves wrote:
>
>> These patches add a home feature-laminar, which can help with
>> continuous integration of projects.
>>
>> The laminar service in Guix uses a special system account, which doesn't
>> have access to user projects repositories, which makes it very difficult
>> to actually use on a project. Instead, this version integrates quite
>> well with local project development: you can have your laminar jobs
>> versionned in projects and simply link the job in
>> $LAMINAR_HOME/cfg/jobs properly.
>>
>> Nicolas Graves (2):
>>   rde: Add a laminar home service.
>>   rde: Add feature-laminar.
>>
>>  src/contrib/features/ci.scm  | 52 +++++++++++++++++++++
>>  src/gnu/home-services/ci.scm | 89 ++++++++++++++++++++++++++++++++++++
>>  2 files changed, 141 insertions(+)
>>  create mode 100644 src/contrib/features/ci.scm
>>  create mode 100644 src/gnu/home-services/ci.scm
>
> Seems that I was able to fix my issue with git send-email in
> emacs-sway@0.7! Will probably retry the emacs package, but at least I
> can properly send cover letters now. 

Good, let us know what was the problem and how you fixed it!

IIRC, it was because format-patch generated cover-letter with
placeholder subject line and send-email complained about it?

`git send-email @~2 --cover-letter --force --annotate` should work I
guess, but very interesting to see how you solved it.

-- 
Best regards,
Andrew Tropin

Re: [PATCH 0/2] feature-laminar

Details
Message ID
<87msxn4kts.fsf@ngraves.fr>
In-Reply-To
<87bke3hdxu.fsf@trop.in> (view parent)
DKIM signature
missing
Download raw message
On 2023-09-15 15:29, Andrew Tropin wrote:

> On 2023-09-14 14:18, Nicolas Graves wrote:
>
>> On 2023-09-14 14:09, Nicolas Graves wrote:
>>
>>> These patches add a home feature-laminar, which can help with
>>> continuous integration of projects.
>>>
>>> The laminar service in Guix uses a special system account, which doesn't
>>> have access to user projects repositories, which makes it very difficult
>>> to actually use on a project. Instead, this version integrates quite
>>> well with local project development: you can have your laminar jobs
>>> versionned in projects and simply link the job in
>>> $LAMINAR_HOME/cfg/jobs properly.
>>>
>>> Nicolas Graves (2):
>>>   rde: Add a laminar home service.
>>>   rde: Add feature-laminar.
>>>
>>>  src/contrib/features/ci.scm  | 52 +++++++++++++++++++++
>>>  src/gnu/home-services/ci.scm | 89 ++++++++++++++++++++++++++++++++++++
>>>  2 files changed, 141 insertions(+)
>>>  create mode 100644 src/contrib/features/ci.scm
>>>  create mode 100644 src/gnu/home-services/ci.scm
>>
>> Seems that I was able to fix my issue with git send-email in
>> emacs-sway@0.7! Will probably retry the emacs package, but at least I
>> can properly send cover letters now.
>
> Good, let us know what was the problem and how you fixed it!

You can see more about the issue I had here :
https://github.com/thblt/sway.el/issues/9

Basically, when calling git send-email from the command line, a
call-process in sway.el would cause emacs to get stuck with a
non-existing default-directory, thus breaking the minibuffer, and I
often had to restart-emacs at that point.

Wrapping the call-process function with with-existing-directory solved
the issue and I now properly had the cover-letter draft popping in a new
buffer. 

Here's the associated commit:
https://github.com/thblt/sway.el/commit/425005713af1e9269f1d5b5221fb4ea3046f52e4

>
> IIRC, it was because format-patch generated cover-letter with
> placeholder subject line and send-email complained about it?
>
> `git send-email @~2 --cover-letter --force --annotate` should work I
> guess, but very interesting to see how you solved it.

-- 
Best regards,
Nicolas Graves

Re: [PATCH v2 1/2] rde: Add a laminar home service.

Details
Message ID
<87o77gaxim.fsf@trop.in>
In-Reply-To
<20230914145402.3353-1-ngraves@ngraves.fr> (view parent)
DKIM signature
pass
Download raw message
On 2023-09-14 16:53, Nicolas Graves wrote:

> ---
>  src/gnu/home-services/ci.scm | 90 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 90 insertions(+)
>  create mode 100644 src/gnu/home-services/ci.scm
>
> diff --git a/src/gnu/home-services/ci.scm b/src/gnu/home-services/ci.scm
> new file mode 100644
> index 00000000..3d8c86f2
> --- /dev/null
> +++ b/src/gnu/home-services/ci.scm
> @@ -0,0 +1,90 @@
> +;;; rde --- Reproducible development environment.
> +;;;
> +;;; Copyright © 2023 Nicolas Graves <ngraves@ngraves.fr>
> +;;;
> +;;; This file is part of rde.
> +;;;
> +;;; rde is free software; you can redistribute it and/or modify it
> +;;; under the terms of the GNU General Public License as published by
> +;;; the Free Software Foundation; either version 3 of the License, or (at
> +;;; your option) any later version.
> +;;;
> +;;; rde is distributed in the hope that it will be useful, but
> +;;; WITHOUT ANY WARRANTY; without even the implied warranty of
> +;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +;;; GNU General Public License for more details.
> +;;;
> +;;; You should have received a copy of the GNU General Public License
> +;;; along with rde.  If not, see <http://www.gnu.org/licenses/>.
> +
> +(define-module (gnu home-services ci)
> +  #:use-module (guix gexp)
> +  #:use-module (guix records)
> +  #:use-module (gnu packages admin)
> +  #:use-module (gnu packages ci)
> +  #:use-module (gnu home services)
> +  #:use-module (gnu services ci)
> +  #:use-module (gnu home services shepherd)
> +  #:use-module (ice-9 match)
> +  #:export (home-laminar-service-type)
> +  #:re-export (laminar-configuration))
> +
> +;;;; Commentary:
> +;;;
> +;;; This module implements a service that to run instances of Laminar, a
> +;;; continuous integration tool.
> +;;;
> +;;; The version in Guix uses a special system account laminar, which doesn't
> +;;; have access to user projects repositories, which makes it very difficult
> +;;; to use. Instead, this version integrates quite well with local project
> +;;; development: you can have your laminar jobs versionned in projects and
> +;;; simply link the job in $LAMINAR_HOME/cfg/jobs properly.
> +;;;
> +;;;; Code:
> +
> +(define laminar-shepherd-service
> +  (match-lambda
> +    (($ (@@ (gnu services ci) <laminar-configuration>)
> +        laminar home-directory
> +        bind-http bind-rpc
> +        title keep-rundirs archive-url
> +        base-url)
> +     (list (shepherd-service
> +            (documentation "Run Laminar.")
> +            (provision '(laminar))
> +            ;; (requirement '(networking))  ; considered provided by the system
> +            (start #~(make-forkexec-constructor
> +                      (list #$(file-append laminar "/sbin/laminard"))
> +                      #:environment-variables
> +                      `(,(string-append "LAMINAR_HOME="
> +                                        #$home-directory)
> +                        ,(string-append "LAMINAR_BIND_HTTP="
> +                                        #$bind-http)
> +                        ,(string-append "LAMINAR_BIND_RPC="
> +                                        #$bind-rpc)
> +                        ,(string-append "LAMINAR_TITLE="
> +                                        #$title)
> +                        ,(string-append "LAMINAR_KEEP_RUNDIRS="
> +                                        #$(number->string
> +                                           keep-rundirs))
> +                        ,@(if #$archive-url
> +                              (list
> +                               (string-append "LAMINAR_ARCHIVE_URL="
> +                                              #$archive-url))
> +                              '())
> +                        ,@(if #$base-url
> +                              (list
> +                               (string-append "LAMINAR_BASE_URL="
> +                                              #$base-url))
> +                              '()))))
> +            (stop #~(make-kill-destructor)))))))
> +
> +(define home-laminar-service-type
> +  (service-type
> +   (name 'home-laminar)
> +   (extensions
> +    (list
> +     (service-extension home-shepherd-service-type laminar-shepherd-service)))
> +   (default-value (laminar-configuration))
> +   (description
> +    "Run the Laminar continuous integration service.")))

I'm a bit slow on response for this one :)

A couple notes on this one: (gnu home-services) is deperecated, re-using
system's configuration record can be confusing.  Other than that looks
good for me.

Marking as NEED_REVISION.

-- 
Best regards,
Andrew Tropin

Re: [PATCH v2 1/2] rde: Add a laminar home service.

Details
Message ID
<87a5isnvzq.fsf@ngraves.fr>
In-Reply-To
<87o77gaxim.fsf@trop.in> (view parent)
DKIM signature
missing
Download raw message
On 2024-07-02 14:39, Andrew Tropin wrote:
> I'm a bit slow on response for this one :)
>
> A couple notes on this one: (gnu home-services) is deperecated, re-using
> system's configuration record can be confusing.  Other than that looks
> good for me.
>
> Marking as NEED_REVISION.

I don't use it anymore (went for a simpler custom Makefile for the
project I used to use laminar with), but I've noticed you planned to use
laminar later, I'll update this soon ;)

-- 
Best regards,
Nicolas Graves

Re: [PATCH v2 1/2] rde: Add a laminar home service.

Details
Message ID
<87plrmn5pw.fsf@trop.in>
In-Reply-To
<87a5isnvzq.fsf@ngraves.fr> (view parent)
DKIM signature
pass
Download raw message
On 2024-07-08 10:11, Nicolas Graves wrote:

> On 2024-07-02 14:39, Andrew Tropin wrote:
>> I'm a bit slow on response for this one :)
>>
>> A couple notes on this one: (gnu home-services) is deperecated, re-using
>> system's configuration record can be confusing.  Other than that looks
>> good for me.
>>
>> Marking as NEED_REVISION.
>
> I don't use it anymore (went for a simpler custom Makefile for the
> project I used to use laminar with), but I've noticed you planned to use
> laminar later, I'll update this soon ;)

I'm considering to use it, but will see how it goes, probably will take
some time and there is a chance we'll go for another option :)

BTW, talking about CIs, we can host our substitutes on
cuirass.genenetwork.org

-- 
Best regards,
Andrew Tropin
Reply to thread Export thread (mbox)