Skip to content

Conversation

@pzindyaev
Copy link
Collaborator

No description provided.

@pzindyaev pzindyaev force-pushed the feature/systemd branch 4 times, most recently from 74eae1c to 44b0243 Compare November 7, 2025 07:20
fs.BoolVar(listAll, "l", false, "shorthand for --list")
printCACerts := fs.Bool("print-cacerts", false, "When used with --runtime=java, prints the cacerts path and exits")
start := fs.Bool("start", false, "Start the service. Use with simple/exec services")
supervise := fs.Bool("supervise", false, "Supervise the service. Use with norify services")
Copy link
Collaborator

Choose a reason for hiding this comment

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

norify -> notify

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

continue
}
// Split on the first '=' to get the key-value pair
splitSubstrings := 2
Copy link
Collaborator

Choose a reason for hiding this comment

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

We usually encounter xxx-env.sh with a script, maybe add support too (just source)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added support for sourcing the env file before running the executable
It's a bit hacky, but I didn't find any other reliable way to support script sourcing in go

@pzindyaev pzindyaev force-pushed the feature/systemd branch 2 times, most recently from e4f9b5d to a625946 Compare November 7, 2025 14:09
@Asmoday Asmoday self-requested a review November 10, 2025 07:36
@pzindyaev pzindyaev merged commit c73f221 into arenadata:main Nov 10, 2025
3 checks passed
@pzindyaev pzindyaev deleted the feature/systemd branch November 10, 2025 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants