Skip to content

Fix #2411#2418

Merged
eed3si9n merged 4 commits intosbt:0.13from
Duhemm:fix-2411
Jan 24, 2016
Merged

Fix #2411#2418
eed3si9n merged 4 commits intosbt:0.13from
Duhemm:fix-2411

Conversation

@Duhemm
Copy link
Contributor

@Duhemm Duhemm commented Jan 24, 2016

This PR fixes the breakage introduced in #2261.

The fix comes from @adriaanm who suggested to use transformedType rather exitingPostErasure to inspect types after erasure in scala/scala@6dd1495.

I can confirm that with this PR applied, these builds now pass:

https://travis-ci.org/xuwei-k/Play20/jobs/104261656#L497
https://travis-ci.org/seratch/skinny-framework/jobs/104275385#L2107
https://travis-ci.org/xuwei-k/cats/jobs/104294044#L2702

Fixes #2411

eed3si9n and others added 4 commits January 24, 2016 08:56
... not exitingPostErasure, as this phase-travel crashes the compile
(it's only really meant for going back in time, right?)
@typesafe-tools
Copy link

Can one of the admins verify this patch?

@eed3si9n
Copy link
Member

Fix should target 0.13.10 branch. In this case we should be able to automatically merge, but 0.13 can move ahead.

@eed3si9n
Copy link
Member

eed3si9n commented May 1, 2016

Should this be forward ported?

@eed3si9n
Copy link
Member

eed3si9n commented May 2, 2016

I guess this is superseded by sbt/zinc#95

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