Closed
Bug 877653
Opened 13 years ago
Closed 13 years ago
add MPL header to puppetagain source files
Categories
(Infrastructure & Operations :: RelOps: General, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: arich, Assigned: arich)
Details
Attachments
(1 file)
|
4.03 KB,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
Several puppetagain source files are missing the MPL header. This patch adds them back in. It also fixes one ruby syntax error in assert_aspect.rb
Attachment #755931 -
Flags: review?(dustin)
Comment 1•13 years ago
|
||
Comment on attachment 755931 [details] [diff] [review]
Add MPL header for puppet source files; fix ruby syntax error in assert_aspect.rb
Review of attachment 755931 [details] [diff] [review]:
-----------------------------------------------------------------
::: modules/shared/lib/puppet/parser/functions/assert_aspect.rb
@@ +6,4 @@
> newfunction(:assert_aspect, :type => :statement) do |args|
> aspect = args[0]
> message = args[1]
> + if not message
Unrelated, but a fine fix to land at the same time
Attachment #755931 -
Flags: review?(dustin) → review+
| Assignee | ||
Comment 2•13 years ago
|
||
landed https://hg.mozilla.org/build/puppet/rev/c12e5838012e
(copypasta put in the newline before I could add the description :S)
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Component: Server Operations: RelEng → RelOps
Product: mozilla.org → Infrastructure & Operations
You need to log in
before you can comment on or make changes to this bug.
Description
•