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

Rpm should have option to make systemd file config #1144

Closed
tolikr opened this issue Aug 2, 2018 · 4 comments · Fixed by #1351
Closed

Rpm should have option to make systemd file config #1144

tolikr opened this issue Aug 2, 2018 · 4 comments · Fixed by #1351
Labels

Comments

@tolikr
Copy link
Contributor

tolikr commented Aug 2, 2018

Expected behaviour

app.service file should not be marked as config in rpm spec file.

Actual behaviour

app.service marked as config, and there is no option to make it non config.

Information

  • What sbt-native-packager are you using 1.3.6
  • What sbt version 1.1.6
  • What is your build system (e.g. Ubuntu, MacOS, Windows, Debian ) Ubuntu
  • What package are you building (e.g. docker, rpm, ...) rpm
  • What version has your build tool (find out with e.g. rpm --version) RPM version 4.14.1
  • What is your target system (e.g. Ubuntu 16.04, CentOS 7) CentOS 7

Related SO:
https://stackoverflow.com/questions/51611115/sbt-native-packager-set-systemd-service-file-to-non-config

@tolikr
Copy link
Contributor Author

tolikr commented Aug 2, 2018

@tolikr
Copy link
Contributor Author

tolikr commented Aug 2, 2018

Workaround for me is:

linuxPackageMappings in Rpm := {
   val before = (linuxPackageMappings in Rpm).value
   val after = before.find(_.mappings.exists(_._2.contains(".service"))).fold(before) {
     mapping => before.filter(_ != mapping) :+ mapping.withConfig("false")
   }
   after
}

@muuki88 muuki88 added bug rpm and removed bug labels Aug 2, 2018
@muuki88
Copy link
Contributor

muuki88 commented Aug 2, 2018

What's the reasons you want this file to be non-config? After all the systemd file is nothing but a config that specifies how to start you service and under which conditions.

@tolikr
Copy link
Contributor Author

tolikr commented Aug 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants