Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SQ 5.2 to Travis build #648

Closed
guwirth opened this issue Oct 18, 2015 · 8 comments
Closed

Add SQ 5.2 to Travis build #648

guwirth opened this issue Oct 18, 2015 · 8 comments
Assignees
Milestone

Comments

@guwirth
Copy link
Collaborator

guwirth commented Oct 18, 2015

Add SQ 5.2 to Travis build:
https://sonarsource.bintray.com/Distribution/sonarqube/sonarqube-5.2-RC2.zip

See #649

@guwirth guwirth self-assigned this Oct 18, 2015
@guwirth guwirth added this to the M 0.9.4 milestone Oct 18, 2015
@jmecosta
Copy link
Member

Curious to see if this works. At least I was not able to run c++ analysis without the colorizer

@guwirth
Copy link
Collaborator Author

guwirth commented Oct 18, 2015

Also forceZeroCoverage seems no more to work?

@jmecosta
Copy link
Member

Does that thing do some processing that access db? This is no longer
possible in plugin side. We need to update to 5.2 rc2 to see what
components are broken. Problem I can't find those artifacts anywhere

On Sun, Oct 18, 2015, 16:18 Günter Wirth [email protected] wrote:

Also forceZeroCoverage seems no more to work?


Reply to this email directly or view it on GitHub
#648 (comment).

@guwirth
Copy link
Collaborator Author

guwirth commented Oct 18, 2015

Currently known issues:

  • additional warning: WARN - Unable to get a valid mac address, will use a dummy address
  • Seems there is a problem to access the analysis log. This is different with SQ 5.2? Think SQ will create it on server side and provide it delayed? Blocker to use sonar runner in build automation.
  • Also forceZeroCoverage seems no more to work?

@guwirth
Copy link
Collaborator Author

guwirth commented Oct 18, 2015

Does that thing do some processing that access db?

Code is here https://github.com/wenns/sonar-cxx/blob/master/sonar-cxx-plugin/src/main/java/org/sonar/plugins/cxx/coverage/CxxCoverageSensor.java. Maybe context.GetMeasure? How do I know which API is using the DB?

We need to update to 5.2 rc2 to see what components are broken.

Depends. If there are breaking changes in SQ 5.2 we should first release a final 0.9.4. For my understanding plugin should work with SQ 4.5 to 5.2.

Problem I can't find those artifacts anywhere

What are you looking for?

@jmecosta
Copy link
Member

On Sun, 18 Oct 2015 at 18:33 Günter Wirth [email protected] wrote:

Does that thing do some processing that access db?

Code is here
https://github.com/wenns/sonar-cxx/blob/master/sonar-cxx-plugin/src/main/java/org/sonar/plugins/cxx/coverage/CxxCoverageSensor.java.
Maybe context.GetMeasure? How do I know which API is using the DB?

yep, it looks like its that: changes are here:
https://groups.google.com/forum/#!topic/sonarqube/ZCIsvd41Esk but unclear
to me what needs changing

We need to update to 5.2 rc2 to see what components are broken.

Depends. If there are breaking changes in SQ 5.2 we should first release a
final 0.9.4. For my understanding plugin should work with SQ 4.5 to 5.2.

is not 0.9.3 already supporting 5.1.2? or is there something really
important in 0.9.4-SNAPSHOT. if we release 0.9.4 only supporting 5.1.2 then
there will be a big period where we dont have a version compatible with
5.2. or you need to provide two releases really close to each other.

Problem I can't find those artifacts anywhere

What are you looking for?

was trying to follow these:
https://groups.google.com/forum/#!topic/sonarqube/sJLMsMUxo5A, so bump the
api to 5.2-RC2 but build fails


Reply to this email directly or view it on GitHub
#648 (comment).

@guwirth
Copy link
Collaborator Author

guwirth commented Oct 18, 2015

is not 0.9.3 already supporting 5.1.2?

Because there are 121 changes since v0.9.3: https://github.com/wenns/sonar-cxx/issues?q=milestone%3A%22M+0.9.4%22+is%3Aclosed. Make them only for 5.2 users available makes less sense. 5.2 is also a big change of SQ. Fear it needs a little bit longer to be able to use it really productive.

@jmecosta
Copy link
Member

, let's release it. It's been in RC 3 for long enough

On Sun, Oct 18, 2015, 19:46 Günter Wirth [email protected] wrote:

is not 0.9.3 already supporting 5.1.2?

Because there are 121 changes since v0.9.3:
https://github.com/wenns/sonar-cxx/issues?q=milestone%3A%22M+0.9.4%22+is%3Aclosed.
Make them only for 5.2 users available makes less sense. 5.2 is also a big
change of SQ. Fear it needs a little bit longer to be able to use it really
productive.


Reply to this email directly or view it on GitHub
#648 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants