-
Notifications
You must be signed in to change notification settings - Fork 587
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
Solus / Gentoo / RHEL Instructions! OMG, It's gonna blow! #968
Conversation
Not that much documentation on the web on Solus. Glad I could add!
Is python included? |
@Kubuxu It's in there. We need to spread Cjdns to all corners of the operating system universe. I think I'm doing a good job. |
added some failsafe packages in case your running a weekly build (I don't know who does this. Developers, Perhaps??) |
Why hasn't this merged! |
Hello. The graphic may be overkill. NodeJS may also not be required (because cjdns may download it) so you will want to test the build without NodeJS installed. Last, the instructions for Solus are incomplete. We do not (currently) have Solus builders. Hope this helps. Looking forward to another supported OS, cheers. |
Thanks so much @wfleurant! Overkill is my middle name. |
@wfleurant I tested on a few versions. CJDNS will fail if I don't have |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please set the file modes back to 644 because as they are, the files are set executable which is not correct. Also beware of making any changes in map without testing them...
@@ -44,7 +44,7 @@ var getData = function() | |||
var log2 = function(number) | |||
{ | |||
var out = 0; | |||
while ((number >>= 1) > 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is meant to be that way, your "typo fix" introduced a typo :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do'h! I tested it and I just got a warning for "invalid statement on line 47"
We also got some crud in /contrib/