From: "David Seifert" <soap@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: app-accessibility/brltty/files/, app-accessibility/brltty/
Date: Mon, 4 Dec 2017 10:26:57 +0000 (UTC) [thread overview]
Message-ID: <1512383112.9972fed3f21b47e7315949e39f762d9b209c1fd5.soap@gentoo> (raw)
commit: 9972fed3f21b47e7315949e39f762d9b209c1fd5
Author: David Seifert <soap <AT> gentoo <DOT> org>
AuthorDate: Mon Dec 4 10:24:38 2017 +0000
Commit: David Seifert <soap <AT> gentoo <DOT> org>
CommitDate: Mon Dec 4 10:25:12 2017 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9972fed3
app-accessibility/brltty: [QA] Fix multiple issues
* Don't add dead python impls to PYTHON_COMPAT
* Don't remove python metadata vars (${PYTHON_REQUIRED_USE})
from REQUIRED_USE
* Don't remove the SLOT specifier for sys-libs/ncurses
* Get rid of the awfully complicated, hacky and
non-generalisable solution for the libtinfo split,
which also doesn't generalise to Prefix.
Using pkg-config is easier, safer, more portable and
less brittle than the previous hacks.
* Get rid of the totally unnecessary code paths
created by dispatching on glibc versions.
'sys/sysmacros.h' can always be included on glibc,
packages just have relied on this include implicitly.
Package-Manager: Portage-2.3.16, Repoman-2.3.6
app-accessibility/brltty/brltty-5.2-r1.ebuild | 12 +++++-------
.../brltty/files/brltty-5.2-sysmacros.patch | 20 ++++++++------------
2 files changed, 13 insertions(+), 19 deletions(-)
diff --git a/app-accessibility/brltty/brltty-5.2-r1.ebuild b/app-accessibility/brltty/brltty-5.2-r1.ebuild
index 12ea62ef94e..71cbc57d9d1 100644
--- a/app-accessibility/brltty/brltty-5.2-r1.ebuild
+++ b/app-accessibility/brltty/brltty-5.2-r1.ebuild
@@ -3,7 +3,7 @@
EAPI=5
-PYTHON_COMPAT=( python2_7 python3_3 python3_4 python3_5 )
+PYTHON_COMPAT=( python2_7 python3_{4,5} )
FINDLIB_USE="ocaml"
inherit findlib eutils multilib toolchain-funcs java-pkg-opt-2 flag-o-matic \
@@ -22,7 +22,7 @@ IUSE="+api +beeper bluetooth +contracted-braille doc +fm gpm iconv icu
REQUIRED_USE="doc? ( api )
java? ( api )
ocaml? ( api )
- python? ( api )
+ python? ( api ${PYTHON_REQUIRED_USE} )
tcl? ( api )"
COMMON_DEP="bluetooth? ( net-wireless/bluez )
@@ -30,7 +30,7 @@ COMMON_DEP="bluetooth? ( net-wireless/bluez )
iconv? ( virtual/libiconv )
icu? ( dev-libs/icu:= )
python? ( ${PYTHON_DEPS} )
- ncurses? ( sys-libs/ncurses:= )
+ ncurses? ( sys-libs/ncurses:0= )
nls? ( virtual/libintl )
tcl? ( >=dev-lang/tcl-8.4.15:0= )
usb? ( virtual/libusb:0 )
@@ -64,10 +64,8 @@ src_prepare() {
}
src_configure() {
- filter-flags "_*_SOURCE*"
- append-cppflags -D_DEFAULT_SOURCE
- has_version ">=sys-libs/glibc-2.25-r5" && append-cppflags -DHAVE_SYS_SYSMACROS_H
- has_version "sys-libs/ncurses[tinfo]" && append-libs -ltinfo
+ append-cppflags "$($(tc-getPKG_CONFIG) --cflags ncurses)"
+ append-libs "$($(tc-getPKG_CONFIG) --libs ncurses)"
tc-export AR LD PKG_CONFIG
# override prefix in order to install into /
diff --git a/app-accessibility/brltty/files/brltty-5.2-sysmacros.patch b/app-accessibility/brltty/files/brltty-5.2-sysmacros.patch
index d45d6104e4b..f33e46a1890 100644
--- a/app-accessibility/brltty/files/brltty-5.2-sysmacros.patch
+++ b/app-accessibility/brltty/files/brltty-5.2-sysmacros.patch
@@ -1,24 +1,20 @@
---- a/Programs/system_linux.c 2017-12-02 12:54:34.098643832 -0800
-+++ b/Programs/system_linux.c 2017-12-02 12:59:00.683592161 -0800
-@@ -24,6 +24,9 @@
+--- a/Programs/system_linux.c
++++ b/Programs/system_linux.c
+@@ -24,6 +24,7 @@
#include <fcntl.h>
#include <sys/ioctl.h>
#include <sys/stat.h>
-+#ifdef HAVE_SYS_SYSMACROS_H
-+#include <sys/sysmacros.h> /* major() w/newer glibc */
-+#endif
++#include <sys/sysmacros.h>
#include "log.h"
#include "file.h"
---- a/Programs/brlapi_client.c 2017-12-02 14:29:37.524205316 -0800
-+++ b/Programs/brlapi_client.c 2017-12-02 14:32:45.679948051 -0800
-@@ -73,6 +73,9 @@
+--- a/Programs/brlapi_client.c
++++ b/Programs/brlapi_client.c
+@@ -73,6 +73,7 @@
#include <linux/tty.h>
#include <linux/vt.h>
#define MAXIMUM_VIRTUAL_CONSOLE MAX_NR_CONSOLES
-+#ifdef HAVE_SYS_SYSMACROS_H
-+#include <sys/sysmacros.h> /* major() w/newer glibc */
-+#endif
++#include <sys/sysmacros.h>
#endif /* linux */
#ifdef __OpenBSD__
next reply other threads:[~2017-12-04 10:27 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-04 10:26 David Seifert [this message]
-- strict thread matches above, loose matches on Subject: below --
2023-10-10 5:44 [gentoo-commits] repo/gentoo:master commit in: app-accessibility/brltty/files/, app-accessibility/brltty/ Sam James
2023-06-30 11:50 David Seifert
2023-01-14 13:55 Sam James
2020-08-13 20:25 Michał Górny
2019-03-22 14:59 Lars Wendler
2018-03-26 20:19 Aaron Bauman
2016-11-28 9:21 Sergei Trofimovich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1512383112.9972fed3f21b47e7315949e39f762d9b209c1fd5.soap@gentoo \
--to=soap@gentoo.org \
--cc=gentoo-commits@lists.gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox