sbt / sbt/sbt-eclipse

Xmax-classfile-name should be put in scala.compiler.additionalParams

Open
#210 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Scala
Stars
712
Forks
163
PR merge metrics
No merged PRs in 30d

Description

As I see in current (3.0.3) and upcoming (4.0.0 M2) versions of scala-ide plugin for eclipse there is no "Xmax-classfile-name" option in Preferences > Scala > Compilier. In that case I think this option should go to scala.compiler.additionalParams in org.scala-ide.sdt.core.prefs file not as separate option as it is now. I test that when it is as separate option scala-ide ignores it. It works when it is added to scala.compiler.additionalParams.

I think the change should be done in com.typesafe.sbteclipse.core.EclipseSDTConfig.scala: remove line "TakesArg("Xmax-classfile-name"),"

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Open com.typesafe.sbteclipse.core.EclipseSDTConfig.scala and inspect the TakesArg("Xmax-classfile-name") entry. Remove that separate option and verify that Xmax-classfile-name is emitted through scala.compiler.additionalParams in org.scala-ide.sdt.core.prefs, as described in the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
tooling
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.