Issue #12557 has been updated by Chris Price.

Status changed from Unreviewed to Investigating
Assignee set to Chris Price

Can you provide a full manifest that demonstrates this?  I tried creating a 
file test.pp with these contents:

<pre>
define deploy::php_dev {
}

deploy::php_dev { "02" }
</pre>

and then ran:

<pre>
$ puppet apply ./test.pp
<b>Could not parse for environment production: Syntax error at '}' at 
/home/cprice/work/puppet/git/puppet/scratch/12557.pp:4 on node 
cosmicshame.puppetlabs.lan</b>
</pre>

Which, while not as complete an error message as we would prefer or as you are 
suggesting, at least isn't quite so misleading.


----------------------------------------
Bug #12557: Inaccurate error message when calling a defined function without 
colon after $name
https://projects.puppetlabs.com/issues/12557#change-54683

Author: zach armstrong
Status: Investigating
Priority: Low
Assignee: Chris Price
Category: error reporting
Target version: 
Affected Puppet version: 2.7.9
Keywords: 
Branch: 


Calling a function, like 
<pre>deploy::php_dev { "02" }  </pre>

throws this error: `err: Could not retrieve catalog from remote server: Error 
400 on SERVER: Could not parse for environment production: Syntax error at '}'; 
expected '}' at /etc/puppet/manifests/site.pp:72`

It should say `expected ':'`


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to