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

crypt3 and windows :/ #119

Closed
ciaranj opened this issue Sep 15, 2014 · 3 comments
Closed

crypt3 and windows :/ #119

ciaranj opened this issue Sep 15, 2014 · 3 comments

Comments

@ciaranj
Copy link

ciaranj commented Sep 15, 2014

Now it is bundled and no longer an optional dep, I can't seem to install sinopia (as crypt fails to compile for me.) Previously this appeared to be a non-issue, now it appears to be blocking me, is that as intended? (i.e. should I spend the time figuring out how to get crypt3 to build now)

(As a workaround I removed the entry from the package.yaml locally)

@rlidwka
Copy link
Owner

rlidwka commented Sep 15, 2014

Now it is bundled and no longer an optional dep

Oops, that's my mistake. All native modules should be optional.

@rlidwka
Copy link
Owner

rlidwka commented Sep 15, 2014

I moved crypt3 out to sinopia-htpasswd, and brought it back later, because I wasn't sure how nested bundled modules work. But I placed it to the wrong place in package.yaml.

Please check if it works in sinopia@0.11.1. It should be bundled, but considered optional.

@ciaranj
Copy link
Author

ciaranj commented Sep 15, 2014

:/ I rolled back to a 'known position' I'm afraid ( https://github.com/ciaranj/sinopia/tree/fix_for_67 ), I still get random quits when pulling fresh packages on the current release, I'll try to check again tomorrow!

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