Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] vba-m: make work on archs other than x86.
@ 2020-10-11  4:56 fosslinux
  2020-10-13  5:43 ` [PR PATCH] [Updated] " fosslinux
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: fosslinux @ 2020-10-11  4:56 UTC (permalink / raw)
  To: ml

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

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

https://github.com/fosslinux/void-packages vba-m-archs
https://github.com/void-linux/void-packages/pull/25509

vba-m: make work on archs other than x86.
There was a block of inline asm that was for x86 only, a patch from
master fixed this months ago; chuck it in.

Also cross compiling to the ppc* family works now with the fixed
hostmakedepends.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-vba-m-archs-25509.patch --]
[-- Type: text/x-diff, Size: 3338 bytes --]

From 16e2e673984edb0a7f303f43b0a9103c086a9468 Mon Sep 17 00:00:00 2001
From: fosslinux <fosslinux@aussies.space>
Date: Sun, 11 Oct 2020 15:51:22 +1100
Subject: [PATCH] vba-m: make work on archs other than x86.

There was a block of inline asm that was for x86 only, a patch from
master fixed this months ago; chuck it in.

Also cross compiling to the ppc* family works now with the fixed
hostmakedepends.
---
 srcpkgs/vba-m/patches/x86-asm.patch | 37 +++++++++++++++++++++++++++++
 srcpkgs/vba-m/template              |  6 ++---
 2 files changed, 39 insertions(+), 4 deletions(-)
 create mode 100644 srcpkgs/vba-m/patches/x86-asm.patch

diff --git a/srcpkgs/vba-m/patches/x86-asm.patch b/srcpkgs/vba-m/patches/x86-asm.patch
new file mode 100644
index 00000000000..e2ce3a48e77
--- /dev/null
+++ b/srcpkgs/vba-m/patches/x86-asm.patch
@@ -0,0 +1,37 @@
+From af0de1c4b308ef8d9a081ecf407805b75a99d877 Mon Sep 17 00:00:00 2001
+From: Rafael Kitover <rkitover@gmail.com>
+Date: Fri, 4 Oct 2019 07:35:49 +0000
+Subject: [PATCH] xbrz: fix inline asm check
+
+Use correct cpp code to detect x86/amd64 architecture to use inline asm.
+
+Signed-off-by: Rafael Kitover <rkitover@gmail.com>
+---
+ src/filters/xBRZ/xbrz.cpp | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git src/filters/xBRZ/xbrz.cpp src/filters/xBRZ/xbrz.cpp
+index 36d70be26..13e6cdc13 100644
+--- src/filters/xBRZ/xbrz.cpp
++++ src/filters/xBRZ/xbrz.cpp
+@@ -66,17 +66,17 @@ uint32_t gradientARGB(uint32_t pixFront, uint32_t pixBack) //find intermediate c
+ 
+ inline double fastSqrt(double n)
+ {
+-#ifdef __GNUC__ || __clang__ || __MINGW64_VERSION_MAJOR || __MINGW32_MAJOR_VERSION
++#if (defined(__GNUC__) || defined(__clang__)) && (defined(__x86_64__) || defined(__i386__))
+     __asm__ ("fsqrt" : "+t" (n));
+     return n;
+-#elif _MSC_VER && _M_IX86
++#elif defined(_MSC_VER) && defined(_M_IX86)
+     // speeds up xBRZ by about 9% compared to std::sqrt which internally uses
+     // the same assembler instructions but adds some "fluff"
+     __asm {
+         fld n
+         fsqrt
+     }
+-#else // _MSC_VER && _M_X64 OR other platforms
++#else // defined(_MSC_VER) && defined(_M_X64) OR other platforms
+     // VisualStudio x86_64 does not allow inline ASM
+     return std::sqrt(n);
+ #endif
diff --git a/srcpkgs/vba-m/template b/srcpkgs/vba-m/template
index 8d69e721e36..c8e7d28dbb4 100644
--- a/srcpkgs/vba-m/template
+++ b/srcpkgs/vba-m/template
@@ -2,12 +2,11 @@
 pkgname=vba-m
 reverts=1292_2
 version=2.1.4
-revision=2
-archs="x86_64* i686*"
+revision=3
 wrksrc="visualboyadvance-m-${version}"
 build_style=cmake
 configure_args="-DENABLE_GTK=TRUE -DENABLE_FFMPEG=TRUE -DENABLE_LINK=TRUE"
-hostmakedepends="gettext pkg-config unzip zip yasm wxWidgets-common-devel"
+hostmakedepends="gettext pkg-config unzip zip yasm wxWidgets-gtk3-devel"
 makedepends="zlib-devel libpng-devel MesaLib-devel libopenal-devel SDL2-devel
  gtkmm2-devel gtkglext-devel gtkglextmm-devel ffmpeg-devel gtk+3-devel
  SFML-devel wxWidgets-gtk3-devel libgomp-devel"
@@ -17,4 +16,3 @@ license="GPL-2.0-or-later"
 homepage="https://github.com/visualboyadvance-m/visualboyadvance-m/"
 distfiles="https://github.com/visualboyadvance-m/visualboyadvance-m/archive/v${version}.tar.gz"
 checksum=8342e017212842de66b0d86bc7610c82349af2d4e59951d969a33fff956c39ba
-nocross=y

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

* Re: [PR PATCH] [Updated] vba-m: make work on archs other than x86.
  2020-10-11  4:56 [PR PATCH] vba-m: make work on archs other than x86 fosslinux
@ 2020-10-13  5:43 ` fosslinux
  2020-10-16  7:58 ` fosslinux
  2020-10-20 20:15 ` [PR PATCH] [Closed]: " ahesford
  2 siblings, 0 replies; 4+ messages in thread
From: fosslinux @ 2020-10-13  5:43 UTC (permalink / raw)
  To: ml

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

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

https://github.com/fosslinux/void-packages vba-m-archs
https://github.com/void-linux/void-packages/pull/25509

vba-m: make work on archs other than x86.
There was a block of inline asm that was for x86 only, a patch from
master fixed this months ago; chuck it in.

Also cross compiling to the ppc* family works now with the fixed
hostmakedepends.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-vba-m-archs-25509.patch --]
[-- Type: text/x-diff, Size: 3338 bytes --]

From f34c4c29e82f383bacfb36f6c1621281b82c9b04 Mon Sep 17 00:00:00 2001
From: fosslinux <fosslinux@aussies.space>
Date: Sun, 11 Oct 2020 15:51:22 +1100
Subject: [PATCH] vba-m: make work on archs other than x86.

There was a block of inline asm that was for x86 only, a patch from
master fixed this months ago; chuck it in.

Also cross compiling to the ppc* family works now with the fixed
hostmakedepends.
---
 srcpkgs/vba-m/patches/x86-asm.patch | 37 +++++++++++++++++++++++++++++
 srcpkgs/vba-m/template              |  6 ++---
 2 files changed, 39 insertions(+), 4 deletions(-)
 create mode 100644 srcpkgs/vba-m/patches/x86-asm.patch

diff --git a/srcpkgs/vba-m/patches/x86-asm.patch b/srcpkgs/vba-m/patches/x86-asm.patch
new file mode 100644
index 00000000000..e2ce3a48e77
--- /dev/null
+++ b/srcpkgs/vba-m/patches/x86-asm.patch
@@ -0,0 +1,37 @@
+From af0de1c4b308ef8d9a081ecf407805b75a99d877 Mon Sep 17 00:00:00 2001
+From: Rafael Kitover <rkitover@gmail.com>
+Date: Fri, 4 Oct 2019 07:35:49 +0000
+Subject: [PATCH] xbrz: fix inline asm check
+
+Use correct cpp code to detect x86/amd64 architecture to use inline asm.
+
+Signed-off-by: Rafael Kitover <rkitover@gmail.com>
+---
+ src/filters/xBRZ/xbrz.cpp | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git src/filters/xBRZ/xbrz.cpp src/filters/xBRZ/xbrz.cpp
+index 36d70be26..13e6cdc13 100644
+--- src/filters/xBRZ/xbrz.cpp
++++ src/filters/xBRZ/xbrz.cpp
+@@ -66,17 +66,17 @@ uint32_t gradientARGB(uint32_t pixFront, uint32_t pixBack) //find intermediate c
+ 
+ inline double fastSqrt(double n)
+ {
+-#ifdef __GNUC__ || __clang__ || __MINGW64_VERSION_MAJOR || __MINGW32_MAJOR_VERSION
++#if (defined(__GNUC__) || defined(__clang__)) && (defined(__x86_64__) || defined(__i386__))
+     __asm__ ("fsqrt" : "+t" (n));
+     return n;
+-#elif _MSC_VER && _M_IX86
++#elif defined(_MSC_VER) && defined(_M_IX86)
+     // speeds up xBRZ by about 9% compared to std::sqrt which internally uses
+     // the same assembler instructions but adds some "fluff"
+     __asm {
+         fld n
+         fsqrt
+     }
+-#else // _MSC_VER && _M_X64 OR other platforms
++#else // defined(_MSC_VER) && defined(_M_X64) OR other platforms
+     // VisualStudio x86_64 does not allow inline ASM
+     return std::sqrt(n);
+ #endif
diff --git a/srcpkgs/vba-m/template b/srcpkgs/vba-m/template
index 8d69e721e36..c8e7d28dbb4 100644
--- a/srcpkgs/vba-m/template
+++ b/srcpkgs/vba-m/template
@@ -2,12 +2,11 @@
 pkgname=vba-m
 reverts=1292_2
 version=2.1.4
-revision=2
-archs="x86_64* i686*"
+revision=3
 wrksrc="visualboyadvance-m-${version}"
 build_style=cmake
 configure_args="-DENABLE_GTK=TRUE -DENABLE_FFMPEG=TRUE -DENABLE_LINK=TRUE"
-hostmakedepends="gettext pkg-config unzip zip yasm wxWidgets-common-devel"
+hostmakedepends="gettext pkg-config unzip zip yasm wxWidgets-gtk3-devel"
 makedepends="zlib-devel libpng-devel MesaLib-devel libopenal-devel SDL2-devel
  gtkmm2-devel gtkglext-devel gtkglextmm-devel ffmpeg-devel gtk+3-devel
  SFML-devel wxWidgets-gtk3-devel libgomp-devel"
@@ -17,4 +16,3 @@ license="GPL-2.0-or-later"
 homepage="https://github.com/visualboyadvance-m/visualboyadvance-m/"
 distfiles="https://github.com/visualboyadvance-m/visualboyadvance-m/archive/v${version}.tar.gz"
 checksum=8342e017212842de66b0d86bc7610c82349af2d4e59951d969a33fff956c39ba
-nocross=y

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

* Re: [PR PATCH] [Updated] vba-m: make work on archs other than x86.
  2020-10-11  4:56 [PR PATCH] vba-m: make work on archs other than x86 fosslinux
  2020-10-13  5:43 ` [PR PATCH] [Updated] " fosslinux
@ 2020-10-16  7:58 ` fosslinux
  2020-10-20 20:15 ` [PR PATCH] [Closed]: " ahesford
  2 siblings, 0 replies; 4+ messages in thread
From: fosslinux @ 2020-10-16  7:58 UTC (permalink / raw)
  To: ml

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

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

https://github.com/fosslinux/void-packages vba-m-archs
https://github.com/void-linux/void-packages/pull/25509

vba-m: make work on archs other than x86.
There was a block of inline asm that was for x86 only, a patch from
master fixed this months ago; chuck it in.

Also cross compiling to the ppc* family works now with the fixed
hostmakedepends.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-vba-m-archs-25509.patch --]
[-- Type: text/x-diff, Size: 3306 bytes --]

From 61a67fdb43c7a658bd01ba564b4b0fee35594d3d Mon Sep 17 00:00:00 2001
From: fosslinux <fosslinux@aussies.space>
Date: Sun, 11 Oct 2020 15:51:22 +1100
Subject: [PATCH] vba-m: make work on archs other than x86.

There was a block of inline asm that was for x86 only, a patch from
master fixed this months ago; chuck it in.

Also cross compiling to the ppc* family works now with the fixed
hostmakedepends.
---
 srcpkgs/vba-m/patches/x86-asm.patch | 37 +++++++++++++++++++++++++++++
 srcpkgs/vba-m/template              |  6 ++---
 2 files changed, 39 insertions(+), 4 deletions(-)
 create mode 100644 srcpkgs/vba-m/patches/x86-asm.patch

diff --git a/srcpkgs/vba-m/patches/x86-asm.patch b/srcpkgs/vba-m/patches/x86-asm.patch
new file mode 100644
index 00000000000..e2ce3a48e77
--- /dev/null
+++ b/srcpkgs/vba-m/patches/x86-asm.patch
@@ -0,0 +1,37 @@
+From af0de1c4b308ef8d9a081ecf407805b75a99d877 Mon Sep 17 00:00:00 2001
+From: Rafael Kitover <rkitover@gmail.com>
+Date: Fri, 4 Oct 2019 07:35:49 +0000
+Subject: [PATCH] xbrz: fix inline asm check
+
+Use correct cpp code to detect x86/amd64 architecture to use inline asm.
+
+Signed-off-by: Rafael Kitover <rkitover@gmail.com>
+---
+ src/filters/xBRZ/xbrz.cpp | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git src/filters/xBRZ/xbrz.cpp src/filters/xBRZ/xbrz.cpp
+index 36d70be26..13e6cdc13 100644
+--- src/filters/xBRZ/xbrz.cpp
++++ src/filters/xBRZ/xbrz.cpp
+@@ -66,17 +66,17 @@ uint32_t gradientARGB(uint32_t pixFront, uint32_t pixBack) //find intermediate c
+ 
+ inline double fastSqrt(double n)
+ {
+-#ifdef __GNUC__ || __clang__ || __MINGW64_VERSION_MAJOR || __MINGW32_MAJOR_VERSION
++#if (defined(__GNUC__) || defined(__clang__)) && (defined(__x86_64__) || defined(__i386__))
+     __asm__ ("fsqrt" : "+t" (n));
+     return n;
+-#elif _MSC_VER && _M_IX86
++#elif defined(_MSC_VER) && defined(_M_IX86)
+     // speeds up xBRZ by about 9% compared to std::sqrt which internally uses
+     // the same assembler instructions but adds some "fluff"
+     __asm {
+         fld n
+         fsqrt
+     }
+-#else // _MSC_VER && _M_X64 OR other platforms
++#else // defined(_MSC_VER) && defined(_M_X64) OR other platforms
+     // VisualStudio x86_64 does not allow inline ASM
+     return std::sqrt(n);
+ #endif
diff --git a/srcpkgs/vba-m/template b/srcpkgs/vba-m/template
index 659de62621c..8e6447f5c6f 100644
--- a/srcpkgs/vba-m/template
+++ b/srcpkgs/vba-m/template
@@ -2,12 +2,11 @@
 pkgname=vba-m
 reverts=1292_2
 version=2.1.4
-revision=3
-archs="x86_64* i686*"
+revision=4
 wrksrc="visualboyadvance-m-${version}"
 build_style=cmake
 configure_args="-DENABLE_GTK=TRUE -DENABLE_FFMPEG=TRUE -DENABLE_LINK=TRUE"
-hostmakedepends="gettext pkg-config unzip zip yasm wxWidgets-common-devel"
+hostmakedepends="gettext pkg-config unzip zip yasm wxWidgets-gtk3-devel"
 makedepends="zlib-devel libpng-devel MesaLib-devel libopenal-devel SDL2-devel
  gtkmm2-devel ffmpeg-devel gtk+3-devel
  SFML-devel wxWidgets-gtk3-devel libgomp-devel"
@@ -17,4 +16,3 @@ license="GPL-2.0-or-later"
 homepage="https://github.com/visualboyadvance-m/visualboyadvance-m/"
 distfiles="https://github.com/visualboyadvance-m/visualboyadvance-m/archive/v${version}.tar.gz"
 checksum=8342e017212842de66b0d86bc7610c82349af2d4e59951d969a33fff956c39ba
-nocross=y

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

* Re: [PR PATCH] [Closed]: vba-m: make work on archs other than x86.
  2020-10-11  4:56 [PR PATCH] vba-m: make work on archs other than x86 fosslinux
  2020-10-13  5:43 ` [PR PATCH] [Updated] " fosslinux
  2020-10-16  7:58 ` fosslinux
@ 2020-10-20 20:15 ` ahesford
  2 siblings, 0 replies; 4+ messages in thread
From: ahesford @ 2020-10-20 20:15 UTC (permalink / raw)
  To: ml

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

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

vba-m: make work on archs other than x86.
https://github.com/void-linux/void-packages/pull/25509

Description:
There was a block of inline asm that was for x86 only, a patch from
master fixed this months ago; chuck it in.

Also cross compiling to the ppc* family works now with the fixed
hostmakedepends.

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

end of thread, other threads:[~2020-10-20 20:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-11  4:56 [PR PATCH] vba-m: make work on archs other than x86 fosslinux
2020-10-13  5:43 ` [PR PATCH] [Updated] " fosslinux
2020-10-16  7:58 ` fosslinux
2020-10-20 20:15 ` [PR PATCH] [Closed]: " ahesford

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