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

fix scala.tools.nsc.io #6305

Closed
scabug opened this issue Sep 1, 2012 · 8 comments
Closed

fix scala.tools.nsc.io #6305

scabug opened this issue Sep 1, 2012 · 8 comments
Assignees
Milestone

Comments

@scabug
Copy link

@scabug scabug commented Sep 1, 2012

Currently the scala.tools.nsc.io package is split up between scala-reflect.jar and scala-compiler.jar. That hurts the build process and OSGi. Discussion: http://groups.google.com/group/scala-internals/browse_thread/thread/6eefdb5471f13a4d

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 1, 2012

Imported From: https://issues.scala-lang.org/browse/SI-6305?orig=1
Reporter: @xeno-by
Affected Versions: 2.10.0
Other Milestones: 2.10.0

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 5, 2012

@jsuereth said:
Just a note: This is a blocker for 2.10.0-RC1. I can jump in and help if you need.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 5, 2012

@xeno-by said:
I would love if you could do that.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 17, 2012

@gkossakowski said:
Fix merged here: scala/scala#1291

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 17, 2012

@gkossakowski said:
I closed the wrong issue because title of scala/scala#1291 was wrong. This should be still open.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 18, 2012

@jsuereth said:
Note: This isn't as simple as moving all of IO into src/reflect. Some dependencies on the nsc.util package from io bits in the src/compiler/

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 18, 2012

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Sep 20, 2012

@xeno-by said:
Fixed in scala/scala@c6dde6a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.