unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / Atom feed
* [bug#48194] [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2.
@ 2021-05-03 15:20 Michael Rohleder
  2021-05-11 10:31 ` bug#48194: " Ludovic Courtès
  2021-05-11 10:51 ` [bug#48194] " Ludovic Courtès
  0 siblings, 2 replies; 5+ messages in thread
From: Michael Rohleder @ 2021-05-03 15:20 UTC (permalink / raw)
  To: 48194; +Cc: Michael Rohleder

* gnu/packages/video.scm (libmediainfo)[arguments]: Add configure-flags.
---
 gnu/packages/video.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 481ab46b6e..1052351357 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -3834,6 +3834,8 @@ practically any type of media.")
     (build-system gnu-build-system)
     (arguments
      '(#:tests? #f ; see above TODO
+       #:configure-flags
+       (list "--with-libcurl" "--with-libtinyxml2")
        #:phases
        ;; build scripts not in root of archive
        (modify-phases %standard-phases
-- 
2.31.1





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

* bug#48194: [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2.
  2021-05-03 15:20 [bug#48194] [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2 Michael Rohleder
@ 2021-05-11 10:31 ` Ludovic Courtès
  2021-05-11 10:51 ` [bug#48194] " Ludovic Courtès
  1 sibling, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2021-05-11 10:31 UTC (permalink / raw)
  To: Michael Rohleder; +Cc: 48194-done

Hi,

Michael Rohleder <mike@rohleder.de> skribis:

> * gnu/packages/video.scm (libmediainfo)[arguments]: Add configure-flags.

Applied, thanks!




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

* [bug#48194] [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2.
  2021-05-03 15:20 [bug#48194] [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2 Michael Rohleder
  2021-05-11 10:31 ` bug#48194: " Ludovic Courtès
@ 2021-05-11 10:51 ` Ludovic Courtès
  2021-05-11 12:14   ` Michael Rohleder
  1 sibling, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2021-05-11 10:51 UTC (permalink / raw)
  To: Michael Rohleder; +Cc: 48194

Michael Rohleder <mike@rohleder.de> skribis:

> * gnu/packages/video.scm (libmediainfo)[arguments]: Add configure-flags.

Actually no, this one appears to break ‘mediainfo’ as I wrote in my
other message.




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

* [bug#48194] [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2.
  2021-05-11 10:51 ` [bug#48194] " Ludovic Courtès
@ 2021-05-11 12:14   ` Michael Rohleder
  2021-05-11 12:30     ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Rohleder @ 2021-05-11 12:14 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 48194


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

Ludovic Courtès <ludo@gnu.org> writes:

> Michael Rohleder <mike@rohleder.de> skribis:
>
>> * gnu/packages/video.scm (libmediainfo)[arguments]: Add configure-flags.
>
> Actually no, this one appears to break ‘mediainfo’ as I wrote in my
> other message.

The consumer (mediainfo) needs curl and tinyxml2 inputs:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: Fix usage of libcurl and tinyxml2. --]
[-- Type: text/x-patch, Size: 1250 bytes --]

>From 6612d1189263cc4670b820e26f5e75d4df9c68b0 Mon Sep 17 00:00:00 2001
From: Michael Rohleder <mike@rohleder.de>
Date: Tue, 11 May 2021 14:05:13 +0200
Subject: [PATCH] gnu: mediainfo: Fix usage of libcurl and tinyxml2.

* gnu/packages/video.scm (mediainfo): Fix usage of libcurl and tinyxml2.
---
 gnu/packages/video.scm | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 481ab46b6e..630edf49b6 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -3834,6 +3834,8 @@ practically any type of media.")
     (build-system gnu-build-system)
     (arguments
      '(#:tests? #f ; see above TODO
+       #:configure-flags
+       (list "--with-libcurl" "--with-libtinyxml2")
        #:phases
        ;; build scripts not in root of archive
        (modify-phases %standard-phases
@@ -3888,6 +3890,8 @@ MPEG-2, MPEG-4, DVD (VOB)...
        ("libtool" ,libtool)
        ("pkg-config" ,pkg-config)
        ("zlib" ,zlib)
+       ("curl" ,curl) ;; In Requires.private of libmediainfo pkg-config files.
+       ("tinyxml2" ,tinyxml2)
        ("libmediainfo" ,libmediainfo)
        ("libzen" ,libzen)))
     (build-system gnu-build-system)
-- 
2.31.1


[-- Attachment #1.3: Type: text/plain, Size: 146 bytes --]


-- 
A million monkeys pounding on a million typewriters will never produce 
Shakespeare. The experiment's been tried; we got USENET instead.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]

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

* [bug#48194] [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2.
  2021-05-11 12:14   ` Michael Rohleder
@ 2021-05-11 12:30     ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2021-05-11 12:30 UTC (permalink / raw)
  To: Michael Rohleder; +Cc: 48194

Michael Rohleder <mike@rohleder.de> skribis:

>>From 6612d1189263cc4670b820e26f5e75d4df9c68b0 Mon Sep 17 00:00:00 2001
> From: Michael Rohleder <mike@rohleder.de>
> Date: Tue, 11 May 2021 14:05:13 +0200
> Subject: [PATCH] gnu: mediainfo: Fix usage of libcurl and tinyxml2.
>
> * gnu/packages/video.scm (mediainfo): Fix usage of libcurl and tinyxml2.

Please mention [arguments] and [propagated-inputs].


>  gnu/packages/video.scm | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index 481ab46b6e..630edf49b6 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -3834,6 +3834,8 @@ practically any type of media.")
>      (build-system gnu-build-system)
>      (arguments
>       '(#:tests? #f ; see above TODO
> +       #:configure-flags
> +       (list "--with-libcurl" "--with-libtinyxml2")
>         #:phases
>         ;; build scripts not in root of archive
>         (modify-phases %standard-phases
> @@ -3888,6 +3890,8 @@ MPEG-2, MPEG-4, DVD (VOB)...
>         ("libtool" ,libtool)
>         ("pkg-config" ,pkg-config)
>         ("zlib" ,zlib)
> +       ("curl" ,curl) ;; In Requires.private of libmediainfo pkg-config files.
> +       ("tinyxml2" ,tinyxml2)

Shouldn’t we also remove them from ‘native-inputs’, which looks bogus
anyway?

Thanks,
Ludo’.




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

end of thread, other threads:[~2021-05-11 12:31 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-03 15:20 [bug#48194] [PATCH] gnu: libmediainfo: Fix usage of libcurl and libtinyxml2 Michael Rohleder
2021-05-11 10:31 ` bug#48194: " Ludovic Courtès
2021-05-11 10:51 ` [bug#48194] " Ludovic Courtès
2021-05-11 12:14   ` Michael Rohleder
2021-05-11 12:30     ` Ludovic Courtès

unofficial mirror of guix-patches@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-patches/1 guix-patches/git/1.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-patches guix-patches/ https://yhetil.org/guix-patches \
		guix-patches@gnu.org
	public-inbox-index guix-patches

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.patches


AGPL code for this site: git clone http://ou63pmih66umazou.onion/public-inbox.git