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

gcovr is dead when specify full path #37

Closed
xsyr opened this issue Feb 17, 2014 · 3 comments
Closed

gcovr is dead when specify full path #37

xsyr opened this issue Feb 17, 2014 · 3 comments
Milestone

Comments

@xsyr
Copy link

xsyr commented Feb 17, 2014

$ pwd
/home/xsyr/workspace/cppprojects/ptp/build
$ gcovr -x -o report.xml -r ..            <------ It works fine.
$ gcovr -x -o report.xml -r /home/xsyr/workspace/cppprojects/ptp/
..... No response for a long time.
@mpontillo
Copy link

It looks to me like it's only taking the first character in the pathname. So it's probably scanning /. For example:

$ ../../external/python/gcovr-3.1/scripts/gcovr -x -r src -o tmp.xml
Traceback (most recent call last):
  File "../../external/python/gcovr-3.1/scripts/gcovr", line 1828, in <module>
    datafiles = get_datafiles(options.root, options)
  File "../../external/python/gcovr-3.1/scripts/gcovr", line 377, in get_datafiles
    files = search_file(".*\.gc(da|no)$", dir)
  File "../../external/python/gcovr-3.1/scripts/gcovr", line 356, in search_file
    raise IOError("Unknown directory '"+path+"'")
IOError: Unknown directory 's'
$ ../../external/python/gcovr-3.1/scripts/gcovr -x -r xen -o tmp.xml
Traceback (most recent call last):
  File "../../external/python/gcovr-3.1/scripts/gcovr", line 1828, in <module>
    datafiles = get_datafiles(options.root, options)
  File "../../external/python/gcovr-3.1/scripts/gcovr", line 377, in get_datafiles
    files = search_file(".*\.gc(da|no)$", dir)
  File "../../external/python/gcovr-3.1/scripts/gcovr", line 356, in search_file
    raise IOError("Unknown directory '"+path+"'")
IOError: Unknown directory 'x'

This is with the 3.1 release (if it wasn't clear from the pathname), by the way.

@s7726
Copy link

s7726 commented Apr 21, 2014

This seems to fix the issue, any hope of getting it pulled in?

@whart222
Copy link
Member

whart222 commented Jul 4, 2014

This is now fixed.

@whart222 whart222 closed this as completed Jul 4, 2014
@whart222 whart222 added this to the Gcovr 3.2 milestone Jul 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants