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

Fix Issue1277: ERROR: sonar.cxx.other.xslt.1.inputs file is not defined. #1348

Closed
wants to merge 2 commits into from

Conversation

rufinio
Copy link
Contributor

@rufinio rufinio commented Nov 30, 2017

XSLT is optional, there should be no message.


This change is Reviewable

@guwirth
Copy link
Collaborator

guwirth commented Dec 2, 2017

Hello @rufinio, can you please explain the use case behind this. In general it would be good to have a description what this xslt feature is doing.

@rufinio
Copy link
Contributor Author

rufinio commented Dec 4, 2017

Hi @guwirth,
I am also curious what's the use case behind this xslt feature.
I have updated sonar-cxx plugin to 0.9.8 and got "ERROR: sonar.cxx.other.xslt.1.inputs file is not defined." without using this feature.
I found the closed issue #1277 that already dealt with this but obviously it was not solved.

@guwirth
Copy link
Collaborator

guwirth commented Dec 4, 2017

@rufinio :-) the original idea was #1096. Idea is to have a built-in preceding XSLT transformation for the sonar.cxx.other. sensor. Therefore I was wondering what you are doing with it without assigning an XSLT? Setting inputs and outputs without an xslt makes no sense to me.

But you are right that sonar.cxx.other. should work without inputs, outputs and xslt.

Copy link
Collaborator

@guwirth guwirth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use #1367 instead

@guwirth guwirth added this to the 0.9.9 milestone Dec 31, 2017
@rufinio rufinio deleted the issue1277 branch January 4, 2018 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants