Sonar Plugins

Plugin does not accept absolute path as src directory

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: WEB-1.0.1
  • Fix Version/s: WEB-1.1
  • Component/s: Web
  • Labels:
    None
  • Number of attachments :
    0

Issue Links

duplicates

Improvement - An improvement or enhancement to an existing feature or task. SONARPLUGINS-837 Add Support for the Use of Absolute Paths When Specifying sonar.web.sourceDirectory

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Matthijs Galesloot added a comment -

This issue was rejected in SONARPLUGINS-837 as it does not fit with the best practices for continuous integration.

Show
Matthijs Galesloot added a comment - This issue was rejected in SONARPLUGINS-837 as it does not fit with the best practices for continuous integration.
Hide
Permalink
Matthijs Galesloot added a comment -

Evgeny

Please comment on the issue and assign back to me.

This is the 2d time we get a request for absolute paths. Should we do it?

Show
Matthijs Galesloot added a comment - Evgeny Please comment on the issue and assign back to me. This is the 2d time we get a request for absolute paths. Should we do it?
Hide
Permalink
Evgeny Mandrikov added a comment -

I didn't changed my mind - absolute path in POM leads to problems with reproducibility of builds. But maybe this issue isn't about this, because "$

{basedir}

/src" is also absolute path So in any case it's better to ask reporter - Olivier.

Show
Evgeny Mandrikov added a comment - I didn't changed my mind - absolute path in POM leads to problems with reproducibility of builds. But maybe this issue isn't about this, because "$ {basedir} /src" is also absolute path So in any case it's better to ask reporter - Olivier.

People

  • Assignee:
    Matthijs Galesloot
    Reporter:
    Olivier Gaudin
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: