Sonar Plugins

On windows, the value of the sources parameter can't contain the '\' character

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Won't Fix
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: Sonar Runner
  • Labels:
    None
  • Environment:
    Sonar 2.6
  • Number of attachments :
    0

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Fabrice Bellingard added a comment -

Tested with a classic Java project by replacing "sources=src/main/java" by "src\\main
java".

=> I guess the problem was that the back-slash was not escaped.

Show
Fabrice Bellingard added a comment - Tested with a classic Java project by replacing "sources=src/main/java" by "src\\main java". => I guess the problem was that the back-slash was not escaped.
Hide
Permalink
Freddy Mallet added a comment -

I see what you mean Fabrice but I think this was the goal of this ticket to be able to use either '/' or '\' as the directory/file separator.

The logic could be the following one :

  • If the source directory exists -> do nothing
  • If the source directory doesn't exist -> replace all '\' by '/' -> after this conversion if the source directory exists, use this path
Show
Freddy Mallet added a comment - I see what you mean Fabrice but I think this was the goal of this ticket to be able to use either '/' or '\' as the directory/file separator. The logic could be the following one : If the source directory exists -> do nothing If the source directory doesn't exist -> replace all '\' by '/' -> after this conversion if the source directory exists, use this path
Hide
Permalink
Fabrice Bellingard added a comment -

As seen with Freddy: if we start allowing '\' for this use case, this means that, to be consistent, we have to do it everywhere in Sonar (platform and plugins) where a property is a path - which would cost too much for the moment without some kind of API for that.

Show
Fabrice Bellingard added a comment - As seen with Freddy: if we start allowing '\' for this use case, this means that, to be consistent, we have to do it everywhere in Sonar (platform and plugins) where a property is a path - which would cost too much for the moment without some kind of API for that.

People

  • Assignee:
    Fabrice Bellingard
    Reporter:
    Olivier Gaudin
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: