Opened 16 years ago

Closed 16 years ago

#199 closed task (fixed)

svndigest property to override file extension setting parsing properties is needed

Reported by: Jari Häkkinen Owned by: Peter Johansson
Priority: major Milestone: 0.6
Component: statistics Version: trunk
Keywords: Cc:


Files like source:trunk/test/ should be treated as a bash script but since the file ends with .in it cannot be assigned to bash parsing. We should add something like svndigest:parsingmode that could override default parsing mode.

Change History (7)

comment:1 Changed 16 years ago by Peter Johansson

Well, from my horizon, it ends with so we could add a rule: * files should be interpreted as bash script.

also see #122

comment:2 Changed 16 years ago by Jari Häkkinen

In principle yes ... but should have a similar rule for all possible files? Does ticket:122 rule out this ticket? I think this ticket is valid even if we add as a rule or have configurable associations.

comment:3 Changed 16 years ago by Peter Johansson

No, ticket:122 does not rule out this issue, but we should have ticket:122 in mind; and I suppose it is clever to wait with this issue until ticket:122 is implemented.

Still, I don't like the idea. Your solution would be great if the problem were problems with exceptions to the file name rules. It could, e.g., be that you had a file named file.h containing perl code. Then it would make sense to mark the file as an exception to not follow regular name conventions.

You are reporting a different problem. The suffix .in is used quite often and for many types of files. It would be stupid to add a rule for every file extension X to also work for Instead one could perhaps make svndigest a bit smarter, by adding the feature to ignore some parts of the name. In this case we want .in to be ignored so when the Parser gets a file named, the Parser ignores the trailing .in and only sees

comment:4 Changed 16 years ago by Jari Häkkinen

Ok, lets make svndigest to parse files according to rules according to filetypes when the trailing .in is removed. If we need my suggestion we can add it sometime later. Is this easy to add, i.e., should we have in miletone:0.6?

comment:5 Changed 16 years ago by Jari Häkkinen

Milestone: 0.6

comment:6 Changed 16 years ago by Peter Johansson

Status: newassigned

comment:7 Changed 16 years ago by Peter Johansson

Resolution: fixed
Status: assignedclosed

fixed in [339]

Note: See TracTickets for help on using tickets.