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

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

[debbugs-tracker] bug#29741: closed ([PATCH] gnu: services: web: Add ser


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#29741: closed ([PATCH] gnu: services: web: Add service for apache-httpd.)
Date: Wed, 17 Jan 2018 08:46:02 +0000

Your message dated Wed, 17 Jan 2018 08:45:24 +0000
with message-id <address@hidden>
and subject line Re: [bug#29741] [PATCH 2/2] gnu: services: web: Add service 
for httpd.
has caused the debbugs.gnu.org bug report #29741,
regarding [PATCH] gnu: services: web: Add service for apache-httpd.
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
29741: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=29741
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: [PATCH] gnu: services: web: Add service for apache-httpd. Date: Sat, 16 Dec 2017 20:12:30 +0000 User-agent: mu4e 0.9.18; emacs 25.3.1
I've got as far as writing the service type, and a simple system
test. I've also managed to deploy this with mod_wsgi (which isn't
packaged for Guix yet), which involved extending the service and adding
some virtualhosts.

There isn't any documentation yet, and I'm not very sure of the
way the configuration is organised. In particular:

 - There is a record type for modules, which are handled specifically
   handled in the configuration file. I guess this might be helpful if
   you want to remove modules, but I'm not sure if this is useful.
 - There is some default extra configuration, which is related to the
   mime_module, which makes me wonder if there should be something
   relating these two things.

Christopher Baines (2):
  gnu: tests: web: Generalise the test data for the nginx test.
  gnu: services: web: Add service for apache-httpd.

 gnu/services/web.scm | 227 ++++++++++++++++++++++++++++++++++++++++++++++++++-
 gnu/tests/web.scm    |  90 ++++++++++++++++++--
 2 files changed, 311 insertions(+), 6 deletions(-)



--- End Message ---
--- Begin Message --- Subject: Re: [bug#29741] [PATCH 2/2] gnu: services: web: Add service for httpd. Date: Wed, 17 Jan 2018 08:45:24 +0000 User-agent: mu4e 0.9.18; emacs 25.3.1
Ludovic Courtès <address@hidden> writes:

> Hi Chris!
>
> Sorry for the delay.
>
> Christopher Baines <address@hidden> skribis:
>
>> * gnu/services/web.scm (<httpd-module>,
>>   <httpd-config-file>, <httpd-virtualhost>
>>   <httpd-configuration>): New record types.
>>   (%default-httpd-modules, %httpd-accounts,
>>   httpd-service-type): New variables.
>>   (httpd-shepherd-services, httpd-activation,
>>   httpd-process-extensions): New procedures.
>> * gnu/tests/web.scm (run-httpd-test): New procedure.
>>   (%httpd-os, %tests-httpd): New variables.
>> * doc/guix.texi (Web Services): Document the Apache HTTP Server.
>
> [...]
>
>> +The @code{(gnu services web)} module provides the Apache HTTP Server,
>> +the nginx web server, and also a fastcgi wrapper daemon.
>> +
>> address@hidden Apache HTTP Server
>> +
>> address@hidden {Scheme Variable} httpd-service-type
>> +Service type for the @uref{https://httpd.apache.org/,Apache HTTP}
>> +server.  The value for this service type is a @code{https-configuration}
>         ^
> Maybe add “(@dfn{httpd})”.
>
> Apart from this super-nitpicky comment, this LGTM!  :-)

Great, I've made that change and pushed. Thanks for reviewing Ludo :)

Attachment: signature.asc
Description: PGP signature


--- End Message ---

reply via email to

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