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 wrong argument type #27

Merged
merged 1 commit into from Jul 3, 2014

Conversation

Projects
None yet
4 participants
@nschum
Copy link
Contributor

commented Dec 9, 2013

If options.root is set, it is passed as a value instead of a list. That means gcovr will start looking for object files in / (the first character of the absolute path) which (almost literally) takes forever.

@schlauch

This comment has been minimized.

Copy link

commented Feb 8, 2014

Please provide a bug fix release for this :) :+1

@ewanmellor

This comment has been minimized.

Copy link

commented Feb 14, 2014

I am also seeing this. This does not happen with 3.0 -- it is a regression in 3.1.

I made the same comment on #31, since these tickets look related.

whart222 added a commit that referenced this pull request Jul 3, 2014

@whart222 whart222 merged commit e98a15e into gcovr:master Jul 3, 2014

whart222 added a commit that referenced this pull request Jul 3, 2014

Adding test logic to verify #27
Most gcovr tests use the option '-r .' or '-r ..'.  This
test uses an explicit path, which will illustrates the error
fixed by #27.

JamesReynolds pushed a commit to JamesReynolds/gcovr that referenced this pull request Mar 8, 2018

JamesReynolds pushed a commit to JamesReynolds/gcovr that referenced this pull request Mar 8, 2018

Adding test logic to verify gcovr#27
Most gcovr tests use the option '-r .' or '-r ..'.  This
test uses an explicit path, which will illustrates the error
fixed by gcovr#27.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.