On 01/07/2017 06:16 PM, William Brown wrote:
Okay, so it depends on where you installed your built code to. If you did "./configure --prefix=/path/to/install, it won't work because we now don't use PREFIX to find and work out the paths. The code that caused this is in lib389 with the paths module and defaults.inf. See _get_defaults_loc in lib389/lib389/paths.py . The reason for the change was that PREFIX was hugely broken. It either worked from RPM, or it worked with make install: not never both. This was due to heaps of assumptions being made by us during coding. So the solution I think is to make it so that if you want a prefixed install like you have here, we make _get_defaults_loc check for PREFIX, and if it exists, appends <prefix>/share/dirsrv/inf/defaults.inf, and from there, it should all work. If you want, raise a bug about this and I'll fix it next week first thing when I'm back at work.
Okay, so it depends on where you installed your built code to.
If you did "./configure --prefix=/path/to/install, it won't work because we now don't use PREFIX to find and work out the paths.
The code that caused this is in lib389 with the paths module and defaults.inf. See _get_defaults_loc in lib389/lib389/paths.py .
The reason for the change was that PREFIX was hugely broken. It either worked from RPM, or it worked with make install: not never both. This was due to heaps of assumptions being made by us during coding.
So the solution I think is to make it so that if you want a prefixed install like you have here, we make _get_defaults_loc check for PREFIX, and if it exists, appends <prefix>/share/dirsrv/inf/defaults.inf, and from there, it should all work.
If you want, raise a bug about this and I'll fix it next week first thing when I'm back at work.
attachment 0001-Ticket-49083-Support-prefix-for-discovery-of-the-def.patch
Thank you, William! It works for me.
commit f624aeb13634ac1bd68dce9554cbcebc20dbb6fc Writing objects: 100% (13/13), 3.07 KiB | 0 bytes/s, done. Total 13 (delta 11), reused 0 (delta 0) To ssh://git.fedorahosted.org/git/389/lib389.git 5c9e2c9..0c7974f master -> master
Metadata Update from @nhosoi: - Issue assigned to firstyear - Issue set to the milestone: CI test 1.0
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 issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/2142
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.