[PATCH v2] fff-hoods: move hiddenapfile variable

Tim Niemeyer tim at tn-x.org
Sa Nov 4 15:52:33 CET 2017


Am Samstag, den 04.11.2017, 14:58 +0100 schrieb
mail at adrianschmutzler.de:
> Hallo, ich finde zwar, der gehört in den anderen File. Aber wenn ihr
> es beide so seht, macht ruhig.
> 
> Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
Danke

Und applied.

Tim

> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> > Behalf
> > Of Christian Dresel
> > Sent: Samstag, 4. November 2017 13:27
> > To: Tim Niemeyer <tim at tn-x.org>; franken-dev at freifunk.net
> > Subject: Re: [PATCH v2] fff-hoods: move hiddenapfile variable
> > 
> > hi
> > 
> > macht Sinn da die Variable auch dort verwendet wird und nicht in
> > der lib.
> > 
> > Reviewed-by: Christian Dresel <fff at chrisi01.de>
> > 
> > mfg
> > 
> > Christian
> > 
> > On 03.11.2017 22:55, Tim Niemeyer wrote:
> > > Move the variable definition to the implementation file. This is
> > > more
> > > intuitive and the variable is not used otherwise.
> > > 
> > > Signed-off-by: Tim Niemeyer <tim at tn-x.org>
> > > ---
> > > 
> > > Changes in v2:
> > > - move variable from lib/functions/fff/keyxchange
> > > 
> > >  src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange |
> > > 1 -
> > >  src/packages/fff/fff-hoods/files/usr/sbin/configurehood       |
> > > 1 +
> > >  2 files changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git
> > > a/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
> > > b/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
> > > index 9aeb434..30963ae 100644
> > > --- a/src/packages/fff/fff-
> > > hoods/files/lib/functions/fff/keyxchange
> > > +++ b/src/packages/fff/fff-
> > > hoods/files/lib/functions/fff/keyxchange
> > > @@ -6,7 +6,6 @@
> > > 
> > >  hoodfile="/tmp/keyxchangev2data"
> > >  hoodfilecopy="/www/hood/keyxchangev2data"
> > > -hiddenapfile="/tmp/hiddenapflag"
> > > 
> > >  getJsonPath() {
> > >  	jsonfile=""
> > > diff --git a/src/packages/fff/fff-
> > > hoods/files/usr/sbin/configurehood
> > > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > index 42eb966..b10d210 100755
> > > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > @@ -9,6 +9,7 @@
> > >  sectorlocal=/etc/sectorfile
> > >  sectortmp=/tmp/sectorfile
> > >  sectorcopy=/www/hood/sectorfile
> > > +hiddenapfile=/tmp/hiddenapflag
> > > 
> > >  rm -f "$hoodfile"
> > > 
> > > 
> 
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20171104/b8c10b3c/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev