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

CLI: -mode=line reports bad results #215

Closed
EgorBu opened this issue Apr 4, 2019 · 2 comments · Fixed by #214
Closed

CLI: -mode=line reports bad results #215

EgorBu opened this issue Apr 4, 2019 · 2 comments · Fixed by #214
Assignees
Labels
Milestone

Comments

@EgorBu
Copy link

EgorBu commented Apr 4, 2019

Hi,
I tried to use enry and got unexpected results:

name@pc:~$ ./enry -mode=line ../some/path/
open cmd.txt: no such file or directory
open feature_extractor.py: no such file or directory
open mutator.py: no such file or directory
open tokenizer/__init__.py: no such file or directory
open tokenizer/langs/__init__.py: no such file or directory
open tokenizer/langs/javascript/__init__.py: no such file or directory
open tokenizer/langs/javascript/roles.py: no such file or directory
open tokenizer/langs/javascript/token_unwrappers.py: no such file or directory
open tokenizer/langs/javascript/tokens.py: no such file or directory
open tokenizer/langs/javascript/uast_fixers.py: no such file or directory
open tokenizer/tokenizer.py: no such file or directory
open tokenizer/virtual_node.py: no such file or directory
NaN%	Text
NaN%	Python

Version v1.7.2

@bzz bzz self-assigned this Apr 4, 2019
@bzz bzz added the bug label Apr 4, 2019
@bzz bzz changed the title [bug] mode=line gives bad results CLI: -mode=line reports bad results Apr 4, 2019
@bzz
Copy link
Contributor

bzz commented Apr 4, 2019

@EgorBu thank you for a report!

Indeed, that is exactly what is supposed to be taken care of by the last TODO item in #214 (comment)

Funny enough, it does not happen all the time, e.g on clean clone of enry it somehow works

./enry -mode=byte ~/src-d/bblfsh/enry
76.61%	Go
22.85%	CSV
0.24%	Shell
0.17%	Java
0.06%	Ruby
0.04%	Makefile
0.03%	Scala
0.01%	Gnuplot
0.00%	Text

@bzz
Copy link
Contributor

bzz commented Apr 4, 2019

it should be fixed by #214 (comment)

@bzz bzz added this to the v1.7.3 milestone Apr 8, 2019
@bzz bzz closed this as completed in #214 Apr 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants