sbt-native-packager icon indicating copy to clipboard operation
sbt-native-packager copied to clipboard

Issue 1264 bugfix refactor

Open archeuclid opened this issue 6 years ago • 6 comments

Hello

Added AshStartScriptPlugin with associated keys, templates, tests. Made it so it does not depend on BashStartScriptPlugin keys and defines. It has to be explicitly enabled with enablePlugins.

The generated scripts do not have quite the same usage as bash/bat scripts. Maybe someone can realign them later and add to docs.

archeuclid avatar Nov 02 '19 17:11 archeuclid

Hi @archeuclid,

Thank you for your contribution! We really value the time you've taken to put this together.

Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:

http://www.lightbend.com/contribute/cla

lightbend-cla-validator avatar Nov 02 '19 17:11 lightbend-cla-validator

Hello Ok

This project need lots of refactor. The scripts must be part of "staging", since it is so close related to JavaAppPackaging. I suggest to write specs for scripts then incorporate them in the "stage".

Also, suggest changing the name from "stage" ("Stager" etc). It is a DevOps term and must be delineated from SoftwareDevs.

Sorry, guys. Lots of stuff to work on hand right now. Use the template .sh in any way you want. I will delete this pull request next time I am here or reject it.

Cheers

archeuclid avatar Nov 20 '19 17:11 archeuclid

Is there any way to use what's here?

briantopping avatar Nov 24 '19 19:11 briantopping

@briantopping unfortunately not. We have no releases for pull requests. You will need to checkout this pull request and build on your own :( Or you finish this PR and we release :)

muuki88 avatar Nov 28 '19 11:11 muuki88

No worries, I converted the code that depends on this to pass arguments as environment variables. My shell skills are poor at best.

briantopping avatar Nov 28 '19 16:11 briantopping

My shell skills are poor at best

You are in good company @briantopping Since kubernetes we have to fight more with YAML than anything else :joy:

muuki88 avatar Nov 29 '19 13:11 muuki88