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

Switch to sass-embedded #466

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ntkme
Copy link

@ntkme ntkme commented Mar 18, 2024

This PR switches to use sass-embedded (dart-sass) to compile scss stylesheets.

@slonopotamus
Copy link
Contributor

I highly expect issues on JRuby.

@mojavelinux
Copy link
Member

It seems to be installable on JRuby. However, it does require compiling native extensions, so there could be additional system requirements.

@ntkme
Copy link
Author

ntkme commented Mar 19, 2024

The sass-embedded gem supports MRI, JRuby, and TruffleRuby. The "native extension" is a dart runtime binary. For JRuby it use a Rakefile extension to install the correct dart executable for the running platform. The main limitation is the platforms that dart-sdk supports. It should cover the vast majority of users, but there are systems that not supported, for example, BSD is not supported. The following are the currently supported platforms:

  • arm64-darwin
  • x86_64-darwin
  • aarch64-linux-android
  • arm-linux-androideabi
  • x86-linux-android
  • x86_64-linux-android
  • aarch64-linux-gnu
  • arm-linux-gnueabihf
  • x86-linux-gnu
  • x86_64-linux-gnu
  • arm-linux-musleabihf
  • aarch64-linux-musl
  • x86-linux-musl
  • x86_64-linux-musl
  • x64-mingw-ucrt
  • x64-mingw32
  • x86-mingw32

@slonopotamus
Copy link
Contributor

The issue is additionally complicated by the way how things are packaged in asciidoctorj-epub3. Need to think a bit.

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