Skip to content

dinitctl: show the service path in the "status" command output#550

Closed
mobin-2008 wants to merge 1 commit into
davmac314:masterfrom
mobin-2008:dinit_show_service_dir
Closed

dinitctl: show the service path in the "status" command output#550
mobin-2008 wants to merge 1 commit into
davmac314:masterfrom
mobin-2008:dinit_show_service_dir

Conversation

@mobin-2008
Copy link
Copy Markdown
Collaborator

Use the existing functionality to show the service path of currently loaded service.

Fixes #508

Use the existing functionality to show the service path of currently
loaded service.

Addresses davmac314#508
@mobin-2008 mobin-2008 added A-Importance: Normal Enhancement/New Feature Improving things or introduce new feature C-dinitctl Things about dinitctl labels May 31, 2026
@davmac314
Copy link
Copy Markdown
Owner

Someone was already working on this (#511). It does look like that effort stalled, but could you confirm with them please before we proceed with this new PR, out of politeness.

It would be good if you could communicate what you are intending to work on before you start doing it, that will help avoid this sort of situation.

@mobin-2008
Copy link
Copy Markdown
Collaborator Author

Someone was already working on this (#511). It does look like that effort stalled, but could you confirm with them please before we proceed with this new PR, out of politeness.

I asked him and I'm waiting for his response.

It would be good if you could communicate what you are intending to work on before you start doing it, that will help avoid this sort of situation.

Yes. I should did more than assigning issue (#508) to myself.

@davmac314
Copy link
Copy Markdown
Owner

Let's give it 3 days to wait for an answer. In the meantime, can you please close this PR and re-open after that time (I assume you have properly self-reviewed, if not please do that before re-opening).

@mobin-2008 mobin-2008 closed this Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Importance: Normal C-dinitctl Things about dinitctl Enhancement/New Feature Improving things or introduce new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ability to Print service file path

2 participants