[PR PATCH] oils: renamed old names

[PR PATCH] oils: renamed old names

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 1238 bytes --]

There is a new pull request by bugcrazy against master on the void-packages repository

https://github.com/bugcrazy/void-packages patch-21
https://github.com/void-linux/void-packages/pull/51964

oils: renamed old names
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/51964.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-patch-21-51964.patch --]
[-- Type: text/x-diff, Size: 1404 bytes --]

From d4555c5dc8981ed5fb92e62c0efb88d7e04f2190 Mon Sep 17 00:00:00 2001
From: bugcrazy <marcoaureliopc@gmail.com>
Date: Fri, 23 Aug 2024 19:24:40 -0300
Subject: [PATCH] oils: renamed old names

---
 srcpkgs/{oil => oils}/template | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)
 rename srcpkgs/{oil => oils}/template (83%)

diff --git a/srcpkgs/oil/template b/srcpkgs/oils/template
similarity index 83%
rename from srcpkgs/oil/template
rename to srcpkgs/oils/template
index c1a9bd148a279d..252b161fa7fadc 100644
--- a/srcpkgs/oil/template
+++ b/srcpkgs/oils/template
@@ -1,5 +1,5 @@
-# Template file for 'oil'
-pkgname=oil
+# Template file for 'oils'
+pkgname=oils
 version=0.22.0
 revision=1
 build_style=configure
@@ -12,7 +12,8 @@ license="Apache-2.0"
 homepage="https://www.oilshell.org"
 distfiles="https://www.oilshell.org/download/oil-${version}.tar.xz"
 checksum=452e7fd428b686aa752cfeb9be2b94f9fcf7ba9ab22e0ae570a87cdcf782242e
-register_shell="/usr/bin/osh /usr/bin/oil"
+replaces="oil<=0.23.0"
+register_shell="/usr/bin/osh /usr/bin/ysh"
 nocross="Build systems gets confused with host and cross toolchains/headers"
 nostrip=yes
 
@@ -28,8 +29,8 @@ do_install() {
 	# Install binaries
 	vbin _bin/oil.ovm
 	vman doc/osh.1
-	# Symlink osh, oil to oil.ovm.
+	# Symlink osh, ysh to oil.ovm.
 	cd "${DESTDIR}"/usr/bin
 	ln -s oil.ovm osh
-	ln -s oil.ovm oil
+	ln -s oil.ovm ysh
 }

Re: [PR PATCH] [Updated] oils: renamed old names

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 1243 bytes --]

There is an updated pull request by bugcrazy against master on the void-packages repository

https://github.com/bugcrazy/void-packages patch-21
https://github.com/void-linux/void-packages/pull/51964

oils: renamed old names
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/51964.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-patch-21-51964.patch --]
[-- Type: text/x-diff, Size: 1632 bytes --]

From aec6387f8338f660dfabd478a0c30b3eafb86adc Mon Sep 17 00:00:00 2001
From: bugcrazy <marcoaureliopc@gmail.com>
Date: Sat, 24 Aug 2024 02:20:54 -0300
Subject: [PATCH] oils: renamed old names

---
 srcpkgs/{oil => oils}/template | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)
 rename srcpkgs/{oil => oils}/template (79%)

diff --git a/srcpkgs/oil/template b/srcpkgs/oils/template
similarity index 79%
rename from srcpkgs/oil/template
rename to srcpkgs/oils/template
index c1a9bd148a279d..f6c2d61c3acc17 100644
--- a/srcpkgs/oil/template
+++ b/srcpkgs/oils/template
@@ -1,18 +1,19 @@
-# Template file for 'oil'
-pkgname=oil
+# Template file for 'oils'
+pkgname=oils
 version=0.22.0
 revision=1
 build_style=configure
 configure_args="--prefix=/usr"
 hostmakedepends="$(vopt_if readline readline-devel)"
 makedepends="$hostmakedepends"
-short_desc="Oil, a new unix shell"
+short_desc="Oils, a new unix shell"
 maintainer="MarcoAPC <marcoaureliopc@gmail.com>"
 license="Apache-2.0"
 homepage="https://www.oilshell.org"
 distfiles="https://www.oilshell.org/download/oil-${version}.tar.xz"
 checksum=452e7fd428b686aa752cfeb9be2b94f9fcf7ba9ab22e0ae570a87cdcf782242e
-register_shell="/usr/bin/osh /usr/bin/oil"
+replaces="oil<=0.23.0"
+register_shell="/usr/bin/osh /usr/bin/ysh"
 nocross="Build systems gets confused with host and cross toolchains/headers"
 nostrip=yes
 
@@ -28,8 +29,8 @@ do_install() {
 	# Install binaries
 	vbin _bin/oil.ovm
 	vman doc/osh.1
-	# Symlink osh, oil to oil.ovm.
+	# Symlink osh, ysh to oil.ovm.
 	cd "${DESTDIR}"/usr/bin
 	ln -s oil.ovm osh
-	ln -s oil.ovm oil
+	ln -s oil.ovm ysh
 }

Re: [PR REVIEW] oils: renamed old names

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 201 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#discussion_r1729776507

Comment:
should be a transitional subpackage not replaces

Re: [PR REVIEW] oils: renamed old names

[ahesford]

[-- Attachment #1: Type: text/plain, Size: 221 bytes --]

New review comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#discussion_r1730008026

Comment:
I think both replace and a transitional subpackage are appropriate here.

Re: [PR REVIEW] oils: renamed old names

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 278 bytes --]

New review comment by bugcrazy on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#discussion_r1730047856

Comment:
I consider it more appropriate to replace, than transitional subpackage, only the names were changed, it is still the same model.

Re: [PR REVIEW] oils: renamed old names

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 278 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#discussion_r1730049204

Comment:
without the transitional package, users would be stuck on the old package forever if they don't know to install the other one

Re: [PR PATCH] [Updated] oils: renamed old names

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 1243 bytes --]

There is an updated pull request by bugcrazy against master on the void-packages repository

https://github.com/bugcrazy/void-packages patch-21
https://github.com/void-linux/void-packages/pull/51964

oils: renamed old names
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/51964.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-patch-21-51964.patch --]
[-- Type: text/x-diff, Size: 2001 bytes --]

From ed38a9b45c943e16b58a235ab931e23be867f875 Mon Sep 17 00:00:00 2001
From: bugcrazy <marcoaureliopc@gmail.com>
Date: Sun, 25 Aug 2024 11:59:32 -0300
Subject: [PATCH] oils: renamed old names

---
 srcpkgs/oil                    |  1 +
 srcpkgs/{oil => oils}/template | 19 +++++++++++++------
 2 files changed, 14 insertions(+), 6 deletions(-)
 create mode 120000 srcpkgs/oil
 rename srcpkgs/{oil => oils}/template (71%)

diff --git a/srcpkgs/oil b/srcpkgs/oil
new file mode 120000
index 00000000000000..fd2981096626b0
--- /dev/null
+++ b/srcpkgs/oil
@@ -0,0 +1 @@
+oils
\ No newline at end of file
diff --git a/srcpkgs/oil/template b/srcpkgs/oils/template
similarity index 71%
rename from srcpkgs/oil/template
rename to srcpkgs/oils/template
index c1a9bd148a279d..cfc1bf0c1807f4 100644
--- a/srcpkgs/oil/template
+++ b/srcpkgs/oils/template
@@ -1,18 +1,19 @@
-# Template file for 'oil'
-pkgname=oil
+# Template file for 'oils'
+pkgname=oils
 version=0.22.0
 revision=1
 build_style=configure
 configure_args="--prefix=/usr"
 hostmakedepends="$(vopt_if readline readline-devel)"
 makedepends="$hostmakedepends"
-short_desc="Oil, a new unix shell"
+short_desc="Oils, a new unix shell"
 maintainer="MarcoAPC <marcoaureliopc@gmail.com>"
 license="Apache-2.0"
 homepage="https://www.oilshell.org"
 distfiles="https://www.oilshell.org/download/oil-${version}.tar.xz"
 checksum=452e7fd428b686aa752cfeb9be2b94f9fcf7ba9ab22e0ae570a87cdcf782242e
-register_shell="/usr/bin/osh /usr/bin/oil"
+replaces="oil<=0.23.0"
+register_shell="/usr/bin/osh /usr/bin/ysh"
 nocross="Build systems gets confused with host and cross toolchains/headers"
 nostrip=yes
 
@@ -28,8 +29,14 @@ do_install() {
 	# Install binaries
 	vbin _bin/oil.ovm
 	vman doc/osh.1
-	# Symlink osh, oil to oil.ovm.
+	# Symlink osh, ysh to oil.ovm.
 	cd "${DESTDIR}"/usr/bin
 	ln -s oil.ovm osh
-	ln -s oil.ovm oil
+	ln -s oil.ovm ysh
+}
+
+oil_package() {
+	build_style=meta
+	depends="${sourcepkg}"
+	short_desc+=" - transitional dummy package"
 }

Re: [PR REVIEW] oils: renamed old names

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 154 bytes --]

New review comment by bugcrazy on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#discussion_r1730376828

Comment:
Done!

Re: [PR PATCH] [Updated] oils: renamed old names

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 1243 bytes --]

There is an updated pull request by bugcrazy against master on the void-packages repository

https://github.com/bugcrazy/void-packages patch-21
https://github.com/void-linux/void-packages/pull/51964

oils: renamed old names
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/51964.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-patch-21-51964.patch --]
[-- Type: text/x-diff, Size: 2107 bytes --]

From db3fb66674865a3292802a62abc2af60b2f40bb0 Mon Sep 17 00:00:00 2001
From: bugcrazy <marcoaureliopc@gmail.com>
Date: Mon, 26 Aug 2024 06:52:53 -0300
Subject: [PATCH] oils: renamed old names and update

---
 srcpkgs/oil                    |  1 +
 srcpkgs/{oil => oils}/template | 23 +++++++++++++++--------
 2 files changed, 16 insertions(+), 8 deletions(-)
 create mode 120000 srcpkgs/oil
 rename srcpkgs/{oil => oils}/template (63%)

diff --git a/srcpkgs/oil b/srcpkgs/oil
new file mode 120000
index 00000000000000..fd2981096626b0
--- /dev/null
+++ b/srcpkgs/oil
@@ -0,0 +1 @@
+oils
\ No newline at end of file
diff --git a/srcpkgs/oil/template b/srcpkgs/oils/template
similarity index 63%
rename from srcpkgs/oil/template
rename to srcpkgs/oils/template
index c1a9bd148a279d..8d0ff45223b0e8 100644
--- a/srcpkgs/oil/template
+++ b/srcpkgs/oils/template
@@ -1,18 +1,19 @@
-# Template file for 'oil'
-pkgname=oil
-version=0.22.0
+# Template file for 'oils'
+pkgname=oils
+version=0.23.0
 revision=1
 build_style=configure
 configure_args="--prefix=/usr"
 hostmakedepends="$(vopt_if readline readline-devel)"
 makedepends="$hostmakedepends"
-short_desc="Oil, a new unix shell"
+short_desc="Oils, a new unix shell"
 maintainer="MarcoAPC <marcoaureliopc@gmail.com>"
 license="Apache-2.0"
 homepage="https://www.oilshell.org"
 distfiles="https://www.oilshell.org/download/oil-${version}.tar.xz"
-checksum=452e7fd428b686aa752cfeb9be2b94f9fcf7ba9ab22e0ae570a87cdcf782242e
-register_shell="/usr/bin/osh /usr/bin/oil"
+checksum=7fdc680e1afb5a110e5f87b4b93a8d3b0f3be0146c515c6ef1a3d01a9aada99c
+replaces="oil<=0.24.0"
+register_shell="/usr/bin/osh /usr/bin/ysh"
 nocross="Build systems gets confused with host and cross toolchains/headers"
 nostrip=yes
 
@@ -28,8 +29,14 @@ do_install() {
 	# Install binaries
 	vbin _bin/oil.ovm
 	vman doc/osh.1
-	# Symlink osh, oil to oil.ovm.
+	# Symlink osh, ysh to oil.ovm.
 	cd "${DESTDIR}"/usr/bin
 	ln -s oil.ovm osh
-	ln -s oil.ovm oil
+	ln -s oil.ovm ysh
+}
+
+oil_package() {
+	build_style=meta
+	depends="${sourcepkg}"
+	short_desc+=" - transitional dummy package"
 }

Re: oils: renamed old names and update

[leahneukirchen]

[-- Attachment #1: Type: text/plain, Size: 296 bytes --]

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2343585365

Comment:
So, while we are renaming the package, should we do it like upstream suggests? https://github.com/oils-for-unix/oils/wiki/Oils-Packaging-Guidelines

Re: oils: renamed old names and update

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 469 bytes --]

New comment by bugcrazy on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2344547485

Comment:
> So, while we are renaming the package, should we do it like upstream suggests? https://github.com/oils-for-unix/oils/wiki/Oils-Packaging-Guidelines

The definition itself is confusing, as there are two packages, one Python and one C, we should adopt more logical standards, this oils standard for package, is more correct.

Re: oils: renamed old names and update

[leahneukirchen]

[-- Attachment #1: Type: text/plain, Size: 549 bytes --]

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2344567736

Comment:
Okay, so this is the old Python-version, but there upstream says:

> The oil tarball is the slow reference interpreter based on CPython.
>
>    Don't rename old distro packages. They can remain oil.
>    It's not worth packaging oil in new distros. It may be removed at some point.

So let's keep this (just adding the link is fine), and perhaps someone can contribute `oils-for-unix` instead?

Re: oils: renamed old names and update

[leahneukirchen]

[-- Attachment #1: Type: text/plain, Size: 590 bytes --]

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2344567736

Comment:
Okay, so this is the old Python-version, but there upstream says: https://www.oilshell.org/release/0.23.0/

> The oil tarball is the slow reference interpreter based on CPython.
>
>    Don't rename old distro packages. They can remain oil.
>    It's not worth packaging oil in new distros. It may be removed at some point.

So let's keep this (just adding the link is fine), and perhaps someone can contribute `oils-for-unix` instead?

Re: oils: renamed old names and update

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 1200 bytes --]

New comment by bugcrazy on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2346181164

Comment:
> So, while we are renaming the package, should we do it like upstream suggests? https://github.com/oils-for-unix/oils/wiki/Oils-Packaging-Guidelines

The definition itself is confusing, as there are two packages, one Python and one C, we should adopt more logical standards, this oils standard for package, is more correct.

> Okay, so this is the old Python-version, but there upstream says: https://www.oilshell.org/release/0.23.0/
> 
> > The oil tarball is the slow reference interpreter based on CPython.
> > Don't rename old distro packages. They can remain oil.
> > It's not worth packaging oil in new distros. It may be removed at some point.
> 
> So let's keep this (just adding the link is fine), and perhaps someone can contribute `oils-for-unix` instead?

Python version is slow, a few seconds, C++ version is slower than bash version.

Python version is more stable and tested, has a history of having fewer bugs.

Here is a speed test and some bugs of the C++ version:

https://github.com/oils-for-unix/oils/issues/1069

Re: oils: renamed old names and update

[ahesford]

[-- Attachment #1: Type: text/plain, Size: 346 bytes --]

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2346483806

Comment:
We don't need multiple versions of this package in Void, and the project author notes that the "reference" implementation might go away at some point, so just pick one to be the version packaged by Void.

Re: oils: renamed old names and update

[leahneukirchen]

[-- Attachment #1: Type: text/plain, Size: 251 bytes --]

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2346558224

Comment:
@ahesford doesnt matter, the new version should have a new package name anyway (and conflict, I guess)

Re: oils: renamed old names and update

[meator]

[-- Attachment #1: Type: text/plain, Size: 251 bytes --]

New comment by meator on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2351067155

Comment:
> and perhaps someone can contribute `oils-for-unix` instead?

I've tried to package `oils-for-unix`: #52218

Re: oils: renamed old names and update

[meator]

[-- Attachment #1: Type: text/plain, Size: 242 bytes --]

New comment by meator on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2351108333

Comment:
Also, I vote for keeping `oil` as is and not renaming it to `oils`. This is what upstream recommends.

Re: oils: renamed old names and update

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 306 bytes --]

New comment by bugcrazy on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2383575630

Comment:
I keep the old name oil or rename it to oils, @meator  voted to keep the old name, because I need to know what procedure I do, the package needs to be updated too!

Re: oils: renamed old names and update

[leahneukirchen]

[-- Attachment #1: Type: text/plain, Size: 179 bytes --]

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2383897649

Comment:
yes, please keep the old name.

Re: [PR PATCH] [Updated] oils: renamed old names and update

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

There is an updated pull request by bugcrazy against master on the void-packages repository

https://github.com/bugcrazy/void-packages patch-21
https://github.com/void-linux/void-packages/pull/51964

oils: renamed old names and update
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/51964.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-patch-21-51964.patch --]
[-- Type: text/x-diff, Size: 1366 bytes --]

From 80cdbeb1bccfc370c6a6ec32eae876ee2b3b4254 Mon Sep 17 00:00:00 2001
From: bugcrazy <marcoaureliopc@gmail.com>
Date: Mon, 30 Sep 2024 19:53:59 -0300
Subject: [PATCH] oil: update to  0.23.0

---
 srcpkgs/oil/template | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/oil/template b/srcpkgs/oil/template
index c1a9bd148a279d..839120cea17e4a 100644
--- a/srcpkgs/oil/template
+++ b/srcpkgs/oil/template
@@ -1,6 +1,6 @@
 # Template file for 'oil'
 pkgname=oil
-version=0.22.0
+version=0.23.0
 revision=1
 build_style=configure
 configure_args="--prefix=/usr"
@@ -11,8 +11,8 @@ maintainer="MarcoAPC <marcoaureliopc@gmail.com>"
 license="Apache-2.0"
 homepage="https://www.oilshell.org"
 distfiles="https://www.oilshell.org/download/oil-${version}.tar.xz"
-checksum=452e7fd428b686aa752cfeb9be2b94f9fcf7ba9ab22e0ae570a87cdcf782242e
-register_shell="/usr/bin/osh /usr/bin/oil"
+checksum=7fdc680e1afb5a110e5f87b4b93a8d3b0f3be0146c515c6ef1a3d01a9aada99c
+register_shell="/usr/bin/osh /usr/bin/ysh"
 nocross="Build systems gets confused with host and cross toolchains/headers"
 nostrip=yes
 
@@ -28,8 +28,8 @@ do_install() {
 	# Install binaries
 	vbin _bin/oil.ovm
 	vman doc/osh.1
-	# Symlink osh, oil to oil.ovm.
+	# Symlink osh, ysh to oil.ovm.
 	cd "${DESTDIR}"/usr/bin
 	ln -s oil.ovm osh
-	ln -s oil.ovm oil
+	ln -s oil.ovm ysh
 }

Re: oil: update to 0.23.0

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 184 bytes --]

New comment by bugcrazy on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2384313375

Comment:
> yes, please keep the old name.

Done!

Re: oil: update to 0.23.0

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 203 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2384606667

Comment:
what will happen for anyone using `oil` as login shell? 

Re: [PR PATCH] [Merged]: oil: update to 0.23.0

[leahneukirchen]

[-- Attachment #1: Type: text/plain, Size: 1082 bytes --]

There's a merged pull request on the void-packages repository

oil: update to 0.23.0
https://github.com/void-linux/void-packages/pull/51964

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->

Re: oil: update to 0.23.0

[bugcrazy]

[-- Attachment #1: Type: text/plain, Size: 226 bytes --]

New comment by bugcrazy on void-packages repository

https://github.com/void-linux/void-packages/pull/51964#issuecomment-2386599735

Comment:
> what will happen for anyone using `oil` as login shell?
It will work normally.