Github messages for voidlinux
 help / color / mirror / Atom feed
From: hazen2215 <hazen2215@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] uim: update to 1.8.9.
Date: Sat, 05 Oct 2024 11:29:01 +0200	[thread overview]
Message-ID: <20241005092901.A588528856@inbox.vuxu.org> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-49125@inbox.vuxu.org>

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

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

https://github.com/hazen2215/void-packages uim
https://github.com/void-linux/void-packages/pull/49125

uim: update to 1.8.9.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### 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**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
#### Local build testing
- I built this PR locally for my native architecture, (x86_64-musl)


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

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

From af1a6d5c6016cf06f25d0356d1b2e69b12ac077c Mon Sep 17 00:00:00 2001
From: hazen2215 <haz@disroot.org>
Date: Tue, 5 Mar 2024 20:58:40 +0900
Subject: [PATCH] uim: update to 1.8.9.

---
 .../replace_obsolete_since_emacs22.1.patch    | 38 ---------
 srcpkgs/uim/patches/uim-kde.patch             | 80 +++++++++++++++++++
 srcpkgs/uim/template                          |  6 +-
 3 files changed, 83 insertions(+), 41 deletions(-)
 delete mode 100644 srcpkgs/uim/patches/replace_obsolete_since_emacs22.1.patch
 create mode 100644 srcpkgs/uim/patches/uim-kde.patch

diff --git a/srcpkgs/uim/patches/replace_obsolete_since_emacs22.1.patch b/srcpkgs/uim/patches/replace_obsolete_since_emacs22.1.patch
deleted file mode 100644
index 499549c3f736f3..00000000000000
--- a/srcpkgs/uim/patches/replace_obsolete_since_emacs22.1.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-From 164e2eb050b5fec25033124834cf49ea1a7d8cbb Mon Sep 17 00:00:00 2001
-From: multiSnow <infinity.blick.winkel@gmail.com>
-Date: Thu, 7 Feb 2019 01:03:47 +0800
-Subject: [PATCH] replace process-kill-without-query with
- set-process-query-on-exit-flag
-
-see https://www.gnu.org/software/emacs/news/NEWS.22.1 and https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=f1c48b0ec521744826ed43ae27eed0e152c472bf
----
- emacs/uim-helper.el | 2 +-
- emacs/uim.el        | 2 +-
- 2 files changed, 2 insertions(+), 2 deletions(-)
-
-diff --git emacs/uim-helper.el emacs/uim-helper.el
-index e86e32d06..19ecd4538 100644
---- a/emacs/uim-helper.el
-+++ b/emacs/uim-helper.el
-@@ -106,7 +106,7 @@
-     (if (not proc)
- 	(error "uim.el: Couldn't invoke uim-el-helper-agent."))
- 
--    (process-kill-without-query proc)
-+    (set-process-query-on-exit-flag proc nil)
- 
-     ;; wait "OK"
-     (let ((patience uim-startup-timeout) (ok nil))
-diff --git emacs/uim.el emacs/uim.el
-index 9110c3cb6..7e1fadbba 100644
---- a/emacs/uim.el
-+++ b/emacs/uim.el
-@@ -488,7 +488,7 @@
- 	(error "uim.el: Couldn't invoke uim-el-agent."))
-     
-     ;; don't ask kill
--    (process-kill-without-query proc)
-+    (set-process-query-on-exit-flag proc nil)
- 
-     ;; wait "OK"
-     (let ((patience uim-startup-timeout) (ok nil))
diff --git a/srcpkgs/uim/patches/uim-kde.patch b/srcpkgs/uim/patches/uim-kde.patch
new file mode 100644
index 00000000000000..dfac6b5564352a
--- /dev/null
+++ b/srcpkgs/uim/patches/uim-kde.patch
@@ -0,0 +1,80 @@
+--- a/configure.ac
++++ b/configure.ac
+@@ -1449,6 +1449,28 @@
+ 
+ AM_CONDITIONAL(KDE4_APPLET, test "x$enable_kde4_applet" = xyes)
+ 
++dnl ****************************
++dnl *** test for KDE5 applet ***
++dnl ****************************
++AC_ARG_ENABLE(kde5-applet,
++  AC_HELP_STRING([--disable-kde5-applet],
++    [do not build uim KDE Plasma 5 widget]),
++  [],
++  [enable_kde5_applet=yes])
++
++if test "x$enable_kde5_applet" != xno; then
++  if test "x$with_qt5" != xyes; then
++    enable_kde5_applet=no
++  fi
++fi
++
++if test "x$enable_kde5_applet" = xyes && test "x$CMAKE" = xno; then
++  AC_MSG_WARN([no CMake found])
++  enable_kde5_applet=no
++fi
++
++AM_CONDITIONAL(KDE5_APPLET, test "x$enable_kde5_applet" = xyes)
++
+ AC_ARG_ENABLE(pref,
+   AC_HELP_STRING([--disable-pref],
+     [do not build graphical utility to edit user settings]),
+@@ -1883,7 +1905,6 @@
+   cmake_option="-DCMAKE_BUILD_TYPE=${cmake_build_type}"
+   cmake_option="${cmake_option} -DCMAKE_INSTALL_PREFIX=${prefix}"
+   cmake_option="${cmake_option} -DCMAKE_SKIP_RPATH=true"
+-  cmake_option="${cmake_option} -DKDE4_USE_ALWAYS_FULL_RPATH=false"
+ fi
+ 
+ if test "x$with_qt4" = xyes; then
+@@ -1948,12 +1969,15 @@
+     qt5_immodule_option="DEFINES+=QT5_IMMODULE"
+   fi
+   set -e
+-  # FIXME: move CMAKE_CXX_FLAGS into cmake_option
+-  ${CMAKE} \
+-    -B ${ac_abs_top_builddir}/qt5/applet.cmake \
+-    -S ${ac_abs_top_srcdir}/qt5/applet \
+-    -DCMAKE_CXX_FLAGS="${CXXFLAGS} ${X_CFLAGS}" \
+-    ${cmake_option}
++  # Generate a Makefile for KDE5 applet by cmake
++  if test "x$enable_kde5_applet" = xyes; then
++    # FIXME: move CMAKE_CXX_FLAGS into cmake_option
++    ${CMAKE} \
++      -B ${ac_abs_top_builddir}/qt5/applet.cmake \
++      -S ${ac_abs_top_srcdir}/qt5/applet \
++      -DCMAKE_CXX_FLAGS="${CXXFLAGS} ${X_CFLAGS}" \
++      ${cmake_option}
++  fi
+   ${QMAKE5} -o ${ac_abs_top_builddir}/qt5/candwin/Makefile.qmake \
+     ${ac_abs_top_builddir}/qt5/candwin/uim-candwin-qt5.pro
+   ${QMAKE5} -o ${ac_abs_top_builddir}/qt5/chardict/Makefile.qmake \
+@@ -2007,6 +2031,7 @@
+    Qt5 immodule    : ${with_qt5_immodule}
+    KDE3 Applet     : ${enable_kde_applet}
+    KDE4 Applet     : ${enable_kde4_applet}
++   KDE5 Applet     : ${enable_kde5_applet}
+    FEP             : ${enable_fep}
+    Emacs           : ${enable_emacs}
+    XIM             : ${use_xim}
+--- a/qt5/applet/Makefile.am
++++ b/qt5/applet/Makefile.am
+@@ -6,6 +6,8 @@
+ 	plasmoid
+ 
+ if QT5
++if KDE5_APPLET
+ all clean install uninstall:
+ 	$(MAKE) $(AM_MAKEFLAGS) -C ../applet.cmake $@
+ endif
++endif
diff --git a/srcpkgs/uim/template b/srcpkgs/uim/template
index 1a782b680f78ca..1615a2d8025d34 100644
--- a/srcpkgs/uim/template
+++ b/srcpkgs/uim/template
@@ -1,7 +1,7 @@
 # Template file for 'uim'
 pkgname=uim
-version=1.8.8
-revision=5
+version=1.8.9
+revision=1
 build_style=gnu-configure
 build_helper=qmake
 configure_args="--enable-pref --enable-fep --with-gtk2 --with-gtk3
@@ -16,7 +16,7 @@ maintainer="Kazuho Sakoda <hyonhyoro.kazuho@gmail.com>"
 license="BSD-3-Clause"
 homepage="https://github.com/uim/uim/wiki"
 distfiles="https://github.com/uim/uim/releases/download/${version}/uim-${version}.tar.bz2"
-checksum=34599bbcc4e5ab87832370763e38be5100984a64237555e9234a1ea225a0fadc
+checksum=dbbd983768bf748449551644f330dbebe859bfeb6f024fea6697ac75131c7aa4
 
 CFLAGS="-fcommon"
 

  parent reply	other threads:[~2024-10-05  9:29 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-05 12:01 [PR PATCH] " hazen2215
2024-06-04  1:48 ` github-actions
2024-06-04  8:45 ` [PR PATCH] [Updated] " hazen2215
2024-07-06 11:11 ` hazen2215
2024-10-05  1:58 ` github-actions
2024-10-05  9:29 ` hazen2215 [this message]
2024-11-01  3:10 ` [PR PATCH] [Merged]: " classabbyamp

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=20241005092901.A588528856@inbox.vuxu.org \
    --to=hazen2215@users.noreply.github.com \
    --cc=ml@inbox.vuxu.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;
as well as URLs for NNTP newsgroup(s).