From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] remove forgotten Nagios files, if any
Date: Tue, 24 Apr 2018 15:58:03 +0100
Message-ID: <1524581883.2274586.39.camel@ipfire.org>
In-Reply-To: <e98682ee-b3d4-4b20-9758-d12529898327@link38.eu>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2661480541994158373=="
List-Id: <development.lists.ipfire.org>

--===============2661480541994158373==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hello,

the patch is missing the apache vhost configuration file, /usr/bin/p1.pl and =
the
symlinks in /etc/rc.d/rc*.d

https://git.ipfire.org/?p=3Dipfire-2.x.git;a=3Dblob;f=3Dconfig/rootfiles/pack=
ages/nagi
os;h=3D971a36061f7c2215973e0c4dad47e6ca6824e2a7;hb=3Drefs/heads/core116

-Michael

On Tue, 2018-04-24 at 16:54 +0200, Peter M=C3=BCller wrote:
> Hello Michael,
> > Hey,
> >=20
> > the nagios package should have been removed automatically and remove all
> > program
> > files with it. Did you install a testing core update?
>=20
> At the moment, yes (Core Update 120), but that appeared before, too. Maybe =
the
> dependency to NRPE blocked something from being removed here (issue can be
> reproduced
> on two systems, one of them was never on testing tree).
> >=20
> > Also, Core Update 120 is being packaged and distributed on the servers so
> > far.
> > If nothing major needs fixing, I would like to avoid rebuilding this only
> > for
> > this patch.
>=20
> From my point of view, there is no rush here. I am fine pushing this in Core
> Update
> 121. Just want to clean up the disk.
>=20
> Best regards,
> Peter M=C3=BCller
> >=20
> > Best,
> > -Michael
> >=20
> > On Mon, 2018-04-23 at 16:38 +0200, Peter M=C3=BCller wrote:
> > > When we decided to drop Nagios, some files were not removed on the
> > > installations. Since the package does not exist anymore, "pakfire remove
> > > nagios"
> > > does not work so we need to clean them up manually in case they exist.
> > >=20
> > > Signed-off-by: Peter M=C3=BCller <peter.mueller(a)link38.eu>
> > > ---
> > >  config/rootfiles/core/120/update.sh | 6 ++++++
> > >  1 file changed, 6 insertions(+)
> > >=20
> > > diff --git a/config/rootfiles/core/120/update.sh
> > > b/config/rootfiles/core/120/update.sh
> > > index e4ee15b52..310f669c1 100644
> > > --- a/config/rootfiles/core/120/update.sh
> > > +++ b/config/rootfiles/core/120/update.sh
> > > @@ -86,6 +86,12 @@ done
> > >  # Remove deprecated SSH configuration option
> > >  sed -e "/UsePrivilegeSeparation/d" -i /etc/ssh/sshd_config
> > > =20
> > > +# Remove forgotten Nagios files, if any...
> > > +test -f /etc/rc.d/init.d/nagios && rm -f /etc/rc.d/init.d/nagios
> > > +test -f /usr/bin/nagios && rm -f /usr/bin/nagios
> > > +test -d /usr/share/nagios/ && rm -rf /usr/share/nagios/
> > > +test -d /var/nagios/ && rm -rf /var/nagios/
> > > +
> > >  # Remove any pakfire keys stored in /
> > >  rm -rfv /.gnupg
> > > =20

--===============2661480541994158373==--