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

update documentation with cffi '\r' error #1115

Closed
ctb opened this issue Jul 20, 2020 · 3 comments
Closed

update documentation with cffi '\r' error #1115

ctb opened this issue Jul 20, 2020 · 3 comments

Comments

@ctb
Copy link
Contributor

ctb commented Jul 20, 2020

if you get a cffi error around '\r', you might need to make autocrlf false, per these docs

This error can be reproduced by some using this set of commands:

put

[core]
    autocrlf = true

in your .gitconfig on a linux box, then:

%  conda create -n sourmash_dev python=3.7 pip rust
%  conda activate sourmash_dev
% git clone https://github.com/dib-lab/sourmash.git
% cd sourmash
% pip install -e .
...
     File "/home/ctbrown/miniconda3/envs/sourmash_dev/lib/python3.7/site-packages/cffi/cparser.py", line 336, in convert_pycparser_error
        raise CDefError(msg)
    cffi.CDefError: cannot parse ""
    <cdef source string>:1:2: Illegal character '\r'
    ----------------------------------------
ERROR: Command errored out with exit status 1: /home/ctbrown/miniconda3/envs/sourmash_dev/bin/python3.7 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/sourmash_foo/sourmash/setup.py'"'"'; __file__='"'"'/tmp/sourmash_foo/sourmash/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' develop --no-deps Check the logs for full command output.

I think this should also be fixed in cffi...

@ctb
Copy link
Contributor Author

ctb commented Jul 20, 2020

this looks like the home of the cffi project? https://foss.heptapod.net/pypy/cffi

Copy link
Member

yup

@ctb
Copy link
Contributor Author

ctb commented May 4, 2022

this seems to no longer happen :)

@ctb ctb closed this as completed May 4, 2022
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

No branches or pull requests

2 participants