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.
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-----
_______________________________________________
[email protected] mailing list
UNSUBSCRIBE and account-management ->
http://lists.puredata.info/listinfo/pd-list
_______________________________________________
[email protected] mailing list
UNSUBSCRIBE and account-management ->
http://lists.puredata.info/listinfo/pd-list
