Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove conscriptConfigs task, not used and needed(?) anymore #7353

Merged
merged 1 commit into from Aug 12, 2023

Conversation

mkurz
Copy link
Member

@mkurz mkurz commented Aug 11, 2023

The task was part of the release process, but is not used anymore since the release of sbt 1.5.1: 43b2f1e#diff-5634c415cd8c8504fdb973a3ed092300b43c4b8fc1e184f7249eb29a55511f91L1525

Also, the conscript launchconfig files already diverged from the source sbt.boot.properties since it does not contain the local-preloaded[-ivy] repos anymore: 181052f, but the conscript launchconfigs do: https://github.com/sbt/sbt/tree/1.9.x/src/main/conscript
It seems those config were and get updated by hand anyway.

Since the task is not used in the release process anymore, I think it's more confusing to keep it (or @eed3si9n did run that task in the last years manually?).
(I stumpled over this because I was looking where/how the default repos are set up (which is https://github.com/sbt/sbt/blob/v1.9.3/launch/src/main/input_resources/sbt/sbt.boot.properties#L13-L19).

Copy link
Member

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

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

Yea. I think we can remove it.

@eed3si9n eed3si9n merged commit 8857ff2 into sbt:1.9.x Aug 12, 2023
10 checks passed
@mkurz mkurz deleted the rm_conscriptConfigs branch August 12, 2023 06:14
@mkurz
Copy link
Member Author

mkurz commented Aug 12, 2023

BTW, all the conscript launchconfigs have 1.3.0 set as version, shouldn't that be upgraded on each release:

version: ${sbt.version-read(sbt.version)[1.3.0]}

(Personally I don't care because I don't use that conscripts, just wanted to let you know)

cwienberg pushed a commit to cwienberg/spark-sorting-helpers that referenced this pull request Aug 25, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [sbt/sbt](https://togithub.com/sbt/sbt) | patch | `1.9.3` -> `1.9.4` |

---

### Release Notes

<details>
<summary>sbt/sbt (sbt/sbt)</summary>

### [`v1.9.4`](https://togithub.com/sbt/sbt/releases/tag/v1.9.4): 1.9.4

[Compare Source](https://togithub.com/sbt/sbt/compare/v1.9.3...v1.9.4)

##### CVE-2022-46751

[CVE-2022-46751](https://togithub.com/advisories/GHSA-2jc4-r94c-rp7h) is
a security vulnerability discovered in Apache Ivy, but found also in
Coursier.

With coordination with Apache Foundation, Adrien Piquerez
([@&#8203;adpi2](https://togithub.com/adpi2)) from Scala Center
backported the fix to both our Ivy 2.3 fork and Coursier. sbt 1.9.4
updates them to the fixed versions.

#### Other updates

- Fixes `sbt_script` lookup by replacing all spaces with `%20` (not only
the first one) in the path. by
[@&#8203;arturaz](https://togithub.com/arturaz) in
[sbt/sbt#7349
- Fixes
[scala-debug-adapter#543](https://togithub.com/scala-debug-adapter/sbt/issues/543):
Maintain order of internal deps by
[@&#8203;adpi2](https://togithub.com/adpi2) in
[sbt/sbt#7347
- Removes `conscriptConfigs` task, not used and needed(?) anymore by
[@&#8203;mkurz](https://togithub.com/mkurz) in
[sbt/sbt#7353
- Adds a Scala 3 seed to the `sbt new` menu by
[@&#8203;SethTisue](https://togithub.com/SethTisue) in
[sbt/sbt#7354

#### new contributors

- [@&#8203;arturaz](https://togithub.com/arturaz) made their first
contribution in
[sbt/sbt#7349

**Full Changelog**: sbt/sbt@v1.9.3...v1.9.4

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/cwienberg/spark-sorting-helpers).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzYuNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

None yet

2 participants