jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/368406 )

Change subject: Require MobileFrontend to be installed when enabling Minerva
......................................................................


Require MobileFrontend to be installed when enabling Minerva

Enabling the Minerva role fails because it depends on MobileFrontend
and will throw an exception during the provisioning process.

Bug: T171942
Change-Id: I3f97ea6e5eb6bab4ce6f5e740034ca03e4bbdfb6
---
M puppet/modules/role/manifests/minerva.pp
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  BryanDavis: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/puppet/modules/role/manifests/minerva.pp 
b/puppet/modules/role/manifests/minerva.pp
index 81a967c..65e820c 100644
--- a/puppet/modules/role/manifests/minerva.pp
+++ b/puppet/modules/role/manifests/minerva.pp
@@ -2,5 +2,8 @@
 # Configures Minerva, a MediaWiki skin, as an option.
 # https://www.mediawiki.org/wiki/Skin:MinervaNeue
 class role::minerva {
+    require ::role::mediawiki
+    include ::role::mobilefrontend
+
     mediawiki::skin { 'MinervaNeue': }
 }

-- 
To view, visit https://gerrit.wikimedia.org/r/368406
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3f97ea6e5eb6bab4ce6f5e740034ca03e4bbdfb6
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Mainframe98 <k.s.w...@hotmail.com>
Gerrit-Reviewer: BryanDavis <bda...@wikimedia.org>
Gerrit-Reviewer: Dduvall <dduv...@wikimedia.org>
Gerrit-Reviewer: Mainframe98 <k.s.w...@hotmail.com>
Gerrit-Reviewer: Zfilipin <zfili...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to