Filippo Giunchedi has submitted this change and it was merged.
Change subject: sudoers.erb - deprecated variable access
......................................................................
sudoers.erb - deprecated variable access
Variable access via 'privileges' is deprecated.
Use '@privileges' instead.
Variable access via 'user' is deprecated.
Use '@user' instead
...
Change-Id: I5b2a93ec5cf6a3307596a4630cf18dd175ea82a9
---
M templates/sudo/sudoers.erb
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Filippo Giunchedi: Verified; Looks good to me, approved
Matanya: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/templates/sudo/sudoers.erb b/templates/sudo/sudoers.erb
index f5c2472..fd7e9b0 100644
--- a/templates/sudo/sudoers.erb
+++ b/templates/sudo/sudoers.erb
@@ -3,6 +3,6 @@
# vim: sts=0 noet ts=8
-<% privileges.each do |privilege| -%>
- <% if has_variable?("user") then %><%= user %><% elsif
has_variable?("group") then %>%<%= group %><% end %> <%= privilege %>
+<% @privileges.each do |privilege| -%>
+ <% if has_variable?("user") then %><%= @user %><% elsif
has_variable?("group") then %>%<%= @group %><% end %> <%= @privilege %>
<% end -%>
--
To view, visit https://gerrit.wikimedia.org/r/154372
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5b2a93ec5cf6a3307596a4630cf18dd175ea82a9
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Alexandros Kosiaris <akosia...@wikimedia.org>
Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@wikimedia.org>
Gerrit-Reviewer: Matanya <mata...@foss.co.il>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
