[ 
https://issues.apache.org/jira/browse/OFBIZ-7963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16848356#comment-16848356
 ] 

Jacques Le Roux commented on OFBIZ-7963:
----------------------------------------

Hi Pierre,

Your last patch gives the same "result" than I put above (normal it's the same 
thing).

I looked at the reason it fails. This is related: 
[https://github.com/SonarOpenCommunity/sonar-cxx/wiki/FAQ] (look for "Q: ERROR: 
Caused by: File [...] can't be indexed twice.").
 They explain:
{noformat}
An example could look like this:

sonar.sources=.
sonar.tests=.
sonar.test.inclusions=**/*Test*/**
sonar.exclusions=**/*Test*/**
{noformat}
Should those be properties in sonarqube section? What should we put there 
actually? Would you mind investigating?
If I understand well the 1st test failing 
(MultivaluedMapContextAdapterTests.java) is the 1st running, all others are the 
same. So as long as we have not defined some sonarqube parameters (maybe 
beggining with the explanation above) we will not get any usable result, as you 
did with OFBIZ-10919 and OFBIZ-10918. If I refer to INFRA-18188, where Jenkins 
was used, it's not that easy to transpose to Gradle: Jenkins do the job for you 
there as says Fabrice Bellingard.

So you don't need to answer my questions at OFBIZ-10917, I think I got the big 
picture and I guess we "only" need now to correctly define the properties in 
sonarqube section...



> Create a Gradle Sonar task
> --------------------------
>
>                 Key: OFBIZ-7963
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7963
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Jacques Le Roux
>            Assignee: Pierre Smits
>            Priority: Minor
>         Attachments: OFBIZ-7963-sonar.patch, OFBIZ-7963-sonar.patch, 
> OFBIZ-7963-sonar.patch
>
>
> As ever the devil is in details. There is no longer a Sonar plugin available 
> https://docs.gradle.org/current/userguide/sonar_plugin.htmland we should 
> rather refer to 
> http://docs.sonarqube.org/display/SCAN/Analyzing+with+SonarQube+Scanner+for+Gradle
> Anyway the most important part is to revivify INFRA-3590



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to