[PR PATCH] New package: alass-2.0.0

[PR PATCH] New package: alass-2.0.0

[drichline]

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

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

https://github.com/drichline/void-packages alass-branch
https://github.com/void-linux/void-packages/pull/39972

New package: alass-2.0.0
Closes #38965 

#### 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:
  - aarch64-glibc

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

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

From 3eceea0a6b1f32d1a6884b7a86894a80207e2ba3 Mon Sep 17 00:00:00 2001
From: Dakota Richline <43507179+drichline@users.noreply.github.com>
Date: Sat, 15 Oct 2022 15:33:48 -0400
Subject: [PATCH] New package: alass-2.0.0

---
 srcpkgs/alass/template | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 srcpkgs/alass/template

diff --git a/srcpkgs/alass/template b/srcpkgs/alass/template
new file mode 100644
index 000000000000..588aadd5e4cb
--- /dev/null
+++ b/srcpkgs/alass/template
@@ -0,0 +1,17 @@
+# Template file for 'alass'
+pkgname=alass
+version=2.0.0
+revision=1
+build_style=cargo
+depends="ffmpeg"
+short_desc="Command line tool to synchronize subtitles to movies"
+maintainer="Dakota Richline <drichline@protonmail.com>"
+license="GPL-3.0-only"
+homepage="https://github.com/kaegi/alass"
+distfiles="https://github.com/kaegi/alass/archive/refs/tags/v${version}.tar.gz"
+checksum=ce88f92c7a427b623edcabb1b64e80be70cca2777f3da4b96702820a6cdf1e26
+
+do_install() {
+	vbin target/${RUST_TARGET}/release/alass-cli
+	vdoc README.md
+}

Re: [PR PATCH] [Updated] New package: alass-2.0.0

[drichline]

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

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

https://github.com/drichline/void-packages alass-branch
https://github.com/void-linux/void-packages/pull/39972

New package: alass-2.0.0
Closes #38965 

#### 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:
  - aarch64-glibc

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

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

From 9100037fcf9fff4385f2af188feb4faafdb4bacb Mon Sep 17 00:00:00 2001
From: Dakota Richline <43507179+drichline@users.noreply.github.com>
Date: Sat, 15 Oct 2022 15:33:48 -0400
Subject: [PATCH] New package: alass-2.0.0

---
 srcpkgs/alass/template | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)
 create mode 100644 srcpkgs/alass/template

diff --git a/srcpkgs/alass/template b/srcpkgs/alass/template
new file mode 100644
index 000000000000..18940c3de4f6
--- /dev/null
+++ b/srcpkgs/alass/template
@@ -0,0 +1,21 @@
+# Template file for 'alass'
+pkgname=alass
+version=2.0.0
+revision=1
+build_style=cargo
+depends="ffmpeg"
+short_desc="Command line tool to synchronize subtitles to movies"
+maintainer="Dakota Richline <drichline@protonmail.com>"
+license="GPL-3.0-only"
+homepage="https://github.com/kaegi/alass"
+distfiles="https://github.com/kaegi/alass/archive/refs/tags/v${version}.tar.gz"
+checksum=ce88f92c7a427b623edcabb1b64e80be70cca2777f3da4b96702820a6cdf1e26
+
+do_install() {
+	vbin target/${RUST_TARGET}/release/alass-cli
+	vdoc README.md
+}
+
+if [ "$XBPS_TARGET_MACHINE" = "i686" ]; then
+	make_check=ci-skip # Tests fail to build on i686 container with error "ran out of registers during register allocation"
+fi

Re: [PR REVIEW] New package: alass-2.0.0

[classabbyamp]

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#discussion_r996354544

Comment:
```suggestion
post_install() {
```

Re: [PR REVIEW] New package: alass-2.0.0

[classabbyamp]

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#discussion_r996355070

Comment:
let's use a case statement here instead because, while we don't currently have i686-musl, we should still consider it

and also change the `ci-skip` to `no` because it fails locally too

```suggestion
case "$XBPS_TARGET_MACHINE" in
	i686*)
		make_check=no # Tests fail to build on i686 with error "ran out of registers during register allocation"
		;;
esac
```

and, as a stylistic thing, let's put this before the functions and after the main variable block

Re: [PR REVIEW] New package: alass-2.0.0

[classabbyamp]

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#discussion_r996354516

Comment:
this will make it install properly
```suggestion
build_style=cargo
make_install_target="--path alass-cli"
```

Re: [PR REVIEW] New package: alass-2.0.0

[drichline]

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

New review comment by drichline on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#discussion_r997235261

Comment:
Thanks again for the help!! With the new `make_install_target`, I'm still getting the same error "`found a virtual manifest at `/builddir/alass-2.0.0/Cargo.toml` instead of a package manifest`" as before. It seems like the path should be right, but this is my first time using cargo. 

Re: [PR REVIEW] New package: alass-2.0.0

[drichline]

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

New review comment by drichline on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#discussion_r997235261

Comment:
Thanks again for the help!! With the new `make_install_target`, I'm still getting the same error "`found a virtual manifest at /builddir/alass-2.0.0/Cargo.toml instead of a package manifest`" as before. It seems like the path should be right, but this is my first time using cargo. 

Re: [PR REVIEW] New package: alass-2.0.0

[drichline]

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

New review comment by drichline on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#discussion_r997235261

Comment:
Thanks again for the help!! With the new `make_install_target`, I'm still getting the same error "`found a virtual manifest at /builddir/alass-2.0.0/Cargo.toml instead of a package manifest`" as before (even after an xbps-src clean). It seems like the path should be right, but this is my first time using cargo. 

Re: [PR REVIEW] New package: alass-2.0.0

[classabbyamp]

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#discussion_r997984706

Comment:
hm weird, it worked for me when I tried it locally

Re: New package: alass-2.0.0

[github-actions]

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

New comment by github-actions[bot] on void-packages repository

https://github.com/void-linux/void-packages/pull/39972#issuecomment-1384741694

Comment:
Pull Requests become stale 90 days after last activity and are closed 14 days after that.  If this pull request is still relevant bump it or assign it.

Re: [PR PATCH] [Closed]: New package: alass-2.0.0

[github-actions]

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

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

New package: alass-2.0.0
https://github.com/void-linux/void-packages/pull/39972

Description:
Closes #38965 

#### 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:
  - aarch64-glibc