Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] fish-shell: Update to 4.0b1
@ 2025-01-02 19:18 ChrisW-B
  2025-01-02 19:22 ` zdykstra
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: ChrisW-B @ 2025-01-02 19:18 UTC (permalink / raw)
  To: ml

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

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

https://github.com/ChrisW-B/void-packages fish-beta
https://github.com/void-linux/void-packages/pull/53814

fish-shell: Update to 4.0b1
<!-- Uncomment relevant sections and delete options which are not applicable -->

Adding this as a draft because I wanted to use fish 4.0b1 to get around a fish fzf focus bug, and noticed that fish-shell is currently orphaned. This works, I've built and run on both a x86_64 glibc and musl machine. 
However I'm new to void packages, so this might be doing more work than necessary, I'd love any feedback on things before fish v4 releases.

To explain the changes- Fish v4 has moved to Rust, but still uses CMake for build at the moment. There's also a note on the distributors section that notes
> fish no longer depends on the ncurses library, but still uses a terminfo database. When packaging fish, please add a dependency on the package containing your terminfo database instead of curses.

so I've removed the ncurses dependency- I couldn't find any packages in void-packages that noted a terminfo dependency though, so I haven't included that. I may have just been searching for the wrong thing though, please let me know if there's something I should be doing!

Current release notes are here: https://fishshell.com/docs/4.0b1/relnotes.html#fish-4-0b1-released-december-17-2024

I'm more than happy to take over maintenance for fish-shell, but at the moment I'm pretty inexperienced with packaging!

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

<!--
#### 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, (x86_64-musl, x86_64-glibc)
<!--
- 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/53814.patch is attached

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

From c256a2a8dac9476d7be210381db28e64dac7f445 Mon Sep 17 00:00:00 2001
From: Chris Barry <blue.iron1752@chrisb.xyz>
Date: Thu, 2 Jan 2025 16:35:15 +0000
Subject: [PATCH] Update fish to 4.0b1

---
 srcpkgs/fish-shell/template | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/fish-shell/template b/srcpkgs/fish-shell/template
index 41c526ad417df9..b6a8b00765ce45 100644
--- a/srcpkgs/fish-shell/template
+++ b/srcpkgs/fish-shell/template
@@ -1,10 +1,12 @@
 # Template file for 'fish-shell'
 pkgname=fish-shell
-version=3.7.1
+version=4.0b1
 revision=1
 build_style=cmake
-hostmakedepends="gettext"
-makedepends="ncurses-devel pcre2-devel"
+build_helper="rust"
+hostmakedepends="rust cargo gettext pcre2-devel"
+configure_args="-DCMAKE_BUILD_TYPE=Release"
+makedepends="rust-std pcre2-devel"
 checkdepends="python3-pexpect procps-ng"
 short_desc="User friendly shell intended mostly for interactive use"
 maintainer="Orphaned <orphan@voidlinux.org>"
@@ -12,7 +14,7 @@ license="GPL-2.0-only"
 homepage="https://fishshell.com/"
 changelog="https://github.com/fish-shell/fish-shell/raw/master/CHANGELOG.rst"
 distfiles="https://github.com/fish-shell/fish-shell/releases/download/${version}/fish-${version}.tar.xz"
-checksum=614c9f5643cd0799df391395fa6bbc3649427bb839722ce3b114d3bbc1a3b250
+checksum=534334e10f85722214e9daff82a57cc3501235523f16f8f131c2344e4ec98da7
 register_shell="/bin/fish /usr/bin/fish"
 # tests don't work as root
 make_check=ci-skip

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: fish-shell: Update to 4.0b1
  2025-01-02 19:18 [PR PATCH] fish-shell: Update to 4.0b1 ChrisW-B
@ 2025-01-02 19:22 ` zdykstra
  2025-01-02 19:23 ` zdykstra
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: zdykstra @ 2025-01-02 19:22 UTC (permalink / raw)
  To: ml

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

New comment by zdykstra on void-packages repository

https://github.com/void-linux/void-packages/pull/53814#issuecomment-2568258838

Comment:
`ncurses-term` provides the contents of `/usr/share/terminfo/`

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: fish-shell: Update to 4.0b1
  2025-01-02 19:18 [PR PATCH] fish-shell: Update to 4.0b1 ChrisW-B
  2025-01-02 19:22 ` zdykstra
@ 2025-01-02 19:23 ` zdykstra
  2025-01-02 19:25 ` [PR PATCH] [Closed]: " classabbyamp
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: zdykstra @ 2025-01-02 19:23 UTC (permalink / raw)
  To: ml

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

New comment by zdykstra on void-packages repository

https://github.com/void-linux/void-packages/pull/53814#issuecomment-2568258838

Comment:
`ncurses-term` and `ncurses-base` provide the contents of `/usr/share/terminfo/`

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Closed]: fish-shell: Update to 4.0b1
  2025-01-02 19:18 [PR PATCH] fish-shell: Update to 4.0b1 ChrisW-B
  2025-01-02 19:22 ` zdykstra
  2025-01-02 19:23 ` zdykstra
@ 2025-01-02 19:25 ` classabbyamp
  2025-01-02 19:25 ` classabbyamp
  2025-01-02 19:43 ` ChrisW-B
  4 siblings, 0 replies; 6+ messages in thread
From: classabbyamp @ 2025-01-02 19:25 UTC (permalink / raw)
  To: ml

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

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

fish-shell: Update to 4.0b1
https://github.com/void-linux/void-packages/pull/53814

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

Adding this as a draft because I wanted to use fish 4.0b1 to get around a fish fzf focus bug, and noticed that fish-shell is currently orphaned. This works, I've built and run on both a x86_64 glibc and musl machine. 
Of course I'm not asking to merge a beta version of fish-shell, I'm just new to void packages, so this might just have incorrect build dependency setup and I'd love any feedback on things before fish v4 releases.

To explain the changes- Fish v4 has moved to Rust, but still uses CMake for build at the moment. There's also a note on the distributors section that notes
> fish no longer depends on the ncurses library, but still uses a terminfo database. When packaging fish, please add a dependency on the package containing your terminfo database instead of curses.

so I've removed the ncurses dependency- I couldn't find any packages in void-packages that noted a terminfo dependency though, so I haven't included that. I may have just been searching for the wrong thing though, please let me know if there's something I should be doing!

Current release notes are here: https://fishshell.com/docs/4.0b1/relnotes.html#fish-4-0b1-released-december-17-2024

I'm more than happy to take over maintenance for fish-shell, but at the moment I'm pretty inexperienced with packaging!

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

<!--
#### 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, (x86_64-musl, x86_64-glibc)
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: fish-shell: Update to 4.0b1
  2025-01-02 19:18 [PR PATCH] fish-shell: Update to 4.0b1 ChrisW-B
                   ` (2 preceding siblings ...)
  2025-01-02 19:25 ` [PR PATCH] [Closed]: " classabbyamp
@ 2025-01-02 19:25 ` classabbyamp
  2025-01-02 19:43 ` ChrisW-B
  4 siblings, 0 replies; 6+ messages in thread
From: classabbyamp @ 2025-01-02 19:25 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/53814#issuecomment-2568264270

Comment:
let's wait until the actual release before opening PRs

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: fish-shell: Update to 4.0b1
  2025-01-02 19:18 [PR PATCH] fish-shell: Update to 4.0b1 ChrisW-B
                   ` (3 preceding siblings ...)
  2025-01-02 19:25 ` classabbyamp
@ 2025-01-02 19:43 ` ChrisW-B
  4 siblings, 0 replies; 6+ messages in thread
From: ChrisW-B @ 2025-01-02 19:43 UTC (permalink / raw)
  To: ml

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

New comment by ChrisW-B on void-packages repository

https://github.com/void-linux/void-packages/pull/53814#issuecomment-2568283902

Comment:
@classabbyamp no problem, sorry to spam!

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2025-01-02 19:43 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-01-02 19:18 [PR PATCH] fish-shell: Update to 4.0b1 ChrisW-B
2025-01-02 19:22 ` zdykstra
2025-01-02 19:23 ` zdykstra
2025-01-02 19:25 ` [PR PATCH] [Closed]: " classabbyamp
2025-01-02 19:25 ` classabbyamp
2025-01-02 19:43 ` ChrisW-B

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).