Re: [PATCH] vnstat: Update to 1.15

[Matthias Fischer <matthias.fischer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 3188 bytes --]

Hi,

I thought we might keep the LOG and PID-settings from 1.11:

After building there were differences - I didn't find how to change
these with appropriate 'configure'-options, so I used 'sed' instead:

1.11:
LogFile "/var/log/vnstat.log"
PidFile "/var/run/vnstat.pid"

1.15:
LogFile "/var/log/vnstat/vnstat.log"
PidFile "/var/run/vnstat/vnstat.pid"

Best,
Matthias

On 12.07.2016 13:07, Michael Tremer wrote:
> Hi,
> 
> why do we have to move the PID and log files?
> 
> -Michael
> 
> On Tue, 2016-07-12 at 12:56 +0200, Matthias Fischer wrote:
>> Changelog:
>> http://humdi.net/vnstat/CHANGES
>> 
>> I had to add some 'configure'-lines to build this - nevertheless: its
>> working. ;-)
>> 
>> 'vnstat.conf' needed some additional 'sed'-lines, too.
>> 
>> Please review, test and confirm.
>> 
>> Best,
>> Matthias
>> 
>> Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
>> ---
>>  lfs/vnstat | 15 +++++++++++----
>>  1 file changed, 11 insertions(+), 4 deletions(-)
>> 
>> diff --git a/lfs/vnstat b/lfs/vnstat
>> index 1c1333b..07cca69 100644
>> --- a/lfs/vnstat
>> +++ b/lfs/vnstat
>> @@ -1,7 +1,7 @@
>>  #############################################################################
>> ##
>>  #                                                                            
>>  #
>>  # IPFire.org - A linux based
>> firewall                                         #
>> -# Copyright (C) 2007  Michael Tremer & Christian
>> Schmidt                      #
>> +# Copyright (C) 2007-2016  IPFire Team  <info(a)ipfire.org>                    
>>  #
>>  #                                                                            
>>  #
>>  # This program is free software: you can redistribute it and/or
>> modify        #
>>  # it under the terms of the GNU General Public License as published
>> by        #
>> @@ -24,7 +24,7 @@
>>  
>>  include Config
>>  
>> -VER        = 1.11
>> +VER        = 1.15
>>  
>>  THISAPP    = vnstat-$(VER)
>>  DL_FILE    = $(THISAPP).tar.gz
>> @@ -40,7 +40,7 @@ objects = $(DL_FILE)
>>  
>>  $(DL_FILE) = $(DL_FROM)/$(DL_FILE)
>>  
>> -$(DL_FILE)_MD5 = a5a113f9176cd61fb954f2ba297f5fdb
>> +$(DL_FILE)_MD5 = 351051ef3005e3ca99123eec07ac0a7d
>>  
>>  install : $(TARGET)
>>  
>> @@ -73,9 +73,16 @@ $(subst %,%_MD5,$(objects)) :
>>  $(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
>>  	@$(PREBUILD)
>>  	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zxf $(DIR_DL)/$(DL_FILE)
>> -	cd $(DIR_APP) && make all $(MAKETUNING)				
>> 						LOCAL_CONFIGURE_OPTIONS="
>> --enable-readline=yes"
>> +
>> +	cd $(DIR_APP) && ./configure \
>> +		--prefix=/usr \
>> +		--sysconfdir=/etc
>> +
>> +	cd $(DIR_APP) && make all $(MAKETUNING)		LOCAL_CONFIGUR
>> E_OPTIONS="--enable-readline=yes"
>>  	cd $(DIR_APP) && make install
>>  	sed -i 's|eth0|green0|g' /etc/vnstat.conf
>>  	sed -i 's|/var/lib/vnstat|/var/log/vnstat|g' /etc/vnstat.conf
>> +	sed -i 's|/var/log/vnstat/vnstat.log|/var/log/vnstat.log|g'
>> /etc/vnstat.conf
>> +	sed -i 's|/var/run/vnstat/vnstat.pid|/var/run/vnstat.pid|g'
>> /etc/vnstat.conf
>>  	@rm -rf $(DIR_APP)
>>  	@$(POSTBUILD)
>