[Top][All Lists]

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

[bug#28560] [PATCH] web: Add try-files for the nginx-service-type.

From: Julien Lepiller
Subject: [bug#28560] [PATCH] web: Add try-files for the nginx-service-type.
Date: Tue, 26 Sep 2017 09:53:52 +0200
User-agent: K-9 Mail for Android

Le 24 septembre 2017 20:09:26 GMT+02:00, Oleg Pykhalov <address@hidden> a écrit :
Hello Julien,

Julien Lepiller <address@hidden> writes:

Thank you for the patch! I think it will be perfect after you update doc/guix.texi too.

Thank you for review.

I rephrase little bit

Here is a new patch.


I don't think your rephrasing works, I have some troubles making sense of it, even though I know what it is supposed to do. Maybe:

A list of files whose existence is checked in the specified order. @code{nginx} will use the first file it finds to process the request.

I'm not sure about the last part of your sentence, do we explain what the "current context" is? Is it even relevant? Could you add a sentence to explain the usage of variables here, such as "$uri"? Maybe a small example would be enough.

Thank you!
reply via email to

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