#51258 Issue 51121 - Remove hardcoded changelog file name
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base issue51121  into  master

Download 51258.patch

Bug Description:

There are several places in the code where the new changelog name is just a hardcoded string. So if the name changes we will have many places that need to be updated.

Fix Description:

Use a "define" for the file name in the bdb code, and in the changelog get get the filename from backend ldbminfo.

Also extended the ldbminfo flags to include the backend implementation. This will be used as we start to need a way to detect what database backend is in use.

relates: https://pagure.io/389-ds-base/issue/51121

Seems pretty reasonable to me :) Ack

rebased onto e8f069271f79aebed13b3d6c28197a87f40cd7cb

Pull-Request has been merged by mreynolds

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

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/4311

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata