[PR PATCH] New package: qbe-1.1

[PR PATCH] New package: qbe-1.1

[BryceVandegrift]

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

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

https://github.com/BryceVandegrift/void-packages qbe
https://github.com/void-linux/void-packages/pull/43951

New package: qbe-1.1
#### 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 (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64-glibc
  - aarch64-musl

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

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

From 87a3935a338f24fb5297685ed8c9e5fb629603b4 Mon Sep 17 00:00:00 2001
From: Bryce Vandegrift <bryce@brycevandegrift.xyz>
Date: Fri, 19 May 2023 16:36:06 -0400
Subject: [PATCH] New package: qbe-1.1

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

diff --git a/srcpkgs/qbe/template b/srcpkgs/qbe/template
new file mode 100644
index 000000000000..65bbb4eab7cb
--- /dev/null
+++ b/srcpkgs/qbe/template
@@ -0,0 +1,21 @@
+# Template file for 'qbe'
+pkgname=qbe
+version=1.1
+revision=1
+archs="x86_64* aarch64*"
+build_style=gnu-makefile
+make_use_env=yes
+short_desc="Small embeddable C compiler backend"
+maintainer="Bryce Vandegrift <bryce@brycevandegrift.xyz>"
+license="MIT"
+homepage="https://c9x.me/compile"
+distfiles="https://c9x.me/compile/release/${pkgname}-${version}.tar.xz"
+checksum=7d0a53dd40df48072aae317e11ddde15d1a980673160e514e235b9ecaa1db12c
+
+post_install() {
+	vdoc doc/abi.txt
+	vdoc doc/il.txt
+	vdoc doc/llvm.txt
+
+	vlicense LICENSE
+}

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1555226112

Comment:
Looks like somthing is wrong with the aarch64 build system.
```
=> ERROR: failed to install cross-vpkg-dummy (error 1)
```

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1555226112

Comment:
Looks like somthing is wrong with the aarch64 build systems.
```
=> ERROR: failed to install cross-vpkg-dummy (error 1)
```

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1555226112

Comment:
Looks like somthing is wrong with the aarch64 builds.
```
=> ERROR: failed to install cross-vpkg-dummy (error 1)
```

Re: [PR REVIEW] New package: qbe-1.1

[mhmdanas]

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

New review comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#discussion_r1199389467

Comment:
Does qbe not support other architectures?

Re: [PR REVIEW] New package: qbe-1.1

[mhmdanas]

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

New review comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#discussion_r1199389580

Comment:
```suggestion
distfiles="https://c9x.me/compile/release/qbe-${version}.tar.xz"
```

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1555234037

Comment:
aarch64-musl build gives this error, might want to re-build.
```
ERROR: chroot-git-2.40.1_1: the RSA signature is not valid!
ERROR: chroot-git-2.40.1_1: removed pkg archive and its signature.
```

Re: [PR REVIEW] New package: qbe-1.1

[BryceVandegrift]

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

New review comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#discussion_r1199390697

Comment:
It *can* be built on other archs (armv7, i686, etc.) but it only compiles to x86_64, aarch64, and riscv.

Re: [PR REVIEW] New package: qbe-1.1

[BryceVandegrift]

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

New review comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#discussion_r1199390697

Comment:
It *can* be built on other archs (armv7, i686, etc.) but it only targets x86_64, aarch64, and riscv.

Re: [PR PATCH] [Updated] New package: qbe-1.1

[BryceVandegrift]

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

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

https://github.com/BryceVandegrift/void-packages qbe
https://github.com/void-linux/void-packages/pull/43951

New package: qbe-1.1
#### 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 (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64-glibc
  - aarch64-musl

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

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

From 1246dc5543e9ee3653be6f15766d8fe7ac36dceb Mon Sep 17 00:00:00 2001
From: Bryce Vandegrift <bryce@brycevandegrift.xyz>
Date: Fri, 19 May 2023 16:36:06 -0400
Subject: [PATCH] New package: qbe-1.1

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

diff --git a/srcpkgs/qbe/template b/srcpkgs/qbe/template
new file mode 100644
index 000000000000..d683422d4bbb
--- /dev/null
+++ b/srcpkgs/qbe/template
@@ -0,0 +1,21 @@
+# Template file for 'qbe'
+pkgname=qbe
+version=1.1
+revision=1
+archs="x86_64* aarch64*"
+build_style=gnu-makefile
+make_use_env=yes
+short_desc="Small embeddable C compiler backend"
+maintainer="Bryce Vandegrift <bryce@brycevandegrift.xyz>"
+license="MIT"
+homepage="https://c9x.me/compile"
+distfiles="https://c9x.me/compile/release/qbe-${version}.tar.xz"
+checksum=7d0a53dd40df48072aae317e11ddde15d1a980673160e514e235b9ecaa1db12c
+
+post_install() {
+	vdoc doc/abi.txt
+	vdoc doc/il.txt
+	vdoc doc/llvm.txt
+
+	vlicense LICENSE
+}

Re: [PR REVIEW] New package: qbe-1.1

[mhmdanas]

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

New review comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#discussion_r1199405835

Comment:
>  but it only targets x86_64, aarch64, and riscv.

If by this you mean that it can compile C code to just those architectures, then restricting `archs` here is unnecessary (since from what you say qbe can be built to other archs just fine).

Re: [PR PATCH] [Updated] New package: qbe-1.1

[BryceVandegrift]

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

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

https://github.com/BryceVandegrift/void-packages qbe
https://github.com/void-linux/void-packages/pull/43951

New package: qbe-1.1
#### 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 (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64-glibc
  - aarch64-musl

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

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

From 50c99be96fa6402e4be78d86adf8c3ea2f8e2221 Mon Sep 17 00:00:00 2001
From: Bryce Vandegrift <bryce@brycevandegrift.xyz>
Date: Fri, 19 May 2023 16:36:06 -0400
Subject: [PATCH] New package: qbe-1.1

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

diff --git a/srcpkgs/qbe/template b/srcpkgs/qbe/template
new file mode 100644
index 000000000000..f8209b395fd1
--- /dev/null
+++ b/srcpkgs/qbe/template
@@ -0,0 +1,27 @@
+# Template file for 'qbe'
+pkgname=qbe
+version=1.1
+revision=1
+build_style=gnu-makefile
+make_use_env=yes
+short_desc="Small embeddable C compiler backend"
+maintainer="Bryce Vandegrift <bryce@brycevandegrift.xyz>"
+license="MIT"
+homepage="https://c9x.me/compile"
+distfiles="https://c9x.me/compile/release/qbe-${version}.tar.xz"
+checksum=7d0a53dd40df48072aae317e11ddde15d1a980673160e514e235b9ecaa1db12c
+
+case "$XBPS_TARGET_MACHINE" in
+	aarch64*) ;;
+	x86_64*) ;;
+# Don't run checks for non x86_64 or aarch64 machines
+	*) make_check=no;
+esac
+
+post_install() {
+	vdoc doc/abi.txt
+	vdoc doc/il.txt
+	vdoc doc/llvm.txt
+
+	vlicense LICENSE
+}

Re: [PR REVIEW] New package: qbe-1.1

[mhmdanas]

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

New review comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#discussion_r1199590267

Comment:
```suggestion
case "$XBPS_TARGET_MACHINE" in
	aarch64* | x86_64*) ;;
	# Don't run checks for non x86_64 or aarch64 machines
	*) make_check=no ;;
esac
```
But anyway, why is this done? Do the tests fail on other archs?

Re: [PR REVIEW] New package: qbe-1.1

[BryceVandegrift]

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

New review comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#discussion_r1199627299

Comment:
Last time I checked, tests fail for i686. I haven't checked 32-bit arm, but I would assume that they might fail as well.

Re: [PR PATCH] [Updated] New package: qbe-1.1

[BryceVandegrift]

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

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

https://github.com/BryceVandegrift/void-packages qbe
https://github.com/void-linux/void-packages/pull/43951

New package: qbe-1.1
#### 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 (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64-glibc
  - aarch64-musl

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

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

From dd5480fdb1a9aa3410a8287e78b1524ae5a5d505 Mon Sep 17 00:00:00 2001
From: Bryce Vandegrift <bryce@brycevandegrift.xyz>
Date: Fri, 19 May 2023 16:36:06 -0400
Subject: [PATCH] New package: qbe-1.1

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

diff --git a/srcpkgs/qbe/template b/srcpkgs/qbe/template
new file mode 100644
index 000000000000..4a6d91d22bf6
--- /dev/null
+++ b/srcpkgs/qbe/template
@@ -0,0 +1,27 @@
+# Template file for 'qbe'
+pkgname=qbe
+version=1.1
+revision=1
+build_style=gnu-makefile
+make_use_env=yes
+short_desc="Small embeddable C compiler backend"
+maintainer="Bryce Vandegrift <bryce@brycevandegrift.xyz>"
+license="MIT"
+homepage="https://c9x.me/compile"
+distfiles="https://c9x.me/compile/release/qbe-${version}.tar.xz"
+checksum=7d0a53dd40df48072aae317e11ddde15d1a980673160e514e235b9ecaa1db12c
+
+case "$XBPS_TARGET_MACHINE" in
+	aarch64*) ;;
+	x86_64*) ;;
+	# Don't run checks for non x86_64 or aarch64 machines
+	*) make_check=no;
+esac
+
+post_install() {
+	vdoc doc/abi.txt
+	vdoc doc/il.txt
+	vdoc doc/llvm.txt
+
+	vlicense LICENSE
+}

Re: New package: qbe-1.1

[Sertonix]

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

New comment by Sertonix on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1556052561

Comment:
I think you need to overwrite the default target of qbe on 32-bit platforms.
I packaged qbe for termux. Maybe [this](https://github.com/termux-user-repository/tur/blob/master/tur/qbe/build.sh) helps. (I did ignore 32bit builds but the default target overwrite is included)

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1556294884

Comment:
@Sertonix I am unsure what exactly you are referring to. Probably because I am unfamiliar with the termux build system.

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1556294884

Comment:
@Sertonix I am unsure what exactly you are referring to (specifically with the link you provided). Probably because I am unfamiliar with the termux build system.

Re: New package: qbe-1.1

[Sertonix]

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

New comment by Sertonix on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1556301520

Comment:
No problem if you don't know the termux build system. I saw this message and remembered the problems I had.

> Last time I checked, tests fail for i686. I haven't checked 32-bit arm, but I would assume that they might fail as well.

I am not that good with the void build system yet.
I know that the default build target in the [Makefile](https://c9x.me/git/qbe.git/tree/Makefile#n31) defaults to arm64. This means that if the tests are run on an unsupported architecture it tries to execute arm64 binaries.

When you define the `TARGET` enviromental variable to `arm64` [this](https://c9x.me/git/qbe.git/tree/tools/test.sh#n22) code should run the tests using qemu and then they could work.

Re: New package: qbe-1.1

[Sertonix]

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

New comment by Sertonix on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1556301520

Comment:
No problem if you don't know the termux build system. I saw this message and remembered the problems I had.

> Last time I checked, tests fail for i686. I haven't checked 32-bit arm, but I would assume that they might fail as well.

I am not that good with the void build system yet.
I know that the default build target in the [Makefile](https://c9x.me/git/qbe.git/tree/Makefile#n31) defaults to arm64. This means that if the tests are run on an unsupported architecture it tries to execute arm64 binaries.

The test could work if you set the `TARGET` enviromental variable to `arm64`. Then [this](https://c9x.me/git/qbe.git/tree/tools/test.sh#n22) code code should run the tests using qemu.

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1556322664

Comment:
The only problem is that an aarch64 compiler (probably gcc) needs to be available for i686. I am unsure if one exists for Void Linux.

Re: New package: qbe-1.1

[BryceVandegrift]

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

New comment by BryceVandegrift on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1556322664

Comment:
The only problem is that an aarch64 compiler (probably gcc) needs to be available for i686, armv7l, etc. I am unsure if one exists for Void Linux. I think that it might be best that these tests are disabled.

Re: New package: qbe-1.1

[tomekw]

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

New comment by tomekw on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1568123991

Comment:
It works perfectly on my x86_64 box. I can't wait to have it :)

Re: New package: qbe-1.1

[tomekw]

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

New comment by tomekw on void-packages repository

https://github.com/void-linux/void-packages/pull/43951#issuecomment-1568229636

Comment:
Also, Hare installed and works perfectly!

Re: [PR PATCH] [Closed]: New package: qbe-1.1

[Duncaen]

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

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

New package: qbe-1.1
https://github.com/void-linux/void-packages/pull/43951

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 (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64-glibc
  - aarch64-musl