Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ash support documentation #652

Merged
merged 1 commit into from
Aug 23, 2015
Merged

ash support documentation #652

merged 1 commit into from
Aug 23, 2015

Conversation

gzoller
Copy link

@gzoller gzoller commented Aug 14, 2015

No description provided.

@muuki88 muuki88 added documentation Documentation should be extended or updated docker labels Aug 15, 2015
@muuki88
Copy link
Contributor

muuki88 commented Aug 15, 2015

Thanks, looks good. The documentation states that the is a need to override the bashScript. I didn't get this at the other pull request. I think we should provide this by default, so it works out-of-the-box.

bashScriptTemplateLocation := {
   // generate the file, so it can be found
   val script = (target in Universal) / "tmp" / "ash-start-script"
   val content = .. // getResource from internal 
   // fill in the contents from the interally shipped jar
   IO.write(script, content)
   // return this as template location
   script
}

Simplifies usage and documentation :)

@gzoller
Copy link
Author

gzoller commented Aug 15, 2015

Makes sense. Let me look at it. That will probably require an ash-template file but that shouldn't be a big deal. I'll put it in this PR.

Sent from my iPhone

On Aug 15, 2015, at 5:41 AM, Nepomuk Seiler [email protected] wrote:

Thanks, looks good. The documentation states that the is a need to override the bashScript. I didn't get this at the other pull request. I think we should provide this by default, so it works out-of-the-box.

bashScriptTemplateLocation := {
// generate the file, so it can be found
val script = (target in Universal) / "tmp" / "ash-start-script"
val content = .. // getResource from internal
// fill in the contents from the interally shipped jar
IO.write(script, content)
// return this as template location
script
}
Simplifies usage and documentation :)


Reply to this email directly or view it on GitHub.

@gzoller
Copy link
Author

gzoller commented Aug 16, 2015

I've created an overridable ash-template file so it works more "out of the box". That'll be better. I am getting a Travis failure but the CI system is a little flakey today and I can't click into the details to know what's failing.

@kardapoltsev
Copy link
Member

I'm also couldn't get access to details.. :(
'There is not an active repository' Travis said.
@muuki88 is there any spercial security configuration at travis side?

@muuki88
Copy link
Contributor

muuki88 commented Aug 16, 2015

@kardapoltsev yeah, this is for a few days now.

@jsuereth I fear this is something the repo-admin has to reconfigure?

override def projectSettings = Seq(
bashScriptTemplateLocation := (sourceDirectory.value / "templates" / ashTemplate),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fear this won't be enough. You have to create the ash template. Something like

bashScriptTemplateLocation := {
   // generate the file, so it can be found
   val script = (target in Universal) / "tmp" / "ash-start-script"
   val content = .. // getResource from internal 
   // fill in the contents from the interally shipped jar
   IO.write(script, content)
   // return this as template location
   script
}

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm afraid I'm confused on this point. I've got something like this in makeUniversalAshScript() in file AshScriptPlugin.scala. I've built it locally and it seems to work great for a project that consumes this feature. Looks like this:

      val template = resolveTemplate(defaultTemplateLocation)
      val scriptBits = JavaAppAshScript.generateScript(defines, template)
      val script = tmpDir / "tmp" / "bin" / name   // fine since this is an ash-specific archetype
      IO.write(script, scriptBits)
      script.setExecutable(true)
      Some(script)

On a separate note, I'm not sure why one of the tests is failing. I'll check but I don't think this is anything I modified. Perhaps I pulled something I shouldn't?

muuki88 added a commit that referenced this pull request Aug 23, 2015
@muuki88 muuki88 merged commit e1dfbf1 into sbt:master Aug 23, 2015
@muuki88
Copy link
Contributor

muuki88 commented Aug 23, 2015

Travis CI flakiness :( Tests run now. I've tested this on my machine as well and everything works smoothly. Great job :)

@muuki88
Copy link
Contributor

muuki88 commented Aug 23, 2015

@gzoller I forgot to ask. You mentioned that you use busybox within the docker build, right? Is there a busybox-java image that we could use as a default for the AshScriptPlugin?

@gzoller
Copy link
Author

gzoller commented Aug 24, 2015

Kinda... It's not provided by busybox but a developer who goes by errordeveloper I think.

Sent from my iPhone

On Aug 23, 2015, at 2:54 PM, Nepomuk Seiler [email protected] wrote:

@gzoller I forgot to ask. You mentioned that you use busybox within the docker build, right? Is there a busybox-java image that we could use as a default for the AshScriptPlugin?


Reply to this email directly or view it on GitHub.

@fiadliel
Copy link
Contributor

Here's one image you might be able to try: https://hub.docker.com/r/develar/java/
The image is based on Alpine Linux (w/busybox), with the Oracle 8 JRE.

Alpine Linux seems to have quite a mini-community (I only came across it this month).
However, it's hard to know yet which will be well-supported in the long term, I haven't seen any image which I would be certain will be kept up to date (because they are single-person endeavours).

@gzoller
Copy link
Author

gzoller commented Aug 24, 2015

Exactly. That's kinda why I didn't wire in a default at this point. Wasn't really sure who'd stick until some time had passed.

@muuki88
Copy link
Contributor

muuki88 commented Aug 24, 2015

Okay, that makes sense. Thanks @fiadliel for the research.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker documentation Documentation should be extended or updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants