Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Set publish project directory to .publish #180

Closed
wants to merge 1 commit into from
Closed

Set publish project directory to .publish #180

wants to merge 1 commit into from

Conversation

eatkins
Copy link

@eatkins eatkins commented Sep 3, 2020

It is undefined behavior in sbt to have multiple projects that have the
same path. Both the root and publish projects have were defined to be in
the base directory (file(".")). With older versions of sbt, only the
publish project would appear when running the projects command. With
sbt 1.4.0-M2, only root appears when running the projects command.
After applying this change, both root and publish appear.

Copy link
Member

@cb372 cb372 left a comment

Choose a reason for hiding this comment

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

Thanks!

@cb372
Copy link
Member

cb372 commented Sep 4, 2020

I'm pretty confused but it looks like this somehow fixes the CI so it's started trying to run ScalaJS compilation tasks that weren't running before. That's why the build is failing. I'll hold off on merging while I investigate.

@eatkins
Copy link
Author

eatkins commented Sep 4, 2020

Aah, cool. I was wondering about that. It is reproducible on my machine but I just assumed it was a problem with the internet.

@andyscott
Copy link
Member

#181 has been merged, so perhaps we can merge master into this branch and CI will pass?

Can you give that a go @eatkins?

It is undefined behavior in sbt to have multiple projects that have the
same path. Both the root and publish projects have were defined to be in
the base directory (file(".")). With older versions of sbt, only the
publish project would appear when running the `projects` command. With
sbt 1.4.0-M2, only root appears when running the `projects` command.
After applying this change, both root and publish appear.
@eatkins
Copy link
Author

eatkins commented Sep 4, 2020

I went ahead and just rebased the latest master.

cb372 added a commit that referenced this pull request Sep 7, 2020
Also:
* fix the `publish` project's base directory (thanks @eatkins, see #180 for more details)
* set `crossScalaVersions` on the root project to work around an sbt bug
cb372 added a commit that referenced this pull request Sep 7, 2020
Also:
* fix the `publish` project's base directory (thanks @eatkins, see #180 for more details)
* set `crossScalaVersions` on the root project to work around an sbt bug
cb372 added a commit that referenced this pull request Sep 7, 2020
Also:
* fix the `publish` project's base directory (thanks @eatkins, see #180 for more details)
* set `crossScalaVersions` on the root project to work around an sbt bug
@cb372
Copy link
Member

cb372 commented Sep 7, 2020

Closing as I folded your change into #182

@cb372 cb372 closed this Sep 7, 2020
@eatkins
Copy link
Author

eatkins commented Sep 7, 2020

👍

@eatkins eatkins deleted the publish-directory branch September 7, 2020 18:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants