Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] New package: bash-language-server-4.8.3
@ 2023-03-17  7:31 sirkhancision
  2023-03-17 15:04 ` sirkhancision
                   ` (20 more replies)
  0 siblings, 21 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17  7:31 UTC (permalink / raw)
  To: ml

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

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

https://github.com/sirkhancision/void-packages bash-language-server
https://github.com/void-linux/void-packages/pull/42818

New package: bash-language-server-4.8.3
#### 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**

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - i686
  - x86_64-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-bash-language-server-42818.patch --]
[-- Type: text/x-diff, Size: 1645 bytes --]

From 5573f695fa7ee77a1ac35cea40ab27e0e9f7b1b6 Mon Sep 17 00:00:00 2001
From: sirkhancision <jsantiago12tone@gmail.com>
Date: Fri, 17 Mar 2023 04:28:47 -0300
Subject: [PATCH] New package: bash-language-server-4.8.3

---
 srcpkgs/bash-language-server/template | 37 +++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)
 create mode 100644 srcpkgs/bash-language-server/template

diff --git a/srcpkgs/bash-language-server/template b/srcpkgs/bash-language-server/template
new file mode 100644
index 000000000000..a5618ac20daa
--- /dev/null
+++ b/srcpkgs/bash-language-server/template
@@ -0,0 +1,37 @@
+# Template file for 'bash-language-server'
+pkgname=bash-language-server
+version=4.8.3
+revision=1
+archs="i686 x86_64"
+hostmakedepends="jq yarn"
+depends="nodejs"
+short_desc="Bash language server"
+maintainer="sirkhancision <jsantiago12tone@gmail.com>"
+license="MIT"
+homepage="https://github.com/bash-lsp/bash-language-server"
+distfiles="https://github.com/bash-lsp/bash-language-server/archive/refs/tags/server-${version}.tar.gz"
+checksum=a6c367e19a007b889f59fa7aa6a8c4fb0fae83fbfcc928e7a8f4366015dedbd6
+
+do_build() {
+	yarn
+	yarn run compile
+}
+
+do_install() {
+	TARGET_PATH="usr/lib/${pkgname}"
+	cd server
+
+	cp package.json{,.bak}
+	read -ra devDependencies < <(jq -r '.devDependencies | keys | join(" ")' package.json)
+	yarn remove --frozen-lockfile "${devDependencies[@]}"
+	mv package.json{.bak,}
+
+	vmkdir ${TARGET_PATH}
+	vcopy * ${TARGET_PATH}
+
+	vmkdir usr/bin
+	ln -sf /${TARGET_PATH}/out/cli.js ${DESTDIR}/usr/bin/${pkgname}
+	chmod 0755 ${DESTDIR}/${TARGET_PATH}/out/cli.js
+
+	vlicense ${wrksrc}/LICENSE
+}

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
@ 2023-03-17 15:04 ` sirkhancision
  2023-03-17 19:41 ` icp1994
                   ` (19 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 15:04 UTC (permalink / raw)
  To: ml

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

New comment by sirkhancision on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1473981574

Comment:
This also resolves #38606

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
  2023-03-17 15:04 ` sirkhancision
@ 2023-03-17 19:41 ` icp1994
  2023-03-17 21:59 ` classabbyamp
                   ` (18 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: icp1994 @ 2023-03-17 19:41 UTC (permalink / raw)
  To: ml

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

New comment by icp1994 on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474327927

Comment:
Unfortunate to have a language server for bash being available only under `i686` & `x86_64`.

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
  2023-03-17 15:04 ` sirkhancision
  2023-03-17 19:41 ` icp1994
@ 2023-03-17 21:59 ` classabbyamp
  2023-03-17 22:28 ` [PR PATCH] [Updated] " sirkhancision
                   ` (17 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 21:59 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474442320

Comment:
how did you build this for x86_64-musl when `archs="i686 x86_64"`?

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

* Re: [PR PATCH] [Updated] New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (2 preceding siblings ...)
  2023-03-17 21:59 ` classabbyamp
@ 2023-03-17 22:28 ` sirkhancision
  2023-03-17 22:29 ` classabbyamp
                   ` (16 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 22:28 UTC (permalink / raw)
  To: ml

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

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

https://github.com/sirkhancision/void-packages bash-language-server
https://github.com/void-linux/void-packages/pull/42818

New package: bash-language-server-4.8.3
#### 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**

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - i686
  - x86_64-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-bash-language-server-42818.patch --]
[-- Type: text/x-diff, Size: 1654 bytes --]

From b933346ebf00d5c5869da417c6d0ff5e7a7434da Mon Sep 17 00:00:00 2001
From: sirkhancision <jsantiago12tone@gmail.com>
Date: Fri, 17 Mar 2023 04:28:47 -0300
Subject: [PATCH] New package: bash-language-server-4.8.3

---
 srcpkgs/bash-language-server/template | 37 +++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)
 create mode 100644 srcpkgs/bash-language-server/template

diff --git a/srcpkgs/bash-language-server/template b/srcpkgs/bash-language-server/template
new file mode 100644
index 000000000000..1537340f3f37
--- /dev/null
+++ b/srcpkgs/bash-language-server/template
@@ -0,0 +1,37 @@
+# Template file for 'bash-language-server'
+pkgname=bash-language-server
+version=4.8.3
+revision=1
+#archs="i686-glibc x86_64-*"
+hostmakedepends="jq yarn"
+depends="nodejs"
+short_desc="Bash language server"
+maintainer="sirkhancision <jsantiago12tone@gmail.com>"
+license="MIT"
+homepage="https://github.com/bash-lsp/bash-language-server"
+distfiles="https://github.com/bash-lsp/bash-language-server/archive/refs/tags/server-${version}.tar.gz"
+checksum=a6c367e19a007b889f59fa7aa6a8c4fb0fae83fbfcc928e7a8f4366015dedbd6
+
+do_build() {
+	yarn
+	yarn run compile
+}
+
+do_install() {
+	TARGET_PATH="usr/lib/${pkgname}"
+	cd server
+
+	cp package.json{,.bak}
+	read -ra devDependencies < <(jq -r '.devDependencies | keys | join(" ")' package.json)
+	yarn remove --frozen-lockfile "${devDependencies[@]}"
+	mv package.json{.bak,}
+
+	vmkdir ${TARGET_PATH}
+	vcopy * ${TARGET_PATH}
+
+	vmkdir usr/bin
+	ln -sf /${TARGET_PATH}/out/cli.js ${DESTDIR}/usr/bin/${pkgname}
+	chmod 0755 ${DESTDIR}/${TARGET_PATH}/out/cli.js
+
+	vlicense ${wrksrc}/LICENSE
+}

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (3 preceding siblings ...)
  2023-03-17 22:28 ` [PR PATCH] [Updated] " sirkhancision
@ 2023-03-17 22:29 ` classabbyamp
  2023-03-17 22:29 ` [PR REVIEW] " classabbyamp
                   ` (15 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 22:29 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474467934

Comment:
> Unfortunate to have a language server for bash being available only under `i686` & `x86_64`.

cross-compiles fine for me 🤷‍♀️

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

* Re: [PR REVIEW] New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (4 preceding siblings ...)
  2023-03-17 22:29 ` classabbyamp
@ 2023-03-17 22:29 ` classabbyamp
  2023-03-17 22:29 ` classabbyamp
                   ` (14 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 22:29 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#discussion_r1140761902

Comment:
```suggestion
	read -ra _devDependencies < <(jq -r '.devDependencies | keys | join(" ")' package.json)
	yarn remove --frozen-lockfile "${_devDependencies[@]}"
```

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

* Re: [PR REVIEW] New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (5 preceding siblings ...)
  2023-03-17 22:29 ` [PR REVIEW] " classabbyamp
@ 2023-03-17 22:29 ` classabbyamp
  2023-03-17 22:29 ` sirkhancision
                   ` (13 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 22:29 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#discussion_r1140762181

Comment:
```suggestion
	vlicense LICENSE
```

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (6 preceding siblings ...)
  2023-03-17 22:29 ` classabbyamp
@ 2023-03-17 22:29 ` sirkhancision
  2023-03-17 22:30 ` sirkhancision
                   ` (12 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 22:29 UTC (permalink / raw)
  To: ml

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

New comment by sirkhancision on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474468393

Comment:
> how did you build this for x86_64-musl when `archs="i686 x86_64"`?

My bad, I had built it before restricting the architectures, and I thought x86_64 meant both glibc and musl. It's corrected now, and limiting i686 to glibc, as building it for me fails in other architectures.

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (7 preceding siblings ...)
  2023-03-17 22:29 ` sirkhancision
@ 2023-03-17 22:30 ` sirkhancision
  2023-03-17 22:31 ` classabbyamp
                   ` (11 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 22:30 UTC (permalink / raw)
  To: ml

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

New comment by sirkhancision on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474468393

Comment:
> how did you build this for x86_64-musl when `archs="i686 x86_64"`?

My bad, I had built it before restricting the architectures, and I thought x86_64 meant both glibc and musl. It's corrected now, and limiting i686 to glibc, as building it in i686-musl fails for me.

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (8 preceding siblings ...)
  2023-03-17 22:30 ` sirkhancision
@ 2023-03-17 22:31 ` classabbyamp
  2023-03-17 22:36 ` [PR REVIEW] " sirkhancision
                   ` (10 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 22:31 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474470176

Comment:
i686-musl doesn't really exist, so don't worry about it.

`[archname]` means glibc, `[archname]-musl` means musl, `[archname]-*` is both

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

* Re: [PR REVIEW] New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (9 preceding siblings ...)
  2023-03-17 22:31 ` classabbyamp
@ 2023-03-17 22:36 ` sirkhancision
  2023-03-17 22:37 ` classabbyamp
                   ` (9 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 22:36 UTC (permalink / raw)
  To: ml

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

New review comment by sirkhancision on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#discussion_r1140777513

Comment:
This is because of the previous `cd` used, meaning that the license is at the child directory.

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

* Re: [PR REVIEW] New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (10 preceding siblings ...)
  2023-03-17 22:36 ` [PR REVIEW] " sirkhancision
@ 2023-03-17 22:37 ` classabbyamp
  2023-03-17 22:42 ` sirkhancision
                   ` (8 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 22:37 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#discussion_r1140778523

Comment:
../LICENSE then

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (11 preceding siblings ...)
  2023-03-17 22:37 ` classabbyamp
@ 2023-03-17 22:42 ` sirkhancision
  2023-03-17 22:43 ` classabbyamp
                   ` (7 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 22:42 UTC (permalink / raw)
  To: ml

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

New comment by sirkhancision on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474478893

Comment:
> i686-musl doesn't really exist, so don't worry about it.
> 
> `[archname]` means glibc, `[archname]-musl` means musl, `[archname]-*` is both

I rewrote it as `x86_64-*` but it's now saying that the package can't be built for x86_64, any ideas as to why that is happening now?

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (12 preceding siblings ...)
  2023-03-17 22:42 ` sirkhancision
@ 2023-03-17 22:43 ` classabbyamp
  2023-03-17 22:44 ` sirkhancision
                   ` (6 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 22:43 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474479923

Comment:
sorry `x86_64*` (no dash), but you shouldn't need to specify the archs, it should work on all of them

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (13 preceding siblings ...)
  2023-03-17 22:43 ` classabbyamp
@ 2023-03-17 22:44 ` sirkhancision
  2023-03-17 22:47 ` classabbyamp
                   ` (5 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 22:44 UTC (permalink / raw)
  To: ml

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

New comment by sirkhancision on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474480051

Comment:
> > Unfortunate to have a language server for bash being available only under `i686` & `x86_64`.
> 
> cross-compiles fine for me woman_shrugging

To me, it says:
`=> ERROR: nodejs-16.19.0_1: cannot be cross compiled...`

`=> ERROR: nodejs-16.19.0_1: host and target must have the same pointer size`

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (14 preceding siblings ...)
  2023-03-17 22:44 ` sirkhancision
@ 2023-03-17 22:47 ` classabbyamp
  2023-03-17 23:10 ` [PR PATCH] [Updated] " sirkhancision
                   ` (4 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-03-17 22:47 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474483939

Comment:
it works on aarch64, but you shouldn't limit the arch because of that error: someone can still build it for themselves either natively or cross from 32bit

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

* Re: [PR PATCH] [Updated] New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (15 preceding siblings ...)
  2023-03-17 22:47 ` classabbyamp
@ 2023-03-17 23:10 ` sirkhancision
  2023-03-17 23:11 ` sirkhancision
                   ` (3 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 23:10 UTC (permalink / raw)
  To: ml

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

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

https://github.com/sirkhancision/void-packages bash-language-server
https://github.com/void-linux/void-packages/pull/42818

New package: bash-language-server-4.8.3
#### 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**

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - i686
  - x86_64-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-bash-language-server-42818.patch --]
[-- Type: text/x-diff, Size: 1619 bytes --]

From 93fb792e99e77eb5e71dce6b544e41bb8e85675e Mon Sep 17 00:00:00 2001
From: sirkhancision <jsantiago12tone@gmail.com>
Date: Fri, 17 Mar 2023 04:28:47 -0300
Subject: [PATCH] New package: bash-language-server-4.8.3

---
 srcpkgs/bash-language-server/template | 36 +++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)
 create mode 100644 srcpkgs/bash-language-server/template

diff --git a/srcpkgs/bash-language-server/template b/srcpkgs/bash-language-server/template
new file mode 100644
index 000000000000..02fcb44dfbf2
--- /dev/null
+++ b/srcpkgs/bash-language-server/template
@@ -0,0 +1,36 @@
+# Template file for 'bash-language-server'
+pkgname=bash-language-server
+version=4.8.3
+revision=1
+hostmakedepends="jq yarn"
+depends="nodejs"
+short_desc="Bash language server"
+maintainer="sirkhancision <jsantiago12tone@gmail.com>"
+license="MIT"
+homepage="https://github.com/bash-lsp/bash-language-server"
+distfiles="https://github.com/bash-lsp/bash-language-server/archive/refs/tags/server-${version}.tar.gz"
+checksum=a6c367e19a007b889f59fa7aa6a8c4fb0fae83fbfcc928e7a8f4366015dedbd6
+
+do_build() {
+	yarn
+	yarn run compile
+}
+
+do_install() {
+	TARGET_PATH="usr/lib/${pkgname}"
+	cd server
+
+	cp package.json{,.bak}
+	read -ra _devDependencies < <(jq -r '.devDependencies | keys | join(" ")' package.json)
+	yarn remove --frozen-lockfile "${_devDependencies[@]}"
+	mv package.json{.bak,}
+
+	vmkdir ${TARGET_PATH}
+	vcopy * ${TARGET_PATH}
+
+	vmkdir usr/bin
+	ln -sf /${TARGET_PATH}/out/cli.js ${DESTDIR}/usr/bin/${pkgname}
+	chmod 0755 ${DESTDIR}/${TARGET_PATH}/out/cli.js
+
+	vlicense ../LICENSE
+}

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (16 preceding siblings ...)
  2023-03-17 23:10 ` [PR PATCH] [Updated] " sirkhancision
@ 2023-03-17 23:11 ` sirkhancision
  2023-05-09  4:37 ` tjkirch
                   ` (2 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-03-17 23:11 UTC (permalink / raw)
  To: ml

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

New comment by sirkhancision on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1474500540

Comment:
> it works on aarch64, but you shouldn't limit the arch because of that error: someone can still build it for themselves either natively or cross from 32bit

Indeed, it built fine on aarch64. Thanks for the help with these packages, and the tips, you're awesome!

I think it might be fine, now.

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

* Re: New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (17 preceding siblings ...)
  2023-03-17 23:11 ` sirkhancision
@ 2023-05-09  4:37 ` tjkirch
  2023-05-09 18:22 ` [PR PATCH] [Updated] " sirkhancision
  2023-05-10 14:46 ` [PR PATCH] [Merged]: New package: bash-language-server-4.9.1 classabbyamp
  20 siblings, 0 replies; 22+ messages in thread
From: tjkirch @ 2023-05-09  4:37 UTC (permalink / raw)
  To: ml

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

New comment by tjkirch on void-packages repository

https://github.com/void-linux/void-packages/pull/42818#issuecomment-1539394096

Comment:
Thanks @sirkhancision!  This is working great for me from a local build.  I also tested with bash-language-server 4.9.1 and it worked fine, in case you want to update the PR.

```diff
diff --git a/srcpkgs/bash-language-server/template b/srcpkgs/bash-language-server/template
index 02fcb44dfb..5c8f99cdd4 100644
--- a/srcpkgs/bash-language-server/template
+++ b/srcpkgs/bash-language-server/template
@@ -1,6 +1,6 @@
 # Template file for 'bash-language-server'
 pkgname=bash-language-server
-version=4.8.3
+version=4.9.1
 revision=1
 hostmakedepends="jq yarn"
 depends="nodejs"
@@ -9,7 +9,7 @@ maintainer="sirkhancision <jsantiago12tone@gmail.com>"
 license="MIT"
 homepage="https://github.com/bash-lsp/bash-language-server"
 distfiles="https://github.com/bash-lsp/bash-language-server/archive/refs/tags/server-${version}.tar.gz"
-checksum=a6c367e19a007b889f59fa7aa6a8c4fb0fae83fbfcc928e7a8f4366015dedbd6
+checksum=eef93c1167394fb01fc300c5b8c326fd9362dd3118a1e075549342e24e9d6e5c
 
 do_build() {
 	yarn
```

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

* Re: [PR PATCH] [Updated] New package: bash-language-server-4.8.3
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (18 preceding siblings ...)
  2023-05-09  4:37 ` tjkirch
@ 2023-05-09 18:22 ` sirkhancision
  2023-05-10 14:46 ` [PR PATCH] [Merged]: New package: bash-language-server-4.9.1 classabbyamp
  20 siblings, 0 replies; 22+ messages in thread
From: sirkhancision @ 2023-05-09 18:22 UTC (permalink / raw)
  To: ml

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

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

https://github.com/sirkhancision/void-packages bash-language-server
https://github.com/void-linux/void-packages/pull/42818

New package: bash-language-server-4.8.3
#### 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**

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - i686
  - x86_64-musl
  - aarch64


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-bash-language-server-42818.patch --]
[-- Type: text/x-diff, Size: 1619 bytes --]

From ca0f2b57c0d061109378c3534e7a43601addc609 Mon Sep 17 00:00:00 2001
From: sirkhancision <jsantiago12tone@gmail.com>
Date: Fri, 17 Mar 2023 04:28:47 -0300
Subject: [PATCH] New package: bash-language-server-4.9.1

---
 srcpkgs/bash-language-server/template | 36 +++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)
 create mode 100644 srcpkgs/bash-language-server/template

diff --git a/srcpkgs/bash-language-server/template b/srcpkgs/bash-language-server/template
new file mode 100644
index 000000000000..5c8f99cdd428
--- /dev/null
+++ b/srcpkgs/bash-language-server/template
@@ -0,0 +1,36 @@
+# Template file for 'bash-language-server'
+pkgname=bash-language-server
+version=4.9.1
+revision=1
+hostmakedepends="jq yarn"
+depends="nodejs"
+short_desc="Bash language server"
+maintainer="sirkhancision <jsantiago12tone@gmail.com>"
+license="MIT"
+homepage="https://github.com/bash-lsp/bash-language-server"
+distfiles="https://github.com/bash-lsp/bash-language-server/archive/refs/tags/server-${version}.tar.gz"
+checksum=eef93c1167394fb01fc300c5b8c326fd9362dd3118a1e075549342e24e9d6e5c
+
+do_build() {
+	yarn
+	yarn run compile
+}
+
+do_install() {
+	TARGET_PATH="usr/lib/${pkgname}"
+	cd server
+
+	cp package.json{,.bak}
+	read -ra _devDependencies < <(jq -r '.devDependencies | keys | join(" ")' package.json)
+	yarn remove --frozen-lockfile "${_devDependencies[@]}"
+	mv package.json{.bak,}
+
+	vmkdir ${TARGET_PATH}
+	vcopy * ${TARGET_PATH}
+
+	vmkdir usr/bin
+	ln -sf /${TARGET_PATH}/out/cli.js ${DESTDIR}/usr/bin/${pkgname}
+	chmod 0755 ${DESTDIR}/${TARGET_PATH}/out/cli.js
+
+	vlicense ../LICENSE
+}

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

* Re: [PR PATCH] [Merged]: New package: bash-language-server-4.9.1
  2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
                   ` (19 preceding siblings ...)
  2023-05-09 18:22 ` [PR PATCH] [Updated] " sirkhancision
@ 2023-05-10 14:46 ` classabbyamp
  20 siblings, 0 replies; 22+ messages in thread
From: classabbyamp @ 2023-05-10 14:46 UTC (permalink / raw)
  To: ml

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

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

New package: bash-language-server-4.9.1
https://github.com/void-linux/void-packages/pull/42818

Description:
#### 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**

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - i686
  - x86_64-musl
  - aarch64


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

end of thread, other threads:[~2023-05-10 14:46 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-17  7:31 [PR PATCH] New package: bash-language-server-4.8.3 sirkhancision
2023-03-17 15:04 ` sirkhancision
2023-03-17 19:41 ` icp1994
2023-03-17 21:59 ` classabbyamp
2023-03-17 22:28 ` [PR PATCH] [Updated] " sirkhancision
2023-03-17 22:29 ` classabbyamp
2023-03-17 22:29 ` [PR REVIEW] " classabbyamp
2023-03-17 22:29 ` classabbyamp
2023-03-17 22:29 ` sirkhancision
2023-03-17 22:30 ` sirkhancision
2023-03-17 22:31 ` classabbyamp
2023-03-17 22:36 ` [PR REVIEW] " sirkhancision
2023-03-17 22:37 ` classabbyamp
2023-03-17 22:42 ` sirkhancision
2023-03-17 22:43 ` classabbyamp
2023-03-17 22:44 ` sirkhancision
2023-03-17 22:47 ` classabbyamp
2023-03-17 23:10 ` [PR PATCH] [Updated] " sirkhancision
2023-03-17 23:11 ` sirkhancision
2023-05-09  4:37 ` tjkirch
2023-05-09 18:22 ` [PR PATCH] [Updated] " sirkhancision
2023-05-10 14:46 ` [PR PATCH] [Merged]: New package: bash-language-server-4.9.1 classabbyamp

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).