Bug #4690

MS-DOS format files causes syntax errors that are hard to debug

Added by R.I. Pienaar over 1 year ago. Updated about 1 year ago.

Status:Accepted Start date:09/02/2010
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:error reporting
Target version:-
Affected Puppet version:0.25.5 Branch:
Keywords:
Votes: 0

Description

Given the simple manifest below saved in DOS format:

class foo {
}

Puppet can’t parse this:

% puppet --parseonly test.pp
err: Could not parse for environment production: Could not match '}' at /home/rip/test.pp:1

It’s worth either giving better errors or supporting DOS format files.

History

#1
Updated by Alan Barrett over 1 year ago

I haven’t tried to find the relevant code, but it might be similar to issue #3514.

#2
Updated by R.I. Pienaar over 1 year ago

  • Affected Puppet version set to 0.25.5

Alan Barrett wrote:

I haven’t tried to find the relevant code, but it might be similar to issue #3514.

Yeh, and that’s already fixed in 2.6.x and 2.6.x doesnt exhibit this behavior. If we’re doing more 0.25.x releases might be worth backporting that.

#3
Updated by James Turnbull over 1 year ago

  • Category set to error reporting
  • Status changed from Unreviewed to Accepted
  • Target version set to 49

#4
Updated by James Turnbull about 1 year ago

  • Target version deleted (49)

Also available in: Atom PDF