search

vergoh / vnstat

vnStat - a network traffic monitor for Linux and BSD
GNU General Public License v2.0
1.36k stars 120 forks source link

Autotools #184

Closed sevu closed 3 years ago

sevu commented 3 years ago

When using ./configure --sbindir=/usr/bin, the changed path was used for vnstatd, but the printed ooutput, which exists to show the user the values which are used, did show the wrong path.

Example output now:

configure:  ============================================== 
configure: 
configure:          host: x86_64-unknown-linux-gnu 
configure:         build: x86_64-unknown-linux-gnu 
configure: 
configure:        prefix: /usr      
configure:   exec_prefix: ${prefix} 
configure:        bindir: ${exec_prefix}/bin      
configure:       sbindir: /usr/bin     
configure:    sysconfdir: ${prefix}/etc  
configure: 
configure:    Install paths 
configure: 
configure:     vnstat: ${exec_prefix}/bin/vnstat 
configure:    vnstati: ${exec_prefix}/bin/vnstati 
configure:    vnstatd: /usr/bin/vnstatd 
configure:     config: ${prefix}/etc/vnstat.conf 
configure: 
configure:           Tests: available 
configure:    Image output: enabled 
configure: 
configure:  ==============================================

(only prefix and sbindir were given)

Previously it was nicely showing full paths, as only a variable was used, which doesn't depend on other variables. However, better correct than nice. Maybe it's even more useful that way, as it is intended for people who might want to change the paths, and that way which options need to be changed for that.

vergoh commented 3 years ago

Thanks. You are right, the output wasn't showing correctly if for example --sbindir was used while Makefile.am did refer to those directories that then got changed.