unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#33505] Patch: Fix aegisub on core-updates
@ 2018-11-25 19:39 Björn Höfling
  2018-11-25 20:35 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Björn Höfling @ 2018-11-25 19:39 UTC (permalink / raw)
  To: 33505


[-- Attachment #1.1: Type: text/plain, Size: 74 bytes --]

aegisub fails to compile with boost 68.

This little patch fixes it.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-aegisub-Fix-build.patch --]
[-- Type: text/x-patch, Size: 3997 bytes --]

From 297f7530262959f7389c4a776907dd13caec69ff Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20H=C3=B6fling?=
 <bjoern.hoefling@bjoernhoefling.de>
Date: Sun, 25 Nov 2018 20:33:13 +0100
Subject: [PATCH 1/1] gnu: aegisub: Fix build.

* gnu/packages/patches/aegisub-boost68.patch: New file
* gnu/local.mk: Add it.
* gnu/packages/video.scm (aegisub)[source]: Use patch.
---
 gnu/local.mk                               |  1 +
 gnu/packages/patches/aegisub-boost68.patch | 35 ++++++++++++++++++++++
 gnu/packages/video.scm                     |  3 +-
 3 files changed, 38 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/aegisub-boost68.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index 8b673fdd8..fda692b0f 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -574,6 +574,7 @@ dist_patch_DATA =						\
   %D%/packages/patches/aegis-test-fixup-1.patch            	\
   %D%/packages/patches/aegis-test-fixup-2.patch            	\
   %D%/packages/patches/aegisub-icu59-include-unistr.patch	\
+  %D%/packages/patches/aegisub-boost68.patch                    \
   %D%/packages/patches/agg-am_c_prototype.patch			\
   %D%/packages/patches/amule-crypto-6.patch			\
   %D%/packages/patches/ansible-wrap-program-hack.patch		\
diff --git a/gnu/packages/patches/aegisub-boost68.patch b/gnu/packages/patches/aegisub-boost68.patch
new file mode 100644
index 000000000..013721f3b
--- /dev/null
+++ b/gnu/packages/patches/aegisub-boost68.patch
@@ -0,0 +1,35 @@
+Source: https://git.archlinux.org/svntogit/community.git/plain/trunk/boost-1.68.patch?h=packages/aegisub
+From d8336d2fed73c72d1227b343d6acfb991bc1651b Mon Sep 17 00:00:00 2001
+From: Jan Beich <jbeich@FreeBSD.org>
+Date: Mon, 9 Jul 2018 20:15:29 +0000
+Subject: [PATCH] Keep using std::distance after Boost 1.68
+
+src/search_replace_engine.cpp:256:14: error: call to
+      'distance' is ambiguous
+                                count += distance(
+                                         ^~~~~~~~
+/usr/include/c++/v1/iterator:511:1: note: candidate function [with _InputIter =
+      boost::u32regex_iterator<std::__1::__wrap_iter<const char *> >]
+distance(_InputIter __first, _InputIter __last)
+^
+/usr/local/include/boost/iterator/distance.hpp:49:9: note: candidate function [with SinglePassIterator =
+      boost::u32regex_iterator<std::__1::__wrap_iter<const char *> >]
+        distance(SinglePassIterator first, SinglePassIterator last)
+        ^
+---
+ src/search_replace_engine.cpp | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/search_replace_engine.cpp b/src/search_replace_engine.cpp
+index 594c21e5e..14c71680d 100644
+--- a/src/search_replace_engine.cpp
++++ b/src/search_replace_engine.cpp
+@@ -253,7 +253,7 @@ bool SearchReplaceEngine::ReplaceAll() {
+ 			if (MatchState ms = matches(&diag, 0)) {
+ 				auto& diag_field = diag.*get_dialogue_field(settings.field);
+ 				std::string const& text = diag_field.get();
+-				count += distance(
++				count += std::distance(
+ 					boost::u32regex_iterator<std::string::const_iterator>(begin(text), end(text), *ms.re),
+ 					boost::u32regex_iterator<std::string::const_iterator>());
+ 				diag_field = u32regex_replace(text, *ms.re, settings.replace_with);
diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 8a5e730bd..3b2a37112 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -2757,7 +2757,8 @@ programmers to access a standard API to open and decompress media files.")
               (sha256
                (base32
                 "11b83qazc8h0iidyj1rprnnjdivj1lpphvpa08y53n42bfa36pn5"))
-              (patches (search-patches "aegisub-icu59-include-unistr.patch"))))
+              (patches (search-patches "aegisub-icu59-include-unistr.patch"
+                                       "aegisub-boost68.patch"))))
     (build-system gnu-build-system)
     (arguments
      `(#:configure-flags
-- 
2.19.1


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

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

* [bug#33505] Patch: Fix aegisub on core-updates
  2018-11-25 19:39 [bug#33505] Patch: Fix aegisub on core-updates Björn Höfling
@ 2018-11-25 20:35 ` Ludovic Courtès
  2018-11-25 22:30   ` bug#33505: " Björn Höfling
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2018-11-25 20:35 UTC (permalink / raw)
  To: Björn Höfling; +Cc: 33505

Hi Björn,

Björn Höfling <bjoern.hoefling@bjoernhoefling.de> skribis:

> From 297f7530262959f7389c4a776907dd13caec69ff Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Bj=C3=B6rn=20H=C3=B6fling?=
>  <bjoern.hoefling@bjoernhoefling.de>
> Date: Sun, 25 Nov 2018 20:33:13 +0100
> Subject: [PATCH 1/1] gnu: aegisub: Fix build.
>
> * gnu/packages/patches/aegisub-boost68.patch: New file
> * gnu/local.mk: Add it.
> * gnu/packages/video.scm (aegisub)[source]: Use patch.

LGTM, thank you!

Ludo’.

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

* bug#33505: Patch: Fix aegisub on core-updates
  2018-11-25 20:35 ` Ludovic Courtès
@ 2018-11-25 22:30   ` Björn Höfling
  0 siblings, 0 replies; 3+ messages in thread
From: Björn Höfling @ 2018-11-25 22:30 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 33505-close

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

On Sun, 25 Nov 2018 21:35:06 +0100
ludo@gnu.org (Ludovic Courtès) wrote:

> Hi Björn,
> 
> Björn Höfling <bjoern.hoefling@bjoernhoefling.de> skribis:
> 
> > From 297f7530262959f7389c4a776907dd13caec69ff Mon Sep 17 00:00:00
> > 2001 From: =?UTF-8?q?Bj=C3=B6rn=20H=C3=B6fling?=
> >  <bjoern.hoefling@bjoernhoefling.de>
> > Date: Sun, 25 Nov 2018 20:33:13 +0100
> > Subject: [PATCH 1/1] gnu: aegisub: Fix build.
> >
> > * gnu/packages/patches/aegisub-boost68.patch: New file
> > * gnu/local.mk: Add it.
> > * gnu/packages/video.scm (aegisub)[source]: Use patch.  
> 
> LGTM, thank you!
> 
> Ludo’.

Thanks for the review, pushed as

644af6c87092936e8a556da192ee195db58d65b3

Björn

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

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

end of thread, other threads:[~2018-11-25 22:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-25 19:39 [bug#33505] Patch: Fix aegisub on core-updates Björn Höfling
2018-11-25 20:35 ` Ludovic Courtès
2018-11-25 22:30   ` bug#33505: " Björn Höfling

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).