all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: gstreamer: Add search path specification for 'GST_PLUGIN_SYSTEM_PATH'.
@ 2015-06-24 15:11 宋文武
  2015-06-24 21:12 ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: 宋文武 @ 2015-06-24 15:11 UTC (permalink / raw)
  To: guix-devel

This reverts commit e53fc0c8a, which add it wrongly to gst-plugins-base.

* gnu/packages/gstreamer.scm (gstreamer)[native-search-paths]: New field.
  (gst-plugins-base)[native-search-paths]: Remove.
---
 gnu/packages/gstreamer.scm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 24dc41b..1832909 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -109,6 +109,10 @@ arrays of data.")
        ("perl" ,perl)
        ("pkg-config" ,pkg-config)
        ("python-wrapper" ,python-wrapper)))
+    (native-search-paths
+     (list (search-path-specification
+            (variable "GST_PLUGIN_SYSTEM_PATH")
+            (files '("lib/gstreamer-1.0")))))
     (home-page "http://gstreamer.freedesktop.org/")
     (synopsis "Multimedia library")
     (description
@@ -171,10 +175,6 @@ This package provides the core library and elements.")
           ;; for g-ir-scanner.
           (setenv "CC" "gcc"))
         %standard-phases)))
-    (native-search-paths
-     (list (search-path-specification
-            (variable "GST_PLUGIN_SYSTEM_PATH")
-            (files '("lib/gstreamer-1.0")))))
     (home-page "http://gstreamer.freedesktop.org/")
     (synopsis
      "Plugins for the GStreamer multimedia library")
-- 
2.2.1

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

* Re: [PATCH] gnu: gstreamer: Add search path specification for 'GST_PLUGIN_SYSTEM_PATH'.
  2015-06-24 15:11 [PATCH] gnu: gstreamer: Add search path specification for 'GST_PLUGIN_SYSTEM_PATH' 宋文武
@ 2015-06-24 21:12 ` Ludovic Courtès
  2015-06-25  2:06   ` 宋文武
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2015-06-24 21:12 UTC (permalink / raw)
  To: 宋文武; +Cc: guix-devel

宋文武 <iyzsong@gmail.com> skribis:

> This reverts commit e53fc0c8a, which add it wrongly to gst-plugins-base.
>
> * gnu/packages/gstreamer.scm (gstreamer)[native-search-paths]: New field.
>   (gst-plugins-base)[native-search-paths]: Remove.

LGTM!

I wonder, why is there the word “SYSTEM” in this variable name?  Does
that means there also exists GST_PLUGIN_PATH?

Thanks,
Ludo’.

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

* Re: [PATCH] gnu: gstreamer: Add search path specification for 'GST_PLUGIN_SYSTEM_PATH'.
  2015-06-24 21:12 ` Ludovic Courtès
@ 2015-06-25  2:06   ` 宋文武
  2015-06-25  9:26     ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: 宋文武 @ 2015-06-25  2:06 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

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

> 宋文武 <iyzsong@gmail.com> skribis:
>
>> This reverts commit e53fc0c8a, which add it wrongly to gst-plugins-base.
>>
>> * gnu/packages/gstreamer.scm (gstreamer)[native-search-paths]: New field.
>>   (gst-plugins-base)[native-search-paths]: Remove.
>
> LGTM!
>
> I wonder, why is there the word “SYSTEM” in this variable name?  Does
> that means there also exists GST_PLUGIN_PATH?
Yes, you're right.  It has:

  GST_PLUGIN_SYSTEM_PATH, GST_PLUGIN_SYSTEM_PATH_1_0
  GST_PLUGIN_PATH, GST_PLUGIN_PATH_1_0

I think any one is sufficent.

http://gstreamer.freedesktop.org/data/doc/gstreamer/head/gstreamer/html/gst-running.html

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

* Re: [PATCH] gnu: gstreamer: Add search path specification for 'GST_PLUGIN_SYSTEM_PATH'.
  2015-06-25  2:06   ` 宋文武
@ 2015-06-25  9:26     ` Ludovic Courtès
  0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2015-06-25  9:26 UTC (permalink / raw)
  To: 宋文武; +Cc: guix-devel

宋文武 <iyzsong@gmail.com> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> 宋文武 <iyzsong@gmail.com> skribis:
>>
>>> This reverts commit e53fc0c8a, which add it wrongly to gst-plugins-base.
>>>
>>> * gnu/packages/gstreamer.scm (gstreamer)[native-search-paths]: New field.
>>>   (gst-plugins-base)[native-search-paths]: Remove.
>>
>> LGTM!
>>
>> I wonder, why is there the word “SYSTEM” in this variable name?  Does
>> that means there also exists GST_PLUGIN_PATH?
> Yes, you're right.  It has:
>
>   GST_PLUGIN_SYSTEM_PATH, GST_PLUGIN_SYSTEM_PATH_1_0
>   GST_PLUGIN_PATH, GST_PLUGIN_PATH_1_0
>
> I think any one is sufficent.
>
> http://gstreamer.freedesktop.org/data/doc/gstreamer/head/gstreamer/html/gst-running.html

OK, thanks.

Ludo’.

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

end of thread, other threads:[~2015-06-25  9:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-24 15:11 [PATCH] gnu: gstreamer: Add search path specification for 'GST_PLUGIN_SYSTEM_PATH' 宋文武
2015-06-24 21:12 ` Ludovic Courtès
2015-06-25  2:06   ` 宋文武
2015-06-25  9:26     ` Ludovic Courtès

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.