-
Notifications
You must be signed in to change notification settings - Fork 204
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
Conflict with Sonarsource C/C++/Objective C plugin #88
Comments
ok, thanks for the report. I have to check the impacts on the history, but it makes sense to change to something else, right?
|
Yes, anything but |
To complete the demand of @jfisbein in order to use both plugins in the same SQ instance and choose which plugin to activate we can force the sonar.language property but both plugins are using the same language's key = objc so indeed would be great to change "objc" in src/main/java/org/sonar/plugins/objectivec/core/ObjectiveC.java to something else. |
I was worried changing this would have really adverse effects on project data, but it seems the only thing you really lose is your quality profiles so that's not too bad. |
Fixed in PR #97 |
How can I change the key then? I am running Octo and getting the "Caused by: No license for objc" error. Thanks! |
You have to build PR #97 from source and then change your key from |
Wow, seems to be a lot of re-work! Is there any chance of uninstalling SonarSource plugin in order to give precedence to Octo? |
Yes, uninstalling one of the plugins will remove the conflict. |
Which is the proper way to uninstall SonarSource plugin? Been trying to achieve this but couldn't... |
Solved the issue, switched plugins and worked. Thanks for your help! |
Sonarsource has a commercial C/C++/Objective C plugin, if you install the Sornarsource plugin it takes precedence even if you only have the license for the C/C++ functionalities (not for Objective C).
It looks like the problem is because both plugins (Octo & SonarSource) share the same key
"objc"
and conflicts.Does it make sense to change the key to something like
"octo-objc"
?The text was updated successfully, but these errors were encountered: