Hi Wolfgang, On Mon, Apr 10, 2017 at 12:41:13AM +0200, Wolfgang Schweer wrote: > On Fri, Apr 07, 2017 at 07:05:58PM +0200, Wolfgang Schweer wrote: > > Another solution might be to drop said symlinking from > > d/debian-edu-config.links and to add it to cf/apache2.cf. > > As the apache2 package should only be installed on the main server by > default, adding apache2 to d-e-config's Recommends is no good solution. > > IMO this should do it right: > > diff --git a/cf/cf.apache2 b/cf/cf.apache2 > index 3f5ba637..6296ea1c 100644 > --- a/cf/cf.apache2 > +++ b/cf/cf.apache2 > @@ -1,6 +1,7 @@ > links: > debian.server.installation:: > /etc/php/apache2/php.ini ->! /etc/php/apache2/php-debian-edu.ini > + /etc/apache2/mods-available/userdir.load ->! > /etc/apache2/mods-available/debian-edu-userdir.load > > shellcommands: > debian.server.installation:: > diff --git a/debian/debian-edu-config.links b/debian/debian-edu-config.links > index 3d6d4821..28cd86b5 100644 > --- a/debian/debian-edu-config.links > +++ b/debian/debian-edu-config.links > @@ -1,2 +1,2 @@ > usr/share/debian-edu-config/tools/ldapdump.sh etc/slbackup/pre.d/ldapdump.sh > -etc/apache2/mods-available/userdir.load > etc/apache2/mods-available/debian-edu-userdir.load
looks good to me, can you please commit this to git?! -- cheers, Holger
signature.asc
Description: Digital signature