Re: [PATCH] gnu: vlc: Add inputs.

2016-09-09 Thread Alex Vong 
David Craven  writes:

>> Sorry, I don't understand what do you mean by libinput-minimal? I didn't
>> add any packages with a "-minial" suffice. As for the necessities of
>> those dependencies, those dependencies are added one by one. The
>> procedure is as followed:
>> 1. I build from source and check for something like "WARNING: XXX not found"
>> 2. Add dependency(s) to silent the warning
>> 3. Repeat
>
> You should get the same warning for libinput too...

Hmm, I don't get the waraning for libinput for some reason. The complete
list of warnings is as followed:


checking for LUA... no
configure: WARNING: No package 'lua5.2' found, trying lua 5.1 instead
checking for LUA... no
configure: WARNING: No package 'lua5.1' found, trying lua >= 5.1 instead
checking for LUA... no
configure: WARNING: No package 'lua' found, trying manual detection instead
checking lua.h usability... yes
checking lua.h presence... yes
checking for lua.h... yes
checking lauxlib.h usability... yes
checking lauxlib.h presence... yes
checking for lauxlib.h... yes
checking lualib.h usability... yes
checking lualib.h presence... yes
checking for lualib.h... yes
checking for luaL_newstate in -llua5.2 ... no
checking for luaL_newstate in -llua5.1 ... no
checking for luaL_newstate in -llua51 ... no
checking for luaL_newstate in -llua ... yes
checking for luac... 
/gnu/store/awvlsvab8g2cjxz214gn8ddz2y61al87-lua-5.1.5/bin/luac
checking for LIVE555... no
configure: WARNING: No package 'live555' found.
checking for live555 version 1324598400 or later... no
configure: WARNING: live555 is missing or its installed version is too old:
Version 2011.12.23 or later is required to proceed.
You can get an updated one from http://www.live555.com/liveMedia .
checking for DC1394... no
configure: WARNING: Library libdc1394-2 >= 2.1.0 needed for dc1394 was not found
checking for DV1394... yes
checking for LINSYS_SDI... no
configure: WARNING: No package 'zvbi-0.2' found.
checking for DVDREAD... yes
checking for DVDNAV... yes
checking for dvdnav_stream_cb in dvdnav/dvdnav.h... yes
checking for BLURAY... yes
checking for OPENCV... no
configure: WARNING: Library opencv > 2.0 needed for opencv was not found
checking for SMBCLIENT... yes
checking for SFTP... yes
checking linux/videodev2.h usability... yes
checking linux/videodev2.h presence... yes
checking for linux/videodev2.h... yes
checking sys/videoio.h usability... no
checking sys/videoio.h presence... no
checking for sys/videoio.h... no
checking DeckLinkAPIDispatch.cpp usability... no
checking DeckLinkAPIDispatch.cpp presence... no
checking for DeckLinkAPIDispatch.cpp... no
configure: WARNING: Blackmagic DeckLink SDI include files not found
checking for GNOMEVFS... no
configure: WARNING: Library gnome-vfs-2.0 needed for gnomevfs was not found
checking for cdrom_msf0 in linux/cdrom.h... yes
checking for scsireq in sys/scsiio.h... no
checking for ioc_toc_header in sys/cdio.h... no
checking for LIBCDDB... yes
checking for Linux DVB version 5.1... yes
checking for VNC... no
configure: WARNING: Library libvncclient >= 0.9.9 needed for vnc was not found
checking for FREERDP... no
configure: WARNING: Library freerdp >= 1.0.1 needed for freerdp was not found
checking for ASDCP... no
checking for DVBPSI... no
checking gme/gme.h usability... no
checking gme/gme.h presence... no
checking for gme/gme.h... no
checking for SID... no
configure: WARNING: No package 'libsidplay2' found (required for sid).
checking for OGG... yes
checking for LIBVORBIS... yes
checking for OGG... yes
checking for MUX_OGG... yes
checking for SHOUT... yes
checking ebml/EbmlVersion.h usability... no
checking ebml/EbmlVersion.h presence... no
checking for ebml/EbmlVersion.h... no
checking for LIBMODPLUG... yes
checking mpc/mpcdec.h usability... no
checking mpc/mpcdec.h presence... no
checking for mpc/mpcdec.h... no
checking mpcdec/mpcdec.h usability... no
checking mpcdec/mpcdec.h presence... no
checking for mpcdec/mpcdec.h... no
checking for SHINE... no
configure: WARNING: Library shine >= 3.0.0 needed for shine was not found
checking interface/mmal/mmal.h usability... no
checking interface/mmal/mmal.h presence... no
checking for interface/mmal/mmal.h... no
configure: WARNING: Cannot find development headers for mmal...
checking for libcrystalhd/libcrystalhd_if.h... no
checking mad.h usability... yes
checking mad.h presence... yes
checking for mad.h... yes
checking for mad_bit_init in -lmad... yes
checking for libavutil variant... ffmpeg
checking for GST_APP... yes
checking for GST_VIDEO... yes
checking for AVCODEC... yes
checking libavcodec/avcodec.h usability... yes
checking libavcodec/avcodec.h presence... yes
checking for libavcodec/avcodec.h... yes
checking libavutil/avutil.h usability... yes
checking libavutil/avutil.h presence... yes
checking for libavutil/avutil.h... yes
checking for LIBVA_DRM... yes
checking for LIBVA_X11... yes
checking libavcodec/vaapi.h usability... yes
checking libavcodec/vaapi.h presence... yes

Re: [PATCH] gnu: vlc: Add inputs.

2016-09-06 Thread David Craven 
> Sorry, I don't understand what do you mean by libinput-minimal? I didn't
> add any packages with a "-minial" suffice. As for the necessities of
> those dependencies, those dependencies are added one by one. The
> procedure is as followed:
> 1. I build from source and check for something like "WARNING: XXX not found"
> 2. Add dependency(s) to silent the warning
> 3. Repeat

You should get the same warning for libinput too...

Re: [PATCH] gnu: vlc: Add inputs.

2016-09-06 Thread Alex Vong 
Hi David and Efraim,

Thanks for the review.

Efraim Flashner  writes:

> On Sat, Sep 03, 2016 at 06:15:48PM +0200, David Craven wrote:
>> Hi Alex,
>> 
>> I'm not sure we need all those dependencies =) Can you explain what
>> they are for? If you are adding dependencies to vlc why did you leave
>> libinput-minimal off the list? ;)
>>
Sorry, I don't understand what do you mean by libinput-minimal? I didn't
add any packages with a "-minial" suffice. As for the necessities of
those dependencies, those dependencies are added one by one. The
procedure is as followed:
1. I build from source and check for something like "WARNING: XXX not found"
2. Add dependency(s) to silent the warning
3. Repeat


>> > 3. Our vlc was built with fast-math disabled as explained in the
>> > comment. However, this is actually not necessary, vlc would built with
>> > '-ffast-math -fno-finite-math-only'.
>> 
>> This should probably be a separate patch, and will probably have a
>> easier path to master...
>>
Sure!

>> Thanks!
>> 
>
> also check the closure size with and without samba. A long time ago we
> removed samba from mpv and it drastically slimmed down the size of the
> closure. IIRC it meant we could no longer do `mpv smb://...'

I haven't checked that. I will check it now.

Cheers,
Alex

Re: [PATCH] gnu: vlc: Add inputs.

2016-09-05 Thread Efraim Flashner 
On Sat, Sep 03, 2016 at 06:15:48PM +0200, David Craven wrote:
> Hi Alex,
> 
> I'm not sure we need all those dependencies =) Can you explain what
> they are for? If you are adding dependencies to vlc why did you leave
> libinput-minimal off the list? ;)
> 
> > 3. Our vlc was built with fast-math disabled as explained in the
> > comment. However, this is actually not necessary, vlc would built with
> > '-ffast-math -fno-finite-math-only'.
> 
> This should probably be a separate patch, and will probably have a
> easier path to master...
> 
> Thanks!
> 

also check the closure size with and without samba. A long time ago we
removed samba from mpv and it drastically slimmed down the size of the
closure. IIRC it meant we could no longer do `mpv smb://...'

-- 
Efraim Flashner      אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted


signature.asc
Description: PGP signature

Re: [PATCH] gnu: vlc: Add inputs.

2016-09-03 Thread David Craven 
Hi Alex,

I'm not sure we need all those dependencies =) Can you explain what
they are for? If you are adding dependencies to vlc why did you leave
libinput-minimal off the list? ;)

> 3. Our vlc was built with fast-math disabled as explained in the
> comment. However, this is actually not necessary, vlc would built with
> '-ffast-math -fno-finite-math-only'.

This should probably be a separate patch, and will probably have a
easier path to master...

Thanks!

[PATCH] gnu: vlc: Add inputs.

2016-09-01 Thread Alex Vong 
Hi guix,

While test-driving vlc on my laptop, I notice our vlc isn't build with
subtitle support. So, I go ahead and add as many inputs as possible.

There are 3 points to pay attention:
1. Add liba52 as input and remove the old FIXME
2. Add a new FIXME since gnome-vfs is not detected by the vlc build system
3. Our vlc was built with fast-math disabled as explained in the
comment. However, this is actually not necessary, vlc would built with
'-ffast-math -fno-finite-math-only'.


>From 56ca3c305354522fdfb7fd9a7fe1e8d57f1a8d87 Mon Sep 17 00:00:00 2001
From: Alex Vong <alexvong1...@gmail.com>
Date: Tue, 30 Aug 2016 02:30:57 +0800
Subject: [PATCH] gnu: vlc: Add inputs.

* gnu/packages/video.scm (vlc)[inputs]: Add eudev, faad2, fluidsynth,
  gnome-vfs, gst-plugins-base, gtk+-2, jack-2, liba52, libass,
  libavc1394, libbluray, libcaca, libdca, libdvdnav, libdvdread,
  libmpeg2, libmodplug, libmtp, libnotify, libraw1394, libshout,
  librsvg, libupnp, libva, libvdpau, samba, taglib, twolame.
  [arguments]: Pass "CFLAGS=-ffast-math -fno-finite-math-only" and
  "CXXFLAGS=-ffast-math -fno-finite-math-only" to #:configure-flags.
---
 gnu/packages/video.scm | 42 ++
 1 file changed, 38 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 5c59ee2..da03b71 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -6,7 +6,7 @@
 ;;; Copyright © 2015, 2016 Efraim Flashner <efr...@flashner.co.il>
 ;;; Copyright © 2015 Andy Patterson <ajpat...@uwaterloo.ca>
 ;;; Copyright © 2015 Ricardo Wurmus <rek...@elephly.net>
-;;; Copyright © 2015 Alex Vong <alexvong1...@gmail.com>
+;;; Copyright © 2015, 2016 Alex Vong <alexvong1...@gmail.com>
 ;;; Copyright © 2016 Alex Griffin <a...@ajgrf.com>
 ;;; Copyright © 2016 Kei Kebreau <k...@openmailbox.org>
 ;;; Copyright © 2016 Dmitry Nikolaev <camelthe...@gmail.com>
@@ -60,9 +60,13 @@
   #:use-module (gnu packages gl)
   #:use-module (gnu packages glib)
   #:use-module (gnu packages guile)
+  #:use-module (gnu packages gnome)
   #:use-module (gnu packages gnupg)
+  #:use-module (gnu packages gstreamer)
   #:use-module (gnu packages gtk)
   #:use-module (gnu packages image)
+  #:use-module (gnu packages libupnp)
+  #:use-module (gnu packages libusb)
   #:use-module (gnu packages linux)
   #:use-module (gnu packages lua)
   #:use-module (gnu packages mp3)
@@ -74,6 +78,7 @@
   #:use-module (gnu packages pulseaudio)
   #:use-module (gnu packages python)
   #:use-module (gnu packages qt)
+  #:use-module (gnu packages samba)
   #:use-module (gnu packages sdl)
   #:use-module (gnu packages ssh)
   #:use-module (gnu packages texinfo)
@@ -611,19 +616,44 @@ audio/video codec library.")
  `(("alsa-lib" ,alsa-lib)
("avahi" ,avahi)
("dbus" ,dbus)
+   ("eudev" ,eudev)
+   ("faad2" ,faad2)
("flac" ,flac)
("ffmpeg" ,ffmpeg-2.8)   ;fails to build against ffmpeg 3.0
+   ("fluidsynth" ,fluidsynth)
("fontconfig" ,fontconfig)
("freetype" ,freetype)
+   ("gnome-vfs" ,gnome-vfs) ; FIXME: the build system cannot detect it
("gnutls" ,gnutls)
+   ("gst-plugins-base" ,gst-plugins-base)
+   ("gtk+" ,gtk+-2)
+   ("jack" ,jack-2)
+   ("liba52" ,liba52)
+   ("libass" ,libass)
+   ("libavc1394" ,libavc1394)
+   ("libbluray" ,libbluray)
+   ("libcaca" ,libcaca)
+   ("libdca" ,libdca)
("libcddb" ,libcddb)
+   ("libdvdnav" ,libdvdnav)
+   ("libdvdread" ,libdvdread)
("libgcrypt" ,libgcrypt)
("libkate" ,libkate)
("libmad" ,libmad)
+   ("libmpeg2" ,libmpeg2)
+   ("libmodplug" ,libmodplug)
+   ("libmtp" ,libmtp)
+   ("libnotify" ,libnotify)
("libogg" ,libogg)
("libpng" ,libpng)
+   ("libraw1394" ,libraw1394)
("libsamplerate" ,libsamplerate)
+   ("libshout" ,libshout)
("libssh2" ,libssh2)
+   ("librsvg" ,librsvg)
+   ("libupnp" ,libupnp)
+   ("libva" ,libva)
+   ("libvdpau" ,libvdpau)
("libvorbis" ,libvorbis)
("libtheora" ,libtheora)
("libxext" ,libxext)
@@ -640,18 +670,22 @@ audio/video codec library.")
("qt" ,qt) ; FIXME: reenable modular qt after update - requires building
;("qtbase" ,qtbase) with -std=gnu++11.
;("qtx11extras" ,qtx11extras)
+   ("samba" ,samba)
("sdl" ,sd