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

Add ruby lang version for each jruby download #10

Closed
wants to merge 1 commit into from
Closed

Add ruby lang version for each jruby download #10

wants to merge 1 commit into from

Conversation

evanjbowling
Copy link

This is an addition that I would like added to the downloads page. I had a brief conversation on the IRC chat that precipitated this change. I've copied a snippet of that conversation here:

ebowling: there is a mapping but we do not have a table on a site (which might be a good idea) 1.7 -> (1.8 or 1.9), 9.1.x (2.3.y)
ebowling: 9.2.x will be 2.4 and 9.3 will be 2.5

I don't have any ruby lang version to add for jruby versions 0.9 - 1.6. Since they're older versions maybe that's ok.

</tr>
<tr>
<td><a href='/files/downloads/9.0.0.0.pre1/index.html'>9.0.0.0.pre1</a></td>
<td>1.8 or 1.9</td>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9.0 wasn't 1.8/1.9 compatible - believe it started with 2.2 compat

</tr>
<tr>
<td><a href='/files/downloads/1.7.9/index.html'>1.7.9</a></td>
<td>1.8 or 1.9</td>
Copy link
Member

@kares kares Feb 4, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

while its better than previous - its a bit unclear due how 1.8/1.9 were versioned.
maybe it should say 1.8.7/1.9.3 compatible - starting 1.7.0 it was 1.9.3 (while 1.6.8 was 1.9.2 - not sure)

</tr>
<tr>
<td><a href='/files/downloads/1.7.0.RC1/index.html'>1.7.0.RC1</a></td>
<td>1.8 or 1.9</td>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say the table would be easier to the eye if the rows with the same value are combined with the rowspan attribute.

<td></td>
</tr>
<tr>
<td><a href='/files/downloads/1.5.0/index.html'>1.5.0</a></td>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I seem to remember 1.5 being the first to add MRI 1.9 compatibility in a serious way. I could be mistaken, though.

<tbody>
<tr>
<td><a href='/files/downloads/0.9.0/index.html'>0.9.0</a></td>
<td></td>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The lack of information here and subsequent rows should be addressed somehow—by having a separate table with no compatibility information, by completing the trade arch, or by some other means.

@evanjbowling
Copy link
Author

Thanks for the feedback everyone - I'm happy to make these changes and resubmit, but I need some help first.

I'm currently unable to build and run the server locally. I followed the directions on the README.md and even made a Dockerfile in a branch on my forked version to try and create a reproducible environment. Here are my findings:

$ rake generate

# [rake help ommitted]
rake aborted!
Command failed with status (1): [/usr/local/bin/ruby -S bundle exec jekyll...]
/src/rakefile:15:in `block in <top (required)>'
/usr/local/bundle/gems/rake-12.2.1/exe/rake:27:in `<top (required)>'
Tasks: TOP => generate
(See full trace by running task with --trace)

After this error I just tried running the server:

$ rake server

This encounters another exception listed below:

/usr/local/bin/ruby -S bundle exec rackup
Configuration file: /src/_config.yml
       Deprecation: Auto-regeneration can no longer be set from your configuration file(s). Use the --[no-]watch/-w command-line option instead.
       Deprecation: The 'server_port' configuration option has been renamed to 'port'. Please update your config file accordingly.
Auto-regenerating enabled: /src -> ./_site
Server running on http://localhost:9292
[2018-02-10 21:04:53] regeneration: 4023 files changed
[2018-02-10 21:04:53] INFO  WEBrick 1.3.1
[2018-02-10 21:04:53] INFO  ruby 2.2.9 (2017-12-15) [x86_64-linux]
[2018-02-10 21:04:53] INFO  WEBrick::HTTPServer#start: pid=12794 port=9292
     Build Warning: Layout 'nil' requested in atom.xml does not exist.
    Liquid Warning: Liquid syntax error (line 24): Expected end_of_string but found close_square in "{{site.posts.first] }}" in /_layouts/home.html

I tried accessing the site with: http://localhost:9292/ but got:

curl: (52) Empty reply from server

Any thoughts? I'll keep digging into the rakefile and jekyll when I get a chance.

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

Successfully merging this pull request may close these issues.

None yet

3 participants