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

Setting scalacOptions twice causing warnings #525

Closed
ckipp01 opened this issue Jan 4, 2022 · 0 comments · Fixed by #536
Closed

Setting scalacOptions twice causing warnings #525

ckipp01 opened this issue Jan 4, 2022 · 0 comments · Fixed by #536
Assignees
Labels
bug Something isn't working

Comments

@ckipp01
Copy link
Contributor

ckipp01 commented Jan 4, 2022

Give an minimal snippet like this:

// using scala 3.1.0

def foo = "bar"

Do a compile and get no warning. Check the .scala/.bloop/<build-target>.json and see that there are no options containing -feature, -deprecation, or -explain.

Change you script to:

// using scala 3.1.0
// using options -deprecation -feature -explain

def foo = "bar"

Do a scala-cli clean . just to be safe and then follow it up with a scala-cli compile . and you see the following:

Compiling project (Scala 3.1.0, JVM)
Warning: Flag -deprecation set repeatedly
Warning: Flag -feature set repeatedly
Warning: Flag -explain set repeatedly

If you now look in the bloop json file again you see that they are duplicated in options. I'd expect this not to happen so the user doesn't see these errors.

@lwronski lwronski added the bug Something isn't working label Jan 5, 2022
@jchyb jchyb self-assigned this Jan 7, 2022
jchyb added a commit to jchyb/scala-cli that referenced this issue Jan 10, 2022
jchyb added a commit to jchyb/scala-cli that referenced this issue Jan 10, 2022
@jchyb jchyb linked a pull request Jan 10, 2022 that will close this issue
alexarchambault added a commit that referenced this issue Jan 11, 2022
Co-authored-by: Alexandre Archambault <alexandre.archambault@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants