All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Jackson <ian.jackson@eu.citrix.com>
To: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: xen-devel@lists.xenproject.org, Marcos.Matsunaga@oracle.com
Subject: Re: [PATCH 3/3] README: Mention the / requirement in Tftp[Dir|Tmp]Dir
Date: Wed, 26 Oct 2016 16:12:47 +0100	[thread overview]
Message-ID: <22544.51183.307779.409346@mariner.uk.xensource.com> (raw)
In-Reply-To: <1477505649-84103-4-git-send-email-konrad.wilk@oracle.com>

Konrad Rzeszutek Wilk writes ("[PATCH 3/3] README: Mention the / requirement in Tftp[Dir|Tmp]Dir"):
> That is the / should be part of the directory name. Otherwise
> we get strange files such as: pxelinux.cfgC0A86A3C
> in the TftpPath directory.

I've edited this slightly.

Ian.

From 9f0b83ebefc90c46fafd8efdba2b258926d18be0 Mon Sep 17 00:00:00 2001
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Date: Wed, 26 Oct 2016 14:14:09 -0400
Subject: [OSSTEST PATCH] README: Mention the / requirement in Tftp[Dir|Tmp]Dir

That is the / should be part of the directory name. Otherwise
we get strange files such as: pxelinux.cfgC0A86A3C
in the TftpPath directory.

Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
v2: Slightly improved wording.
---
 README | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/README b/README
index 167c1bd..e3519a4 100644
--- a/README
+++ b/README
@@ -497,9 +497,10 @@ Tftp*
     TftpPath          The path to the root of the directory which is exposed by
                       the tftpserver (e.g. /tftpboot).
     TftpTmpDir        A directory under `Path' to use for temporary files.
+                      Make sure you include the trailing / (e.g. tmp/)
 
     TftpPxeDir        The path under `Path' to the PXE configuration directory
-                      (e.g. pxelinux.cfg)
+                      (e.g. pxelinux.cfg/).   Include the trailing /.
     TftpPxeGroup      The Unix group which should own files under `PxeDir'.
     TftpPxeTemplates  See TftpPxeTemplates
     TftpPxeTemplatesReal
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

  reply	other threads:[~2016-10-26 15:13 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-26 18:14 [PATCH OSSTEST] Various fixes when using standalone Konrad Rzeszutek Wilk
2016-10-26 18:14 ` [PATCH 1/3] DhcpWatch: Print what is wrong with the line Konrad Rzeszutek Wilk
2016-10-26 15:10   ` Ian Jackson
2016-10-26 15:15     ` Konrad Rzeszutek Wilk
2016-10-26 18:14 ` [PATCH 2/3] standalong-reset: Use the $suite when recreating soft links Konrad Rzeszutek Wilk
2016-10-26 18:14 ` [PATCH 3/3] README: Mention the / requirement in Tftp[Dir|Tmp]Dir Konrad Rzeszutek Wilk
2016-10-26 15:12   ` Ian Jackson [this message]
2016-10-26 15:16     ` Konrad Rzeszutek Wilk
2016-10-26 15:38       ` Ian Jackson
2016-10-26 18:51   ` [PATCH] standalone-reset: Check for TftpPxeDir Konrad Rzeszutek Wilk
2016-10-26 15:48     ` Ian Jackson
2016-10-26 15:52       ` Konrad Rzeszutek Wilk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=22544.51183.307779.409346@mariner.uk.xensource.com \
    --to=ian.jackson@eu.citrix.com \
    --cc=Marcos.Matsunaga@oracle.com \
    --cc=konrad.wilk@oracle.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.