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 #773 #814

Merged
merged 1 commit into from
Mar 26, 2016
Merged

fix #773 #814

merged 1 commit into from
Mar 26, 2016

Conversation

guwirth
Copy link
Collaborator

@guwirth guwirth commented Mar 25, 2016

No description provided.

@guwirth guwirth added the bug label Mar 25, 2016
@guwirth guwirth added this to the 0.9.5 milestone Mar 25, 2016
@guwirth guwirth self-assigned this Mar 25, 2016
@guwirth guwirth mentioned this pull request Mar 25, 2016
@guwirth
Copy link
Collaborator Author

guwirth commented Mar 25, 2016

@jmecosta any idea whats wrong with AppVeyor? It's working with Travis.

@jmecosta
Copy link
Member

@guwirth must be the env variable we are passing might not be having the correct value. we need to see the log.

can you change this
https://github.com/SonarOpenCommunity/sonar-cxx/blob/master/appveyor.yml#L68

to

  - ps: Get-ChildItem *.log | % { Push-AppveyorArtifact $_.FullName -FileName $_.Name }

so we can get all logs

@guwirth
Copy link
Collaborator Author

guwirth commented Mar 26, 2016

@jmecosta done

@jmecosta
Copy link
Member

@guwirth this test can only be run in travis,

https://github.com/SonarOpenCommunity/sonar-cxx/blob/master/integration-tests/testdata/cpp-multimodule-project-2-reports/reports-cppcheck/cppcheck-result-0.xml#L3

the paths are set for travis only, so we can skip this test in appveyour.

use this:

Also now the log shows many times this:
23:45:10.535 DEBUG - property expansion: key 'sonar.cxx.cppcheck.reportPath'; value '${TestDataFolder}/cpp-multimodule-project-2-reports/reports-cppcheck/.xml' => 'C:\projects\sonar-cxx\integration-tests\testdata/cpp-multimodule-project-2-reports/reports-cppcheck/.xml'
23:45:10.535 DEBUG - property expansion: key 'sonar.cxx.cppcheck.reportPath'; value '${TestDataFolder}/cpp-multimodule-project-2-reports/reports-cppcheck/.xml' => 'C:\projects\sonar-cxx\integration-tests\testdata/cpp-multimodule-project-2-reports/reports-cppcheck/.xml'
23:45:10.535 DEBUG - property expansion: key 'sonar.cxx.cppcheck.reportPath'; value '${TestDataFolder}/cpp-multimodule-project-2-reports/reports-cppcheck/.xml' => 'C:\projects\sonar-cxx\integration-tests\testdata/cpp-multimodule-project-2-reports/reports-cppcheck/.xml'
23:45:10.535 DEBUG - property expansion: key 'sonar.cxx.cppcheck.reportPath'; value '${TestDataFolder}/cpp-multimodule-project-2-reports/reports-cppcheck/.xml' => 'C:\projects\sonar-cxx\integration-tests\testdata/cpp-multimodule-project-2-reports/reports-cppcheck/.xml'
23:45:10.550 DEBUG - property expansion: key 'sonar.cxx.cppcheck.reportPath'; value '${TestDataFolder}/cpp-multimodule-project-2-reports/reports-cppcheck/.xml' => 'C:\projects\sonar-cxx\integration-tests\testdata/cpp-multimodule-project-2-reports/reports-cppcheck/.xml'
23:45:10.550 DEBUG - property expansion: key 'sonar.cxx.cppcheck.reportPath'; value '${TestDataFolder}/cpp-multimodule-project-2-reports/reports-cppcheck/.xml' => 'C:\projects\sonar-cxx\integration-tests\testdata/cpp-multimodule-project-2-reports/reports-cppcheck/.xml'
23:45:10.566 DEBUG - property expansion: key 'sonar.cxx.cppcheck.reportPath'; value '${TestDataFolder}/cpp-multimodule-project-2-reports/reports-cppcheck/.xml' => 'C:\projects\sonar-cxx\integration-tests\testdata/cpp-multimodule-project-2-reports/reports-cppcheck/.xml'

Perhaps we dont need the duplicated information or add the reference to the module where the expansion is being done?

@guwirth
Copy link
Collaborator Author

guwirth commented Mar 26, 2016

@jmecosta thx it's working now.

@guwirth
Copy link
Collaborator Author

guwirth commented Mar 26, 2016

close #773

@guwirth guwirth merged commit 7e7020e into SonarOpenCommunity:master Mar 26, 2016
@guwirth guwirth deleted the bugfix/773 branch April 3, 2016 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants