#2978 pid file name decalration is duplicated
Closed: Fixed None Opened 5 years ago by simo.

While looking at the monitor code I realize we define SSSD_PIDFILE_PATH
in monitor.c in a different way than we define SSSD_PIDFILE in
tools_util.h

Although the definitions differ, they end up being effectively the same
string for now.

It seem to me those definition should be merged and harmonized into one.
If not a comment should be put in the code explaining why we have 2
(potentially) different pid file names.


Easy fix, let's do this in 1.14.

milestone: NEEDS_TRIAGE => SSSD 1.14 beta

Fields changed

rhbz: => 0

Code deduplication doesn't have to block the beta release.

milestone: SSSD 1.14 beta => SSSD 1.14.0

Fields changed

owner: somebody => pcech

Fields changed

priority: major => minor

1.14.0 should be released no later than Wednesday next week, this ticket should not block the 1.14.0 release.

milestone: SSSD 1.14.0 => SSSD 1.14.1

Fields changed

status: new => assigned

Fields changed

patch: 0 => 1

resolution: => fixed
status: assigned => closed

pushed also to sssd-1-13 due to other patches

sssd-1-13:

Metadata Update from @simo:
- Issue assigned to pcech
- Issue set to the milestone: SSSD 1.14.1

4 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/4019

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata