877334 - remove hyphenated names from puppetagain

Status: RESOLVED FIXED

(Infrastructure & Operations :: RelOps: General, task)

Description

[Amy Rich [:arr] [:arich]]

Attachment: patch to remove hyphenated names

Both puppetagain and puppet320 have a few hyphenated names still kicking around that we should fix.  Be gentle, this is my first patch.  Hope I managed to get it right.  :}

Comment 1

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 755555 [details] [diff] [review]
patch to remove hyphenated names

This looks like it's hit all of the includes of the hyphenated names, which is good.  However, it doesn't appear to rename the files.  For example, modules/tweaks/manifests/dev-ptmx.pp needs to be hg mv'd to modules/tweaks/manifests/dev_ptmx.pp, otherwise puppet won't be able to find it.  The resulting hg diff should look like

diff --git a/modules/tweaks/manifests/dev-ptmx.pp b/modules/tweaks/manifests/dev_ptmx.pp
--- a/modules/tweaks/manifests/dev-ptmx.pp
+++ b/modules/tweaks/manifests/dev_ptmx.pp

Comment 2

[Amy Rich [:arr] [:arich]]

Attachment: fix hyphens, rename files, a clean up a few other syntax errors

Okay, I renamed the files, too.  I also cleaned up a few other small syntax issues.

Comment 3

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 755578 [details] [diff] [review]
fix hyphens, rename files, a clean up a few other syntax errors

Review of attachment 755578 [details] [diff] [review]:
-----------------------------------------------------------------

Good to commit with this fix.  Better wait until tomorrow to land it, though.

::: modules/tweaks/manifests/dev-ptmx.pp
@@ +1,4 @@
>  # This Source Code Form is subject to the terms of the Mozilla Public
>  # License, v. 2.0. If a copy of the MPL was not distributed with this
>  # file, You can obtain one at http://mozilla.org/MPL/2.0/.
> +class tweaks::dev_ptmx {

This one needs a rename, too!

Comment 4

[Amy Rich [:arr] [:arich]]

Landed!