Skip to content

Publish compiler interface to an alternate local repository for Scripted tests#2169

Merged
eed3si9n merged 2 commits intosbt:0.13from
Duhemm:scripted-local-repo
Apr 5, 2016
Merged

Publish compiler interface to an alternate local repository for Scripted tests#2169
eed3si9n merged 2 commits intosbt:0.13from
Duhemm:scripted-local-repo

Conversation

@Duhemm
Copy link
Contributor

@Duhemm Duhemm commented Aug 21, 2015

Some scripted tests override the default local repository, which
produces errors with the compiler because sbt cannot find the sources
for the compiler interface.

This fix proposes to publish the compiler interface sources to an
alternate local repository before running the scripted tests. This
alternate repository is added to the scripted sbt's configuration, so
that sbt is finally able to find the compiler interface sources.

Fixes #2165

@eed3si9n eed3si9n added the ready label Aug 21, 2015
@typesafe-tools
Copy link

Can one of the admins verify this patch?

@dwijnand
Copy link
Member

In terms of this fixing the issue, I can confirm it worked for me in my #1750 scripted test.

@dwijnand
Copy link
Member

What do we want to do with this? /cc @eed3si9n @Duhemm

Duhemm added 2 commits April 5, 2016 17:33
Some scripted tests override the default local repository, which
produces errors with the compiler because sbt cannot find the sources
for the compiler interface.

This fix proposes to publish the compiler interface sources to an
alternate local repository before running the scripted tests. This
alternate repository is added to the scripted sbt's configuration, so
that sbt is finally able to find the compiler interface sources.
@Duhemm Duhemm force-pushed the scripted-local-repo branch from 058b524 to 5b0c9aa Compare April 5, 2016 15:34
@eed3si9n eed3si9n merged commit bc70ec0 into sbt:0.13 Apr 5, 2016
@eed3si9n eed3si9n removed the ready label Apr 5, 2016
@eed3si9n
Copy link
Member

eed3si9n commented Apr 5, 2016

@Duhemm Could you forward port this too?

@Duhemm Duhemm deleted the scripted-local-repo branch April 6, 2016 06:32
@eed3si9n eed3si9n mentioned this pull request May 1, 2016
7 tasks
@eed3si9n
Copy link
Member

eed3si9n commented May 1, 2016

Has this been forward ported?

@dwijnand
Copy link
Member

dwijnand commented Sep 6, 2016

Yes sbt/zinc#92

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