[PD] pdp removing v4l in favour of v4l2 [WAS] Re: Pd-extended build for Ubuntu 11.04 x64 ?

[Hans-Christoph Steiner]

On Jun 29, 2011, at 10:36 AM, Lorenzo Sutton wrote:

> IOhannes m zmoelnig wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> hi
>>
>> i haven't had a look at your code but:
>
> Should have mentioned this wasn't intended as anything definitive  
> (and probably was a mistake to forward to the ml at this stage)
>
>> On 2011-06-29 15:31, Lorenzo Sutton wrote:
>>> Previously sent to Hans only forwarding to the list.
>>> Hi again...
>>>
>>>> The best thing would be to help Lorenzo get pdp working with v4l2  
>>>> and
>>>> remove the v4l stuff.
>>> It doesn't seem so trivial though... I hadn't understood correctly  
>>> that
>>> pdp_v4l2 is actually part of pidip, not pdp.
>>>
>>> So following the radical approach of totally getting rid of v4l in
>>> favour of v4l2 here's  my first attempt where I brutally replaced
>>> pdp_v4l.c with the contents of pdp_v4l2.c,
>> now this sounds really really like bad practice.
>>
>> #1
>> imho, [pdp_v4l] is a v4l1 object, and should stay like that.
>> by changing it, you break all the patches that rely on [pdp_v4l]
>> accessing their v4l1-webcam.
>>
>>
>> #2
>> pidip has been removed from the puredata repositories because of  
>> license
>> issues. just using the code and assuming whatever license is a no-go.
> Ok. It would make sense then to have pdp_v4l2 in pdp. Anyhow pidip  
> is still in the official source for 0.42 extended presented on the  
> front page of puredata.info
>
> That said I don't have the skills/time to create a pdp_v4l2 so I  
> guess if the pidip code can't be used someone (tm) should pick up  
> the task.


We can definitely help with the tricky bits. It would be a valuable  
contribution to PDP.  You could also contact Yves directly (the pidip  
author) and see if he would be willing to let you include the pdp_v4l2  
code in PDP as the same license.

At the very least, it should be easy to get pdp_v4l compiling again  
using the new header.

.hc



>> replaced #include
>>> <linux/videodev2.h>  with #include<libv4l1-videodev.h>  and  
>>> renamed the
>> you should not hardcode this.
>> pdp comes with an autoconf system. write a check whether a header  
>> exists
>> and use the result of the check to include the header file.
>> (esp. when replacing a header that comes with virtually any modern  
>> linux
>> by one for which you need to install a 3rd party library)
> I don't understand.
>>> setup function and symbol in the setup function to pdp_v4l (i.e. the
>>> object is still [pdp_v4l] when you load the pdp library).
>> see above.
> Yes.
>>> I did some changes to configure.ac concerning the ability to pick  
>>> up the
>>> pd source file (giving for granted puredata-dev is installed) and  
>>> the
>> don't take anything for granted!
>> pdp comes with autoconf: use it to write tests to check whether a  
>> system
>> matches your expectation.
> For granted I meant once the dependency has been checked as satisfied.
>>> videodev.h headers.
>>> - changed the default  prefix to /usr
>> don't!
>> the default prefix for every build should always be /usr/local.
>> /usr is reserved for package managers (like apt).
>>
>> packagers can easily install to /usr by specifiying "--prefix=/usr"  
>> in
>> their packaging rules.
>>
> Ok
>>> Sorry this is all very hacky... but as I said I'm very new to this  
>>> kind
>>> of 'packaging' thing.
>>>
>> sorry for being so down-to-earth.
>> the above mentioned issues shouldn't be too hard to resolve though,  
>> and
>> it would make you skip the "packaging noob" status :-)
> See above.
> Lorenzo
>> fgamdr
>> IOhannes
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v1.4.11 (GNU/Linux)
>> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
>>
>> iEYEARECAAYFAk4LLwYACgkQkX2Xpv6ydvS4TQCg55gsNileBf8h8mG6WAFupaVU
>> fFAAoI+juwrr+u4vcFo1lGaZae1p4nVU
>> =4xlr
>> -----END PGP SIGNATURE-----
>>
>>
>>
>> _______________________________________________
>> Pd-list at iem.at mailing list
>> UNSUBSCRIBE and account-management ->  http://lists.puredata.info/listinfo/pd-list
>
> _______________________________________________
> Pd-list at iem.at mailing list
> UNSUBSCRIBE and account-management -> http://lists.puredata.info/listinfo/pd-list



----------------------------------------------------------------------------

"We have nothing to fear from love and commitment." - New York Senator  
Diane Savino, trying to convince the NY Senate to pass a gay marriage  
bill