jonasplaum opened a new pull request, #183:
URL: https://github.com/apache/couchdb-pkg/pull/183

   ## Overview
   
   **The Debain Packages don't handle the bind_address correct while updating.**
   
   **Problem 1:** If there is more than one bind_address in the local.ini, for 
example one for chttpd and one for prometheus the configure step fails. Now 
only the bind_address of the [chttpd] block is used.
   
   **Problem 2:** The config script also looked into all of the local.d/*.ini 
files. This didn't work because of a wrong if condition. The condition was 
changed, so that i correctly evaluates if config files exist.
   
   ## Testing recommendations
   
   Install CouchDB and configure an bind_address in the chttpd block of the 
local.ini or any local.d/*.ini file. Also add a second block that also contains 
a bind_address to the same file. Then install CouchDB again (using dpkg -i as 
it allows to install the same version over and over)
   The configure step of the package should not fail. 
   If using `DEBCONF_DEBUG=developer dpkg -i` to install it should show the 
most relevant bind_address found:
   ```
   debconf (developer): --> 0 127.0.0.3
   Processing triggers for man-db (2.11.2-2) ...
   ```
   This should be the bind_address from the alphabetical last ini file in 
local.d if it contains or the bind_address from local.ini if there are no 
further config files.
   
   ## GitHub issue number
   
   Fixes #175 
   
   ## Related Pull Requests
   
   ## Checklist
   
   - [x ] Code is written and works correctly;
   - [ ] Changes are covered by tests;
   - [ ] Documentation reflects the changes;
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to