Github messages for voidlinux
 help / color / mirror / Atom feed
* Re: [PR REVIEW] New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
  2020-08-16 13:05 ` [PR REVIEW] New package: qmidiarp-0.6.5 Piraty
@ 2020-08-17 18:50 ` mvf
  2021-07-20 19:57 ` ericonr
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: mvf @ 2020-08-17 18:50 UTC (permalink / raw)
  To: ml

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

New review comment by mvf on void-packages repository

https://github.com/void-linux/void-packages/pull/23072#discussion_r471708906

Comment:
Hm, what would be the benefit? A patch would be more verbose and messier to maintain. The envvar is straightforward and build system agnostic...

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

* Re: [PR REVIEW] New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
  2020-08-16 13:05 ` [PR REVIEW] New package: qmidiarp-0.6.5 Piraty
  2020-08-17 18:50 ` mvf
@ 2021-07-20 19:57 ` ericonr
  2021-07-20 19:57 ` ericonr
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-07-20 19:57 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/23072#discussion_r673438419

Comment:
This release is from 2017... Things using Qt sometimes start accumulating more and more patches :/

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

* Re: [PR REVIEW] New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
                   ` (2 preceding siblings ...)
  2021-07-20 19:57 ` ericonr
@ 2021-07-20 19:57 ` ericonr
  2021-07-26  7:01 ` [PR PATCH] [Updated] " mvf
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-07-20 19:57 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/23072#discussion_r673439386

Comment:
Is the removal because of the `-I$EXTRA_QT_INCLUDE_DIR` inclusion?

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

* Re: [PR PATCH] [Updated] New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
                   ` (3 preceding siblings ...)
  2021-07-20 19:57 ` ericonr
@ 2021-07-26  7:01 ` mvf
  2021-07-26  7:07 ` [PR REVIEW] " mvf
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: mvf @ 2021-07-26  7:01 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mvf/void-packages pr/qmidiarp
https://github.com/void-linux/void-packages/pull/23072

New package: qmidiarp-0.6.5


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

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

From dc1b9c9f350730fbafaa56c4fe69b243aabe056f Mon Sep 17 00:00:00 2001
From: Matthias von Faber <mvf@gmx.eu>
Date: Wed, 17 Jun 2020 22:11:17 +0200
Subject: [PATCH] New package: qmidiarp-0.6.5

---
 .../patches/add-missing-includes.patch        | 21 +++++++++++++++++++
 srcpkgs/qmidiarp/patches/cross.patch          | 15 +++++++++++++
 srcpkgs/qmidiarp/template                     | 18 ++++++++++++++++
 3 files changed, 54 insertions(+)
 create mode 100644 srcpkgs/qmidiarp/patches/add-missing-includes.patch
 create mode 100644 srcpkgs/qmidiarp/patches/cross.patch
 create mode 100644 srcpkgs/qmidiarp/template

diff --git a/srcpkgs/qmidiarp/patches/add-missing-includes.patch b/srcpkgs/qmidiarp/patches/add-missing-includes.patch
new file mode 100644
index 000000000000..13cb6658aed0
--- /dev/null
+++ b/srcpkgs/qmidiarp/patches/add-missing-includes.patch
@@ -0,0 +1,21 @@
+--- a/src/main.cpp
++++ b/src/main.cpp
+@@ -69,6 +69,8 @@
+ #include "mainwindow.h"
+ #include "main.h"
+ 
++#include <sys/types.h>
++#include <unistd.h>
+ 
+ static struct option options[] = {
+     {"version", 0, 0, 'v'},
+--- a/src/midiworker.h
++++ b/src/midiworker.h
+@@ -28,6 +28,7 @@
+ #include "main.h"
+ #include <cstdlib>
+ #include <cstdio>
++#include <sys/types.h>
+ 
+ 
+ /*! @brief MIDI worker base class for QMidiArp modules.
diff --git a/srcpkgs/qmidiarp/patches/cross.patch b/srcpkgs/qmidiarp/patches/cross.patch
new file mode 100644
index 000000000000..0b7c7dae06e6
--- /dev/null
+++ b/srcpkgs/qmidiarp/patches/cross.patch
@@ -0,0 +1,15 @@
+--- a/configure.ac
++++ b/configure.ac
+@@ -127,12 +127,6 @@ if test "x$ac_buildapp" = "xyes" -o ["x$ac_lv2pluginuis" = "xyes"  -a "x$ac_lv2w
+           QT_INCLUDE_DIR=`$PKG_CONFIG --variable=includedir Qt5Gui`
+           QT_CXXFLAGS=`$PKG_CONFIG --cflags Qt5Core Qt5Gui Qt5Widgets`
+           QT_CXXFLAGS="$QT_CXXFLAGS -fPIC"
+-          EXTRA_QT_INCLUDE_DIR="$QT_INCLUDE_DIR/Qt"
+-          AC_CHECK_FILE([$QT_INCLUDE_DIR/QtWidgets/QWidget],
+-            AC_MSG_NOTICE([No extra QT_INCLUDE_DIR needed]),
+-              AC_CHECK_FILE([$EXTRA_QT_INCLUDE_DIR/QtWidgets/QWidget],
+-                QT_CPPFLAGS="$QT_CPPFLAGS -I$EXTRA_QT_INCLUDE_DIR",
+-                  AC_MSG_WARN([QWidget not found])))
+         
+           AC_SUBST(QT_CXXFLAGS)
+           AC_SUBST(Qt5_LIBS)
diff --git a/srcpkgs/qmidiarp/template b/srcpkgs/qmidiarp/template
new file mode 100644
index 000000000000..04ada9038b07
--- /dev/null
+++ b/srcpkgs/qmidiarp/template
@@ -0,0 +1,18 @@
+# Template file for 'qmidiarp'
+pkgname=qmidiarp
+version=0.6.5
+revision=1
+build_style=gnu-configure
+hostmakedepends="automake libtool pkg-config qt5-host-tools"
+makedepends="alsa-lib-devel jack-devel lv2 qt5-devel"
+short_desc="MIDI arpeggiator and LFO"
+maintainer="Matthias von Faber <mvf@gmx.eu>"
+license="GPL-2.0-or-later"
+homepage="http://qmidiarp.sourceforge.net"
+distfiles="${SOURCEFORGE_SITE}/project/${pkgname}/${pkgname}/${version}/${pkgname}-${version}.tar.bz2"
+checksum=22a3a26d3dbe4bf215aa33c0fd4a79c088549328477840d00e72e50c6e807e10
+CXXFLAGS="-D_GNU_SOURCE"
+
+pre_configure() {
+	autoreconf -if
+}

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

* Re: [PR REVIEW] New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
                   ` (4 preceding siblings ...)
  2021-07-26  7:01 ` [PR PATCH] [Updated] " mvf
@ 2021-07-26  7:07 ` mvf
  2021-07-26  7:08 ` mvf
  2021-12-28 13:04 ` [PR PATCH] [Merged]: " Duncaen
  7 siblings, 0 replies; 8+ messages in thread
From: mvf @ 2021-07-26  7:07 UTC (permalink / raw)
  To: ml

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

New review comment by mvf on void-packages repository

https://github.com/void-linux/void-packages/pull/23072#discussion_r676343100

Comment:
Hm, IME it's not worse than with other libs. Of course, there will likely have to be a patch for Qt6 at some point.

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

* Re: [PR REVIEW] New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
                   ` (5 preceding siblings ...)
  2021-07-26  7:07 ` [PR REVIEW] " mvf
@ 2021-07-26  7:08 ` mvf
  2021-12-28 13:04 ` [PR PATCH] [Merged]: " Duncaen
  7 siblings, 0 replies; 8+ messages in thread
From: mvf @ 2021-07-26  7:08 UTC (permalink / raw)
  To: ml

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

New review comment by mvf on void-packages repository

https://github.com/void-linux/void-packages/pull/23072#discussion_r676343746

Comment:
This removes a cross-unaware file existence check that breaks the build:
```
checking for /usr/include/qt5/QtWidgets/QWidget... configure: error: cannot check for file existence when cross compiling
=> ERROR: qmidiarp-0.6.5_1: do_configure: '${configure_script} ${configure_args}' exited with 1
=> ERROR:   in do_configure() at common/build-style/gnu-configure.sh:8
```
It's unnecessary too, because the `pkg-config` based check succeeds and returns the same include paths:
```
checking for Qt5Core Qt5Gui Qt5Widgets >= 5.0... yes
```

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

* Re: [PR PATCH] [Merged]: New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
                   ` (6 preceding siblings ...)
  2021-07-26  7:08 ` mvf
@ 2021-12-28 13:04 ` Duncaen
  7 siblings, 0 replies; 8+ messages in thread
From: Duncaen @ 2021-12-28 13:04 UTC (permalink / raw)
  To: ml

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

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

New package: qmidiarp-0.6.5
https://github.com/void-linux/void-packages/pull/23072

Description:


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

* Re: [PR REVIEW] New package: qmidiarp-0.6.5
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
@ 2020-08-16 13:05 ` Piraty
  2020-08-17 18:50 ` mvf
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: Piraty @ 2020-08-16 13:05 UTC (permalink / raw)
  To: ml

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

New review comment by Piraty on void-packages repository

https://github.com/void-linux/void-packages/pull/23072#discussion_r471110830

Comment:
since you run autoeconf anyway, patch it in?

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

end of thread, other threads:[~2021-12-28 13:04 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-23072@inbox.vuxu.org>
2020-08-16 13:05 ` [PR REVIEW] New package: qmidiarp-0.6.5 Piraty
2020-08-17 18:50 ` mvf
2021-07-20 19:57 ` ericonr
2021-07-20 19:57 ` ericonr
2021-07-26  7:01 ` [PR PATCH] [Updated] " mvf
2021-07-26  7:07 ` [PR REVIEW] " mvf
2021-07-26  7:08 ` mvf
2021-12-28 13:04 ` [PR PATCH] [Merged]: " Duncaen

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