[RFC PATCH] Fix uci error caused by symlink to non-existent file

Fabian Bläse fabian at blaese.de
Mi Sep 6 10:56:46 CEST 2017


Hi Christian,

das ist mir durchaus klar, es bleibt ja auch dort. Dieser Patch ergänzt nur, dass selbst wenn das wan-Interface nicht up ist (in welchem Fall die vpn-select nicht ausgeführt wird), trotzdem die /etc/config/tunneldigger (welche nach /tmp/tunneldigger symlinkt) “berührt” wird, damit sie existiert.
Das “-c /tmp” habe ich entfernt, weil es durch den Symlink nicht nötig ist und auch alle anderen Aufrufe auf die /etc/config/tunneldigger (den Symlink) zugreifen. Damit die Configs alle an einer stelle liegen und im uci angezeigt werden, auch wenn man nicht “-c /tmp” angibt, würde ich den Symlink auf jeden Fall in dieser Form behalten.

Gruß
Fabian

> On 6. Sep 2017, at 10:43, Christian Dresel <fff at chrisi01.de> wrote:
> 
> hi
> 
> On 06.09.2017 01:19, Fabian Bläse wrote:
>> Signed-off-by: Fabian Bläse <fabian at blaese.de>
>> ---
>> .../fff/fff-vpn-select/files/etc/hotplug.d/iface/50-vpn-select         | 3 +++
>> src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select              | 2 +-
>> 2 files changed, 4 insertions(+), 1 deletion(-)
>> 
>> diff --git a/src/packages/fff/fff-vpn-select/files/etc/hotplug.d/iface/50-vpn-select b/src/packages/fff/fff-vpn-select/files/etc/hotplug.d/iface/50-vpn-select
>> index 16d9853..c400889 100755
>> --- a/src/packages/fff/fff-vpn-select/files/etc/hotplug.d/iface/50-vpn-select
>> +++ b/src/packages/fff/fff-vpn-select/files/etc/hotplug.d/iface/50-vpn-select
>> @@ -1,4 +1,7 @@
>> #!/bin/sh
>> +
>> +touch /etc/config/tunneldigger
>> +
>> [ "$ACTION" = "ifup" -a "$INTERFACE" = "wan" ] && {
>>        sleep 3
>>        /usr/sbin/vpn-select
>> diff --git a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
>> index c3d003f..016c940 100755
>> --- a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
>> +++ b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
>> @@ -29,7 +29,7 @@ for file in $filecounts; do
>>         uci set tunneldigger.$count.interface="l2tp$count"
>>         uci set tunneldigger.$count.enabled="1"
>>         uci set tunneldigger.$count.hook_script='/etc/tunneldigger/tunneldigger.hook'
>> -        uci -c /tmp commit tunneldigger
>> +        uci commit tunneldigger
> 
> das wurde extra nach /tmp verlagert, weil das Script sonst alle 5
> Minuten auf dem Flash rumschreibt.. Ist also eher ungünstig das wieder
> weg zu nehmen.
> 
> mfg
> 
> Christian
> 
>>         count=$((count + 1))
>>         # remove this fastd-peer
>>         rm /etc/fastd/${project}/peers/$file
>> 
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 801 bytes
Beschreibung: Message signed with OpenPGP
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20170906/f73dde93/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev