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

[BSP] buildTarget/javacOptions not found #1091

Closed
Iltotore opened this issue Jan 1, 2021 · 7 comments
Closed

[BSP] buildTarget/javacOptions not found #1091

Iltotore opened this issue Jan 1, 2021 · 7 comments
Milestone

Comments

@Iltotore
Copy link
Contributor

Iltotore commented Jan 1, 2021

Hello, my issue is similar to #1024. It should be fixed in #1025 but the problem sill occurs in 0.9.4 on Windows 10.

@lefou
Copy link
Member

lefou commented Jan 2, 2021

Can you explain, how to reproduce this? Which BSP client? What steps?

@Iltotore
Copy link
Contributor Author

Iltotore commented Jan 2, 2021

I just run mill mill.bsp.BSP/Install, and created in my buildscript an empty main module in 2.13.4. When I import the project in idea using BSP, it hangs on javacOptions, so I tried using the Bloop contrib module and I got the same issue as mentionned in #1024

@lefou
Copy link
Member

lefou commented Nov 7, 2021

Should be fixed in main as part of my refactoring #1536.

@Iltotore Can you verify it is fixed in main?

@lefou lefou closed this as completed Nov 7, 2021
@lefou lefou added this to the after 0.10.0-M3 milestone Nov 7, 2021
@Iltotore
Copy link
Contributor Author

Iltotore commented Nov 8, 2021

@lefou I checked it using the last main commit (e972e1). Your refactoring works fine using my old BSP mill.json configuration:

{
  "name": "mill-bsp",
  "argv": [
    "C:\\Users\\rapha\\scoop\\apps\\mill\\current\\mill.bat",
    "-i",
    "--disable-ticker",
    "--color",
    "false",
    "mill.bsp.BSP/start"
  ],
  "millVersion": "0.9.9",
  "bspVersion": "2.0.0",
  "languages": [
    "scala",
    "java"
  ]
}

But it doesn't using the new mill-bsp.json configuration generated using mill.bsp.BSP/install:

{
  "name": "mill-bsp",
  "argv": [
    "C:\\Users\\rapha\\scoop\\apps\\mill\\current\\mill.bat",
    "--bsp",
    "--disable-ticker",
    "--color",
    "false",
    "--jobs",
    "1"
  ],
  "millVersion": "0.10.0-M3-114-e972e1",
  "bspVersion": "2.0.0",
  "languages": [
    "scala",
    "java"
  ]
}

giving this error:

Exception in thread "Thread-2" java.lang.RuntimeException: java.io.IOException: WriteFile() failed: 233

	at mill.main.client.InputPumper.run(InputPumper.java:34)

	at java.lang.Thread.run(Thread.java:748)

Caused by: java.io.IOException: WriteFile() failed: 233

	at org.scalasbt.ipcsocket.Win32NamedPipeSocket$Win32NamedPipeSocketOutputStream.write(Win32NamedPipeSocket.java:198)

	at mill.main.client.InputPumper.run(InputPumper.java:28)

	... 1 more

@lefou
Copy link
Member

lefou commented Nov 8, 2021

Interesting. Thanks for checking!

This could be related to your "C:\\Users\\rapha\\scoop\\apps\\mill\\current\\mill.bat". Is this the mill-assembly but named as mill.bat or some other wrapper script? I probably forgot to adapt the Windows-batch-script-part of our assembly to recognize the --bsp switch.

@lefou
Copy link
Member

lefou commented Nov 8, 2021

Yes, it's a Windows-bug in mill. I opened a PR: #1557

@Iltotore
Copy link
Contributor Author

Iltotore commented Nov 8, 2021

It now works like a charm.

Is this the mill-assembly but named as mill.bat or some other wrapper script?

You probably know the answer now but yes, it was the mill assembly generated by the ci/publish-local.sh script

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

No branches or pull requests

2 participants