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.infoThat 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 theyou 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
thedon'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@iem.at mailing list UNSUBSCRIBE and account-management -> http://lists.puredata.info/listinfo/pd-list
Pd-list@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