Skip to content

Conversation

@sjrd
Copy link
Member

@sjrd sjrd commented Dec 3, 2017

Compared to the code from the core repository, the sbt build is in charge of more operations than was assemble-cli.sh. In particular, it has taken over the tasks of:

  • finding the jars to include in lib/ (most of them are now downloaded through ivy resolution, which a .sh can hardly do),
  • processing the shell scripts to replace the version numbers, and
  • laying out the contents of the archives.

The script assemble-cli.sh is only responsible of invoking the cliPack task in sbt, and create the zip and tgz archives.

@sjrd sjrd requested a review from gzm0 December 3, 2017 15:22
@sjrd
Copy link
Member Author

sjrd commented Dec 10, 2017

Ping @gzm0?

build.sbt Outdated
val scalaJSScalaVersions =
settingKey[Seq[String]]("All the minor versions of Scala for which to build the CLI")

// Derived settings
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe "computed" settings? Derived actually means something in sbt land :-/

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point. I'll change it.

Compared to the code from the core repository, the sbt build is in
charge of more operations than was `assemble-cli.sh`. In
particular, it has taken over the tasks of:

* finding the jars to include in `lib/` (most of them are now
  downloaded through ivy resolution, which a .sh can hardly do),
* processing the shell scripts to replace the version numbers, and
* laying out the contents of the archives.

The script `assemble-cli.sh` is only responsible of invoking the
`cliPack` task in sbt, and create the zip and tgz archives.
@sjrd sjrd merged commit 4abe1a1 into scala-js:master Dec 12, 2017
@sjrd sjrd deleted the initial-implem branch December 12, 2017 10:01
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.

2 participants