On Wed, Aug 02, 2006, Bob Copeland wrote: > > If this returns the device type then that removes the need for the > > extra function that we were discussing in relation to lkarmafs. > > So it all seems to work out nicely. > > Wait, what are we talking about here? Having lk_karma_connect return > the device type? I was asleep during the lkarmafs discussion -- what is > it needed for again?
Ideally, an application should not care whether USB or ethernet is in use so there should be no need for such a call. Unfortunately, there does not seem to be any way of generating correct tags, rid etc. needed by lkarmafs without specifying a filename. lkarmafs doesn't (and can't) have a source filename passed to it and currently the only workable solution seems to be the generation of a temporary file. (However, the details haven't been worked out yet). The long and the short of it is that it's only really necessary for ethernet since the USB device can make use of the file living on the karma. > Right now, lk_karma_connect returns the socket file descriptor when > using the net, and that certainly can't be changed without a fair amount > of rework. > > Personally, I'd like to see lk_karma_connect() return an opaque > lk_context_t* or something like that, so we can stick our globals in it > and make it thread safe. But that's a fairly big departure from where it > is right now. I agree that any changes to lk_karma_connect() are going to have an impact on all the utilities which make use of it. This should certainly wait until after the forthcoming release. The opaque data structure sounds like a good idea. It shields the end user from the internal workings of libkarma and will allow for future changes to be made more easily without affecting applications. Keith. ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ linux-karma-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-karma-devel
