From: "Sam James" <sam@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: app-i18n/jfbterm/files/, app-i18n/jfbterm/
Date: Wed, 12 Feb 2025 18:16:25 +0000 (UTC) [thread overview]
Message-ID: <1739384142.77728d89f28570c51693d1f0278c88a711bf29bb.sam@gentoo> (raw)
commit: 77728d89f28570c51693d1f0278c88a711bf29bb
Author: NHOrus <jy6x2b32pie9 <AT> yahoo <DOT> com>
AuthorDate: Wed Feb 12 17:34:34 2025 +0000
Commit: Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Wed Feb 12 18:15:42 2025 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=77728d89
app-i18n/jfbterm: update EAPI 7 -> 8, fix home URI, port to C23
Changes call to time() to call to gettimeofday() according to utmp man page
Removes font installation from makefile and delegates to portage machinery
so font cache gets updated
Bug: https://bugs.gentoo.org/835793
Closes: https://bugs.gentoo.org/919295
Signed-off-by: NHOrus <jy6x2b32pie9 <AT> yahoo.com>
Closes: https://github.com/gentoo/gentoo/pull/40547
Signed-off-by: Sam James <sam <AT> gentoo.org>
app-i18n/jfbterm/files/jfbterm-0.4.7-fonts.patch | 15 ++++++
.../files/jfbterm-0.4.7-gettimeoftheday.patch | 43 +++++++++++++++
app-i18n/jfbterm/jfbterm-0.4.7-r5.ebuild | 63 ++++++++++++++++++++++
3 files changed, 121 insertions(+)
diff --git a/app-i18n/jfbterm/files/jfbterm-0.4.7-fonts.patch b/app-i18n/jfbterm/files/jfbterm-0.4.7-fonts.patch
new file mode 100644
index 000000000000..264c0499d573
--- /dev/null
+++ b/app-i18n/jfbterm/files/jfbterm-0.4.7-fonts.patch
@@ -0,0 +1,15 @@
+Use portage machinery instead of package makefiles to install fonts
+This does fc-cache, unlike before
+https://bugs.gentoo.org/835793
+--- a/Makefile.am
++++ b/Makefile.am
+@@ -22,9 +22,7 @@
+
+ install-exec-hook:
+ chown root:utmp $(DESTDIR)$(bindir)/jfbterm
+- chmod ug+s $(DESTDIR)$(bindir)/jfbterm
+
+ install-data-local:
+ $(INSTALL_DATA) $(srcdir)/$(JFBTERM_CONFIG_FILE) $(DESTDIR)$(sysconfdir)/$(JFBTERM_CONFIG_FILE)
+- $(INSTALL_DATA) $(srcdir)/fonts/*.pcf.gz $(DESTDIR)$(datadir)/fonts/jfbterm
+
diff --git a/app-i18n/jfbterm/files/jfbterm-0.4.7-gettimeoftheday.patch b/app-i18n/jfbterm/files/jfbterm-0.4.7-gettimeoftheday.patch
new file mode 100644
index 000000000000..f7da1afb8010
--- /dev/null
+++ b/app-i18n/jfbterm/files/jfbterm-0.4.7-gettimeoftheday.patch
@@ -0,0 +1,43 @@
+Man 5 utmp has instructions how to replace time with gettimeofday in utmp.
+Use them.
+https://bugs.gentoo.org/919295
+--- a/term.c
++++ b/term.c
+@@ -248,6 +248,7 @@
+ struct utmp utmp;
+ struct passwd *pw;
+ char *tn;
++ struct timeval tv;
+
+ pw = getpwuid(util_getuid());
+ tn = rindex(p->name, '/') + 1;
+@@ -262,7 +263,9 @@
+ tn = p->name + 5;
+ strncpy(utmp.ut_line, tn, sizeof(utmp.ut_line));
+ strncpy(utmp.ut_user, pw->pw_name, sizeof(utmp.ut_user));
+- time(&(utmp.ut_time));
++ gettimeofday(&tv, NULL);
++ utmp.ut_tv.tv_sec = tv.tv_sec;
++ utmp.ut_tv.tv_usec = tv.tv_usec;
+ pututline(&utmp);
+ endutent();
+ }
+@@ -271,6 +274,7 @@
+ {
+ struct utmp utmp, *utp;
+ char *tn;
++ struct timeval tv;
+
+ tn = rindex(p->name, '/') + 4;
+ memset((char *)&utmp, 0, sizeof(utmp));
+@@ -281,7 +285,9 @@
+ utp->ut_type = DEAD_PROCESS;
+ memset(utp->ut_user, 0, sizeof(utmp.ut_user));
+ utp->ut_type = DEAD_PROCESS;
+- time(&(utp->ut_time));
++ gettimeofday(&tv, NULL);
++ utp->ut_tv.tv_sec = tv.tv_sec;
++ utp->ut_tv.tv_usec = tv.tv_usec;
+ pututline(utp);
+ endutent();
+ }
diff --git a/app-i18n/jfbterm/jfbterm-0.4.7-r5.ebuild b/app-i18n/jfbterm/jfbterm-0.4.7-r5.ebuild
new file mode 100644
index 000000000000..17e2545b6782
--- /dev/null
+++ b/app-i18n/jfbterm/jfbterm-0.4.7-r5.ebuild
@@ -0,0 +1,63 @@
+# Copyright 1999-2025 Gentoo Authors
+# Distributed under the terms of the GNU General Public License v2
+
+EAPI=8
+
+inherit autotools font
+
+DESCRIPTION="The J Framebuffer Terminal/Multilingual Enhancement with UTF-8 support"
+HOMEPAGE="https://osdn.net/projects/jfbterm/"
+SRC_URI="mirror://sourceforge.jp/${PN}/13501/${P}.tar.gz"
+
+LICENSE="BSD"
+SLOT="0"
+KEYWORDS="~amd64 ~ppc ~ppc64 ~sparc ~x86"
+IUSE="debug"
+
+# ncurses is runtime-onlu dependency, because program provides it's own terminfo
+# gzip needed for unpacking font at runtime
+RDEPEND="media-fonts/font-misc-misc
+ media-fonts/intlfonts
+ media-fonts/unifont
+ sys-libs/ncurses
+ app-alternatives/gzip"
+
+
+PATCHES=(
+ "${FILESDIR}"/${PN}-sigchld-debian.patch
+ "${FILESDIR}"/${PN}-no-kernel-headers.patch
+ "${FILESDIR}"/${PN}-gentoo.patch
+ "${FILESDIR}"/${PN}-wrong-inline-gcc5.patch
+ "${FILESDIR}"/${PN}-automake-1.13.patch
+ "${FILESDIR}"/"${P}"-fonts.patch
+ "${FILESDIR}"/"${P}"-gettimeoftheday.patch
+)
+
+FONT_S="${S}/fonts"
+FONT_SUFFIX="pcf.gz"
+
+src_prepare() {
+ default
+
+ eautoreconf
+}
+
+src_configure() {
+ econf $(use_enable debug)
+}
+
+src_install() {
+ dodir /etc
+ default
+
+ mv "${ED}"/etc/${PN}.conf{.sample,} || die
+
+ font_src_install
+
+ doman ${PN}.{1,conf.5}
+
+ # install example config files
+ docinto examples
+ dodoc ${PN}.conf.sample*
+ docompress -x /usr/share/doc/${PF}/examples
+}
next reply other threads:[~2025-02-12 18:16 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-12 18:16 Sam James [this message]
-- strict thread matches above, loose matches on Subject: below --
2021-10-07 14:13 [gentoo-commits] repo/gentoo:master commit in: app-i18n/jfbterm/files/, app-i18n/jfbterm/ Akinori Hattori
2017-02-12 11:36 Michael Palimaka
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=1739384142.77728d89f28570c51693d1f0278c88a711bf29bb.sam@gentoo \
--to=sam@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