Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Clément Bœsch
On Sun, Jul 13, 2014 at 09:57:39PM +0100, Derek Buitenhuis wrote: > On 7/13/2014 9:56 PM, Clément Bœsch wrote: > > In ffplay? > > O U. > > At least we don't move libSDL to avdevice or something... :P > Well, actually... -- Clément B. pgpcRmGxDToMl.pgp Description: PGP signature

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 9:56 PM, Clément Bœsch wrote: > In ffplay? O U. At least we don't move libSDL to avdevice or something... :P - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Clément Bœsch
On Sun, Jul 13, 2014 at 09:36:48PM +0100, Derek Buitenhuis wrote: > On 7/13/2014 9:27 PM, compn wrote: > > googletv does not have easy shell and cannot mount smb. at least mine > > cant > > See what I said about implementing support in the layer above (the player) > instead of ffmpeg. > In ffpla

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 9:27 PM, compn wrote: > googletv does not have easy shell and cannot mount smb. at least mine > cant See what I said about implementing support in the layer above (the player) instead of ffmpeg. - Derek ___ ffmpeg-devel mailing list ffmpeg-

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread compn
On Sun, 13 Jul 2014 18:04:54 +0100 Derek Buitenhuis wrote: > On 7/13/2014 5:54 PM, Lukasz Marek wrote: > > You also ignored the case where user might have no access to shell > > at all to mount remote fs. > > That's a pretty damn specialized case. googletv does not have easy shell and cannot mo

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 06:44:21PM +0200, Lukasz Marek wrote: > On 13.07.2014 18:04, Derek Buitenhuis wrote: > >On 7/12/2014 5:20 PM, Lukasz Marek wrote: > >>+libsmbclient_protocol_deps="libsmbclient gpl" > > > >Samba is actually GPLv3, so you need to add dependency on that. > > Fixed. > Changel

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Lukasz Marek a écrit : > I am not sure about that. I agree it is a bit strange libavformat > has both (de)muxers and protocols. Some protocols are tied tightly to the corresponding muxer or demuxer, or the other way around. And anyway, what good would it do sep

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread wm4
On Sun, 13 Jul 2014 18:54:16 +0100 Derek Buitenhuis wrote: > Personally I've always thought it would be preferable to integrate something > like libcurl instead of writing our own implementations of every little > protocol, and the terrible mess that is our TLS support. +1 __

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 6:51 PM, Nicolas George wrote: > - You can not dig a hole without a shovel. > - Yes, you can, if someone already dug it. Your analogy is flawed, of course. If the hole is "accessing the file on a CIFS share", then it's more like: - You need to dig a hole, but you need a permit. - Pe

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 6:38 PM, Lukasz Marek wrote: > I am not sure about that. I agree it is a bit strange libavformat has > both (de)muxers and protocols. I would expect it to be separated, but > overall I don't don't agree it is bad ffmpeg has them at all. Such a library exists. It's called libcurl. >

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > You claimed its not possible, and it indeed is, with proper setup. - You can not dig a hole without a shovel. - Yes, you can, if someone already dug it. This is getting silly and boring, count me out. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 6:31 PM, Nicolas George wrote: > Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : >> You skillfully ignored the words "with proper setup", good job. > > I did not think you would insist on your "no true Scotsman" fallacy. > > Fact is that mounting is often not possible,

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 19:04, Derek Buitenhuis wrote: On 7/13/2014 5:54 PM, Lukasz Marek wrote: You also ignored the case where user might have no access to shell at all to mount remote fs. That's a pretty damn specialized case. Also, I wouldn't like a player (or anything else) mount anything under c

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > You skillfully ignored the words "with proper setup", good job. I did not think you would insist on your "no true Scotsman" fallacy. Fact is that mounting is often not possible, and running ffmpeg in that case is still legitimate, i

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 5:54 PM, Lukasz Marek wrote: > You also ignored the case where user might have no access to shell at > all to mount remote fs. That's a pretty damn specialized case. > Also, I wouldn't like a player (or anything else) mount anything under > carpet on my workstation. Case when I'm f

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 18:17, Derek Buitenhuis wrote: On 7/13/2014 5:10 PM, Nicolas George wrote: Technically, wrong. Technically right. Do you want an unprivileged account somewhere so I can dare you to mount something with FUSE (without using a local root exploit of course, that would be cheating)?

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 18:04, Derek Buitenhuis wrote: On 7/12/2014 5:20 PM, Lukasz Marek wrote: +libsmbclient_protocol_deps="libsmbclient gpl" Samba is actually GPLv3, so you need to add dependency on that. Fixed. >From 2218dd824524265e563c9df113c07179d74c982c Mon Sep 17 00:00:00 2001 From: Lukasz Ma

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 5:10 PM, Nicolas George wrote: >> Technically, wrong. > > Technically right. Do you want an unprivileged account somewhere so I can > dare you to mount something with FUSE (without using a local root exploit of > course, that would be cheating)? You skillfully ignored the words "with

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > Technically, wrong. Technically right. Do you want an unprivileged account somewhere so I can dare you to mount something with FUSE (without using a local root exploit of course, that would be cheating)? > You're going to mount a re

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/12/2014 5:20 PM, Lukasz Marek wrote: > +libsmbclient_protocol_deps="libsmbclient gpl" Samba is actually GPLv3, so you need to add dependency on that. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/f

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 4:58 PM, Nicolas George wrote: > Not wrong, merely too concise for the inattentive reader. Technically, wrong. Since with proper setup you *can* achieve the goal. > And indeed, I know of several hosting services where FUSE is not enabled. > Low-end VPS services frequently fall in tha

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, Derek Buitenhuis a écrit : > Wrong. FUSE lets you mount samba shares as non-root. Not wrong, merely too concise for the inattentive reader. You need root privileges to install FUSE on a system and enable it for any or all users: FUSE does not allow to mount wit

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 2:39 PM, Lukasz Marek wrote: > - it is right you say for desktops where you have access to shell (and > root?), but how about cases where you create some embedded system > without shell access or have no root? FUSE allows non-rot mounts. > - you should still consider users that are

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 2:36 PM, Nicolas George wrote: > and mounting requires root privileges. Wrong. FUSE lets you mount samba shares as non-root. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 15:22, wm4 wrote: On Sat, 12 Jul 2014 18:20:46 +0200 Lukasz Marek wrote: --- Changelog | 1 + configure | 5 ++ doc/general.texi | 1 + doc/protocols.texi | 29 +++ libavformat/Makefile | 1 + libavform

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Nicolas George
Le quintidi 25 messidor, an CCXXII, wm4 a écrit : > You can mount samba shares directly in most (all?) desktop OSes. FFmpeg is not used on desktop, and mounting requires root privileges. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Lukasz Marek
On 13.07.2014 13:47, Michael Niedermayer wrote: On Sun, Jul 13, 2014 at 12:55:47AM +0200, Lukasz Marek wrote: On 12.07.2014 22:31, Michael Niedermayer wrote: On Sat, Jul 12, 2014 at 06:20:46PM +0200, Lukasz Marek wrote: die_license_disabled gpl frei0r die_license_disabled gpl libcdio +die_

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread wm4
On Sat, 12 Jul 2014 18:20:46 +0200 Lukasz Marek wrote: > --- > Changelog | 1 + > configure | 5 ++ > doc/general.texi | 1 + > doc/protocols.texi | 29 +++ > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + >

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 12:55:47AM +0200, Lukasz Marek wrote: > On 12.07.2014 22:31, Michael Niedermayer wrote: > >On Sat, Jul 12, 2014 at 06:20:46PM +0200, Lukasz Marek wrote: > >>--- > >> Changelog | 1 + > >> configure | 5 ++ > >> doc/general.texi

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-12 Thread Lukasz Marek
On 12.07.2014 22:31, Michael Niedermayer wrote: On Sat, Jul 12, 2014 at 06:20:46PM +0200, Lukasz Marek wrote: --- Changelog | 1 + configure | 5 ++ doc/general.texi | 1 + doc/protocols.texi | 29 +++ libavformat/Makefile

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-12 Thread Michael Niedermayer
On Sat, Jul 12, 2014 at 06:20:46PM +0200, Lukasz Marek wrote: > --- > Changelog | 1 + > configure | 5 ++ > doc/general.texi | 1 + > doc/protocols.texi | 29 +++ > libavformat/Makefile | 1 + > libavformat/allformats.c |

Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-12 Thread Timothy Gu
On Jul 12, 2014 9:21 AM, "Lukasz Marek" wrote: > > --- > Changelog | 1 + > configure | 5 ++ > doc/general.texi | 1 + > doc/protocols.texi | 29 +++ > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libav