GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Update installation; OpenSSL needs Root install by BenjaminHCCarr · Pull Request #40 · IRkernel/irkernel.github.io

Conversation

@BenjaminHCCarr

@BenjaminHCCarr BenjaminHCCarr commented Mar 19, 2017

Copy link
Copy Markdown

It took me a few runs to figure out why dependancies were failing at least on OSX, following the instructions, It took a bunch of log checking to figure out that openssl was failing, but was not obvious, as it would error on a different package that was an intermediary.

OpenSSL couldn't built without root because of the out-of-R tree it required.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@BenjaminHCCarr
Update installation; OpenSSL needs Root install
04bc5f6 
It took me a few runs to figure out why dependancies were failing at least on OSX, following the instructions, It took a bunch of log checking to figure out that openssl was failing, but was not obvious, as it would error on a different package that was an intermediary.

OpenSSL couldn't built without root because of the out-of-R tree it required.
@flying-sheep

flying-sheep commented Mar 21, 2017

Copy link
Copy Markdown
Member

hi, thanks for the info.

but this apparently hasn’t been true for me, so it’s probably platform specific.

maybe you could instead add the information into the <aside/> here? (because that’s already about an OSX quirk)
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@BenjaminHCCarr
Update SSL on OSX warning to an <aside>
4e739b9 
Changed the OSX OpenSSL instructions to an aside.
May possibly need moving out of current block, but I believe it should be active here because of the tabs stanza.
@BenjaminHCCarr

BenjaminHCCarr commented Mar 21, 2017

Copy link
Copy Markdown
Author

@flying-sheep on linux it wouldn't apply, as it appears not to on windows. Maybe OSX and windows should be separated?
similar to: https://irkernel.github.io/installation/#source-panel
if the instructions are platform dependent as well

The Source title is a little misleading, as apt-get and yum are installing binary files, not pulling the raw source and compiling most of the code. It's just some aspects of the Jupyter code that are being recompiled.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
flying-sheep
flying-sheep requested changes Mar 22, 2017
View reviewed changes

@flying-sheep flying-sheep left a comment •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

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.

could you please also show how it looks after the changes in a screenshot?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread installation/index.html
</div>
<div class="mdl-tabs__panel is-active" id=binary-panel>
<aside class="card-note mdl-card mdl-shadow--2dp">
<div class="mdl-card__title">

@flying-sheep flying-sheep Mar 22, 2017

Copy link
Copy Markdown
Member

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 leave out quotes when there’s only one class

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread installation/index.html
<pre class="r mdl-shadow--2dp"><code>>
user$ sudo R
> install.packages(c('openssl')
> quit()</code></pre>

@flying-sheep flying-sheep Mar 22, 2017

Copy link
Copy Markdown
Member

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 don’t add quit() and how to start an R interpreter. people know how to do that 😉

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@flying-sheep

flying-sheep commented Mar 22, 2017

Copy link
Copy Markdown
Member

about your concerns for platform-specific instructions:

maybe we should just switch to a UI where people select their platform and if the want a non-standard install (e.g. dev versions), and only then show instructions
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@BenjaminHCCarr BenjaminHCCarr closed this Jun 7, 2017
@flying-sheep

flying-sheep commented Jun 7, 2017

Copy link
Copy Markdown
Member

hi, why did you close this? it just needed a few changes to be merged
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@BenjaminHCCarr

BenjaminHCCarr commented Jun 7, 2017

Copy link
Copy Markdown
Author

@flying-sheep oh, I was trying to clean up my laundry list of PR's, and didn't realize it was still of interest. But if it is still relevant, I will re-open, I had a huge project sent out and was trying to id things of interest still to upstream.

reopen
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@BenjaminHCCarr BenjaminHCCarr reopened this Jun 7, 2017
@flying-sheep

flying-sheep commented Jun 9, 2017

Copy link
Copy Markdown
Member

thank you! would you like to do the requested changes or should i just close this and make the edits manually?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@flying-sheep flying-sheep flying-sheep requested changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@BenjaminHCCarr @flying-sheep
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.