-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
repackage libfreenect example commands #4
Comments
That seems fair to me. The easiest thing to do would be to patch the Is this the reason I am seeing spam about libfreenect source deb? Thanks! On Tue, May 7, 2013 at 9:56 AM, jack-oquin [email protected] wrote:
|
I agree. We don't actually require them for freenect_camera. Sometimes it would be useful to have some of those programs, perhaps for testing or diagnosing problems. That's why I would consider packaging them separately. The libfreenect sourcedeb problem looks like something different to me. Maybe just a build farm screw-up, or maybe something I did wrong during the release cycle. It's complaining about checksum mismatches. I asked about it on the ros-release ML: https://code.ros.org/lurker/message/20130507.153722.2359b1de.en.html I forgot to copy you on that message. No response yet. |
That's odd. I am pretty sure I signed up for ros-release. Maybe the request On Tue, May 7, 2013 at 11:07 AM, jack-oquin [email protected]:
|
Are those ros-release messages getting caught in a spam filter, perhaps? As a package maintainer, you are supposed to subscribe. If it's not working, contact @tfoote. |
@piyushk I don't see you on the list of subscribers. |
The confirmation email for joining the list was going to spam, and I never On Tue, May 7, 2013 at 12:37 PM, Tully Foote [email protected]:
|
The example commands depend on heavyweight 3D GUI libraries, too big for a library that drivers use.
Also, these commands are installed in the global ros bin/ directory. Their names are not unique enough for that to be appropriate.
Either leave them out entirely, or move them to a separate package and install the commands in that package's lib/PACKAGENAME/ directory.
The text was updated successfully, but these errors were encountered: