Skip to content

Fix support for parsing of classes located in runtime jars#2268

Merged
eed3si9n merged 1 commit intosbt:0.13from
twitter-forks:stuhood/fix-system-inner-classes
Nov 11, 2015
Merged

Fix support for parsing of classes located in runtime jars#2268
eed3si9n merged 1 commit intosbt:0.13from
twitter-forks:stuhood/fix-system-inner-classes

Conversation

@stuhood
Copy link

@stuhood stuhood commented Nov 11, 2015

No description provided.

@typesafe-tools
Copy link

Can one of the admins verify this patch?

@eed3si9n
Copy link
Member

Fixes #2263?

@eed3si9n
Copy link
Member

Coincidently I haven't merged months old sbt/io/pull/13 yet. Could you update that PR too?

eed3si9n added a commit that referenced this pull request Nov 11, 2015
…classes

Fix support for parsing of classes located in runtime jars
@eed3si9n eed3si9n merged commit be9f1cd into sbt:0.13 Nov 11, 2015
@eed3si9n eed3si9n removed the ready label Nov 11, 2015
Copy link
Member

Choose a reason for hiding this comment

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

Bincompat?

Copy link
Member

Choose a reason for hiding this comment

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

@eed3si9n Do we want to do something about this binary compatibility breakage before cutting a 0.13.10 release?

Copy link
Member

Choose a reason for hiding this comment

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

Yes. Could you send a PR or issue against 0.13 to fix this?

Copy link
Member

Choose a reason for hiding this comment

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

sure

@dwijnand
Copy link
Member

Is this a fix to an improvement for which we have notes or is it a fix that requires its own notes?

@eed3si9n
Copy link
Member

@dwijnand I think this is an improvement to #2214, which is a part of #2085, which does have a note entry.

@dwijnand
Copy link
Member

Cool thanks.

@dwijnand
Copy link
Member

Also, this hasn't been forward-ported to 1.0, which I realise is a little hard as it's touching files in different modules..

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.

4 participants