diff mbox

standalone-reset: Check for TftpPxeDir

Message ID 1477507905-85518-1-git-send-email-konrad.wilk@oracle.com (mailing list archive)
State New, archived
Headers show

Commit Message

Konrad Rzeszutek Wilk Oct. 26, 2016, 6:51 p.m. UTC
While patch: "README: Mention the / requirement in Tftp[Dir|Tmp]Dir"
we should also double-check that the directory actually exists.

Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
---
 standalone-reset | 3 +++
 1 file changed, 3 insertions(+)

Comments

Ian Jackson Oct. 26, 2016, 3:48 p.m. UTC | #1
Konrad Rzeszutek Wilk writes ("[PATCH] standalone-reset: Check for TftpPxeDir"):
> While patch: "README: Mention the / requirement in Tftp[Dir|Tmp]Dir"
> we should also double-check that the directory actually exists.
> 
> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Konrad Rzeszutek Wilk Oct. 26, 2016, 3:52 p.m. UTC | #2
On Wed, Oct 26, 2016 at 04:48:33PM +0100, Ian Jackson wrote:
> Konrad Rzeszutek Wilk writes ("[PATCH] standalone-reset: Check for TftpPxeDir"):
> > While patch: "README: Mention the / requirement in Tftp[Dir|Tmp]Dir"

Garh

s/[Dir/[Pxe

now that you fixed that up.

Sorry for that!
> > we should also double-check that the directory actually exists.
> > 
> > Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
diff mbox

Patch

diff --git a/standalone-reset b/standalone-reset
index c587e60..69a3226 100755
--- a/standalone-reset
+++ b/standalone-reset
@@ -101,6 +101,9 @@  else
 	tftptmp=`getconfig TftpTmpDir`
 	ensure_dir "$tftp$tftptmp"
 
+	tftpconfig=`getconfig TftpPxeDir`
+	ensure_dir "$tftp$tftpconfig"
+
 	dibase=`getconfig TftpDiBase`
 	suite=`getconfig DebianSuite`
 	diver=`getconfig_TftpDiVersion_suite $suite`