New Windows Install may fail for want of a Jetty tmp dir

Description

This needs research, but when I smoke tested the installer I got this failure:

2016-11-11 05:42:13 Commons Daemon procrun stderr initialized java.nio.file.NoSuchFileException: C:\Program Files\Shibboleth\IdP\jetty-base\tmp\start_410949774866271910.properties at sun.nio.fs.WindowsException.translateToIOException(Unknown Source) at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source) at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source) at sun.nio.fs.WindowsFileSystemProvider.newByteChannel(Unknown Source) at java.nio.file.Files.newByteChannel(Unknown Source) at java.nio.file.Files.createFile(Unknown Source) at java.nio.file.TempFileHelper.create(Unknown Source) at java.nio.file.TempFileHelper.createTempFile(Unknown Source) at java.nio.file.Files.createTempFile(Unknown Source) at org.eclipse.jetty.start.StartArgs.getMainArgs(StartArgs.java:596) at org.eclipse.jetty.start.Main.invokeMain(Main.java:204) at org.eclipse.jetty.start.Main.start(Main.java:457) at org.eclipse.jetty.start.Main.main(Main.java:75)

Creating the folder allows the service to start. This should be an easy fix but I need to get home to be able to fully test

Environment

None

Activity

Show:

Rod WiddowsonJanuary 6, 2017 at 11:54 AM

66ce060

Rod WiddowsonJanuary 6, 2017 at 8:22 AM

taking back

Tom ZellerJanuary 5, 2017 at 9:38 PM

I think that's fine, Rod. (by "that" I mean renaming tmp/.gitkeep to tmp/.donotdelete).

FWIW We manually exclude .gitkeep from the artifact via the maven-resources-plugin.

https://maven.apache.org/plugins/maven-resources-plugin/copy-resources-mojo.html#addDefaultExcludes

Rod WiddowsonJanuary 4, 2017 at 2:44 PM

Tom,
Can you comment on this one

It seems that the packaging explicitly pulls the files called .gitkeep from the distro. As noted above this gives the window installer tummy troubles.

I could fix this in the installer - by forcing the directory to be made, but that seems like an abuse of the architecture.

My preferred solution is to rename the .gitkeep to .donotdelete. That way it will escape the packaging and turn up in the distro - this is no bad thing to my mind because end users might just care...

Thoughts?

Rod WiddowsonNovember 11, 2016 at 8:33 PM

(for 3.3.0.1 I can use the zip that I created, but I have a complete abhorence of building real products from non-real artefacts, so if we did that I'd want to check in the hand-adjusted zip file into the tag, just like I do the jetty and procrun zips)

Fixed

Details

Assignee

Reporter

Original estimate

Components

Fix versions

Affects versions

Created November 11, 2016 at 10:57 AM
Updated October 3, 2017 at 9:23 PM
Resolved January 6, 2017 at 11:54 AM