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

hardcoded RPM log directory #619

Closed
fommil opened this issue Jul 7, 2015 · 6 comments
Closed

hardcoded RPM log directory #619

fommil opened this issue Jul 7, 2015 · 6 comments

Comments

@fommil
Copy link

fommil commented Jul 7, 2015

In

https://github.com/sbt/sbt-native-packager/blob/master/src/main/resources/com/typesafe/sbt/packager/archetypes/java_server/systemloader/systemv/start-rpm-template#L43

RUN_CMD="$exec >> /var/log/${{app_name}}/daemon.log 2>&1 &"

but the packager may have provided a custom log directory, which should be preferred.

@fommil
Copy link
Author

fommil commented Jul 7, 2015

uugh, I was going to send you a quick PR on this but it's more involved than I thought. The logs directory needs to be available in the default replacements list and changing that looks like a big faff.

Workaround is to use a custom script, of course.

@muuki88
Copy link
Contributor

muuki88 commented Jul 7, 2015

Currently I'm working on a major refactoring of the maintainerScripts. After that, it should be a lot easier.

@giampaolotrapasso
Copy link
Contributor

Any forecast about the resolution of this?

@muuki88 muuki88 added the rpm label Aug 12, 2015
@muuki88
Copy link
Contributor

muuki88 commented Aug 12, 2015

Unfortunately not. #625 is almost done, but needs some clean up. And then lets see if there is a good way fixing this.

@louisq
Copy link
Contributor

louisq commented Oct 7, 2015

I was not aware that a issue was opened for this, but the #675 pull request addresses the custom log directory

@muuki88 muuki88 closed this as completed Oct 7, 2015
@muuki88
Copy link
Contributor

muuki88 commented Oct 7, 2015

ah, thanks :)

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

No branches or pull requests

4 participants