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

Updates and remove redundant settings #101

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

Conversation

keynmol
Copy link

@keynmol keynmol commented Feb 13, 2025

  1. Update versions (scala 3 lts, native version)
  2. Remove redundant native config settings – not only were the comments and values out of sync, but I think it's best to provide a minimal config relying on defaults.
  3. Enable source level debugging – for nicer stacktraces and debug information.
  4. Use Scala 3 syntax

@@ -10,7 +10,7 @@ import scala.scalanative.build._

// defaults set with common options shown
nativeConfig ~= { c =>
c.withLTO(LTO.none) // thin
Copy link
Member

Choose a reason for hiding this comment

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

These were here to help people with these basic settings (using the defaults) with some suggestions in the comments for settings you might want to select - maybe the comments where not explained well but that was deliberate as the first thing people asked for was how to optimize and such.

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.

2 participants