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

Get rid of or rename the "C" module into something more meaningful. #67

Open
ericzolf opened this issue Aug 14, 2019 · 1 comment
Open
Milestone

Comments

@ericzolf
Copy link
Member

Issue by ericzolf
Sunday Apr 14, 2019 at 12:01 GMT
Originally opened as ericzolf#9


C disturbs me as a module name, and anyway I'm not sure this is still required with modern hardware, as the only reason for it seems to be performance.

@dbaarda
Copy link

dbaarda commented Aug 3, 2020

I haven't looked closely at how you are using this, but perhaps you could just exec the rdiff binary?

Though compiling the rdiff binary on windows has it's own issues, as it needs popt, and I'm not sure how easy it is to get/install popt on windows... I think you need cygwin for that.

I've been considering switching to use some other alternative to popt that is more widely available like gengetopt to fix this, but haven't done anything on it yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants