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

instructions, source highlighting, edits #1

Merged
merged 1 commit into from
May 9, 2013
Merged

instructions, source highlighting, edits #1

merged 1 commit into from
May 9, 2013

Conversation

mojavelinux
Copy link
Contributor

  • add basic instructions for running the application
  • enable source highlighting using CodeRay
  • add styles from default CodeRay theme
  • fix spelling of Markdown
  • fix case of AsciiDoc
  • use fast erb impl (erubis)

- add basic instructions for running the application
- enable source highlighting using CodeRay
- add styles from default CodeRay theme
- fix spelling of Markdown
- fix case of AsciiDoc
- use fast erb impl (erubis)
@rmannibucau
Copy link
Owner

Hi Dan,

When using erubis i get warns in the log. Any idea why?

@mojavelinux
Copy link
Contributor Author

Yes, that actually made me aware of a feature we probably need in the Asciidoctor Java integration.

C Ruby is single-threaded, so it can be lazy about how it loads libraries. JRuby (and Rubinius) can run in multi-threaded environments, so certain assumptions have to be revisited. One of those assumptions is how libraries are loaded. Since there's no way for the implementation to know what the library might try to do when it's loaded, a warning is issued to make you aware of the fact that things could go wrong if the library is mutating state or other issues of concurrency.

In this case, the libraries we're loading lazily (erubis and coderay) aren't modifying mutable state, just making new utilities available to the system, so the warning is superfluous. However, it would still be nice if the Asciidoctor Java integration could preload specified libraries when it boots JRuby, removing the possibility of any problems or warnings.

cc: @lordofthejars

@lordofthejars
Copy link

Created issue #32

rmannibucau added a commit that referenced this pull request May 9, 2013
instructions, source highlighting, edits
@rmannibucau rmannibucau merged commit 504394c into rmannibucau:master May 9, 2013
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