[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #97 from Rosen Diankov --- great news! ikfast now works with sympy 0.7.x (trunk r3320). You can download the two necessary files from here and patch up the openrave 0.6.4 files: https://openrave.svn.sourceforge.net/svnroot/openrave/trunk/python/ikfast.py https://openrave.svn.sourceforge.net/svnroot/openrave/trunk/python/ikfast_generator_cpp.py In openrave 0.7.0 we've added inverse dynamics computation and many other cool new features, but it will be several (or more) weeks before we officially release it as openrave 0.8.0 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Peter Lemenkov changed: What|Removed |Added Component|0x |Package Review Assignee|lemen...@gmail.com |nob...@fedoraproject.org -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #98 from Rosen Diankov --- i forgot to say that if you are going to just replace the ikfast* files, you'll need to get rid of the sympy 0.6.x check in CMakeLists.txt -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #99 from Rosen Diankov --- hi guys, great news, we released openrave 0.6.6 last week that has the latest and greatest dependencies (including sympy 0.7.x support and collada 2.4.x support) please tell me if there's anything else needed for an official fedora core rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Till Hofmann changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |DUPLICATE Last Closed||2016-05-21 15:34:49 --- Comment #105 from Till Hofmann --- I submitted a new review request with some additional changes. If anyone wants to review it, I'd be very thankful! *** This bug has been marked as a duplicate of bug 1338339 *** -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Rich Mattes changed: What|Removed |Added Blocks||1225692 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1225692 [Bug 1225692] Tracker for Fedora Robotics SIG -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Till Hofmann changed: What|Removed |Added CC||hofm...@kbsg.rwth-aachen.de --- Comment #103 from Till Hofmann --- Hi everyone, I'm picking up Tim's work from here (with Tim's consent). I'm not sure whether I should submit a new review request, so I'll add my comments here first. SPEC: https://thofmann.fedorapeople.org/openrave.spec SRPM: https://thofmann.fedorapeople.org/openrave-0.9.0-4.20150624gitb32979b.fc20.src.rpm patch0: https://thofmann.fedorapeople.org/openrave.pkgconfig.patch patch1: https://thofmann.fedorapeople.org/openrave.pyscript-permissions.patch The new version contains the following changes: - update to a newer upstream version: Since the last release (0.9.0 from Oct 2012) a lot of fixes have been included, so it makes more sense to use the current git version instead of the last release - move documentantion into a separate doc package - fix AutoProvides and AutoRequires, which included openrave plugins - rename python package to python2-openrave, following the Python Packaging Guidelines (unfortunately, we cannot yet build a python3 package, because there is no python3 version of flann) - exclude arm from the supported architectures (upstream bug: https://github.com/rdiankov/openrave/issues/354) - clean up BR - split models into a separate noarch package Since there is now a separate package for fparser, this package now uses that external fparser, which ships all headers needed to build openrave. Since fparser does not currently provide a cmake module, I've added a patch to find fparser using pkg-config. I've also sent the patch upstream: https://github.com/rdiankov/openrave/pull/352 The second patch contains a simple fix to make all installed python scripts executable. Also sent upstream: https://github.com/rdiankov/openrave/pull/353 We still use two 3rdparty libs: convexdecomposition and ivcon. ivcon is so small (two files) that I think it does not make sense to package it separately. I'm not sure how to proceed with convexdecomposition. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10259352 rpmlint still gives some warnings: openrave.x86_64: W: shared-lib-calls-exit /usr/lib64/libopenrave-core.so.0.9.0 exit@GLIBC_2.2.5 --> reported upstream: https://github.com/tpaviot/oce/issues/161 openrave.x86_64: W: no-documentation openrave-octave.x86_64: W: no-documentation openrave-models.noarch: W: no-documentation openrave-devel.x86_64: W: no-documentation --> separate doc package openrave-devel.x86_64: W: only-non-binary-in-usr-lib --> symlink openrave.x86_64: W: no-manual-page-for-binary openrave openrave-devel.x86_64: W: no-manual-page-for-binary openrave-config openrave-devel.x86_64: W: no-manual-page-for-binary openrave-createplugin.py --> no manpages provided by upstream. openrave.x86_64: E: non-executable-script /usr/share/openrave/openrave.bash 0644L /bin/bash --> I *think* this script is only used to set up the environment, and should always be sourced. But this may actually need to be fixed. Please let me know if I should submit a new review request. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Bug 674008 depends on bug 814458, which changed state. Bug 814458 Summary: Review Request: fparser - Function parser library for C++ https://bugzilla.redhat.com/show_bug.cgi?id=814458 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |DUPLICATE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Rich Mattes changed: What|Removed |Added Depends On||1069257 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1069257 [Bug 1069257] Review Request: fparser - Function parser library for C++ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Till Hofmann changed: What|Removed |Added Depends On|1069257 | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1069257 [Bug 1069257] Review Request: fparser - Function parser library for C++ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Till Hofmann changed: What|Removed |Added Depends On||1069257 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1069257 [Bug 1069257] Review Request: fparser - Function parser library for C++ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 Bug 674008 depends on bug 1069257, which changed state. Bug 1069257 Summary: Review Request: fparser - Function parser library for C++ https://bugzilla.redhat.com/show_bug.cgi?id=1069257 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #100 from Michael Schwendt --- Has anyone ever before asked the FPC about a bundling exception for fparser? https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Copylibs Alternatively, instead of building a shared libfparser (which may be problematic due to the nature of the code, e.g. the various defines in fpconfig.hh and optional features), it would be possible to build a fparser-source binary rpm, which packages like openrave may build-require and *copy* from (since fparser is a copylib). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #101 from Michael Schwendt --- > http://fedorapeople.org/~timn/robotics/openrave-0.6.4-1.fc15.src.rpm That's a damaged file: $ rpm -Kv openrave-0.6.4-1.fc15.src.rpm openrave-0.6.4-1.fc15.src.rpm: Header SHA1 digest: OK (d2a244f989f76bc5dd02d95386a3d86b4473666e) MD5 digest: BAD Expected(7fcab7a1c091b5384736f7d98e6132a8) != (e948197b48aa8bf3705ecbd247b100f3) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #102 from Rosen Diankov --- FYI, i'm planning to make a github repo for fparser and put all my cmake scripts on it so it installs easily on any linux/windows system. this will happen within a month. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #104 from Till Hofmann --- I've created a COPR for openrave: https://copr.fedoraproject.org/coprs/thofmann/openrave/ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Component|Package Review |0x Flag|fedora-review? | --- Comment #65 from Tim Niemueller 2011-12-13 06:54:53 EST --- Peter, thanks for the review. Finally I got around to package OpenRAVE 0.5.0 and fix the problems you mentioned: - dos2unix on some files - no more zero-length octave files (fixed upstream) - did not fix the non-executable-script warnings, they should be fixed upstream - devel sub-package depends on cmake I have also updated the patch, Rosen please re-check if this is suitable to include upstream, it does: - provide compatibility with older Boost versions (think of Enterprise Linux) - Make include directory name a variable that can be overridden (for example to use only a major version suffix at some point) - Allow overriding of share directory (not using version suffix on Fedora, but default to use it - Allow overriding of library suffix (no version suffix on Fedora) - Allow overriding of CMake directory name (default to the original value, set on Fedora from cmake call) - Allow overriding of CPack package install directory - pkg-config file uses the new variables The new SRPM is at http://fedorapeople.org/~timn/robotics/openrave-0.5.0-1.fc15.src.rpm, spec file changed in place. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #66 from Rosen Diankov 2011-12-13 07:23:22 EST --- hi tim, sorry i've been procrastinating on this issue recently, i can take a look at your changes. currently the patch attached to this thread is openrave-0.4.1... where can we get the latest patch? good news is there's still a couple of weeks before we officially release/freeze 0.5.0, so we can include these changes. we're aiming for end of the year. in terms of API changes, there's 3 tickets left to resolve: #158, #182, #228 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Attachment #514810|0 |1 is obsolete|| --- Comment #67 from Tim Niemueller 2011-12-13 07:36:52 EST --- Created attachment 546188 --> https://bugzilla.redhat.com/attachment.cgi?id=546188 Patch to 0.5.0 for upstreaming by Rosen -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #68 from Tim Niemueller 2011-12-13 07:40:08 EST --- I have added the patch, it's in the SRPM as well. 0.5.0 is not released? I thought the version at the top of the OpenRAVE page would state the latest release, no? Really looking forward to this version and having it in Fedora finally. Peter, if you're happy to accept the current state I won't release a package before 0.5.0 final comes out, hence I won't change the package version to a pre-release one for the review right now unless you insist. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674008
--- Comment #69 from Rosen Diankov 2011-12-13
08:38:54 EST ---
hi tim,
sorry for the confusion, the latest_stable are the release candidates, and are
in constant flux. the last official release is 0.4.2. should i write something
on the webpage to avoid this confusion?
Your patches look good except:
1. why did you remove the relpath being called? this is necessary on ubuntu
systems.
COMMAND ${PYTHON_EXECUTABLE} -c "from distutils.sysconfig import
get_python_lib; from os.path import relpath; print
relpath(get_python_lib(1,prefix='${CMAKE_INSTALL_PREFIX}'),'${CMAKE_INSTALL_PREFIX}')"
2. openrave.pc is not supposed to link with libopenrave-core.so. Plugin authors
only need to link with libopenrave.so. I think what you are looking for is
openrave-core.pc, do you want to output it?
3. i would like to rename OPENRAVE_INCLUDE_DIRNAME to OPENRAVE_INCLUDE_DIR.
4. config.h.in, you removed CMAKE_INSTALL_PREFIX. what were your reasons?
unfortunately this is needed or openrave will not be able to find the default
plugin/data directories.
5. as for the boost changes, i think we had those fixed on the recent trunk.
from a quick glance, it looks like the current openrave trunk should compile
fine for you..
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #70 from Rosen Diankov 2011-12-13 08:40:20 EST --- for the boost filesystem, i see you added "boost::filesystem::native", was there a reason for this? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #72 from Tim Niemueller 2011-12-13 09:02:49 EST --- (In reply to comment #70) > for the boost filesystem, i see you added "boost::filesystem::native", was > there a reason for this? Compatibility with older Boost versions as stated in the previous comment. This is also taken from my older patch and though I haven't tried it again on my CentOS box I'd assume it still breaks. And since it is an easy fix I wanted to keep it just in case. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674008
--- Comment #71 from Tim Niemueller 2011-12-13 09:01:00
EST ---
(In reply to comment #69)
> hi tim,
Hi Rosen.
> sorry for the confusion, the latest_stable are the release candidates, and are
> in constant flux. the last official release is 0.4.2. should i write something
> on the webpage to avoid this confusion?
Yes, I'm an example of confusion about this ;-)
> Your patches look good except:
>
> 1. why did you remove the relpath being called? this is necessary on ubuntu
> systems.
>
> COMMAND ${PYTHON_EXECUTABLE} -c "from distutils.sysconfig import
> get_python_lib; from os.path import relpath; print
> relpath(get_python_lib(1,prefix='${CMAKE_INSTALL_PREFIX}'),'${CMAKE_INSTALL_PREFIX}')"
It didn't work for me this way on Fedora. I'll give it another try, I just
copied that from the older patches and did not re-evaluate.
> 2. openrave.pc is not supposed to link with libopenrave-core.so. Plugin
> authors
> only need to link with libopenrave.so. I think what you are looking for is
> openrave-core.pc, do you want to output it?
Or the other way around: have openrave-plugin.pc with the basics, and
openrave.pc for anyone who wants to link against OpenRAVE for embedding? Either
way is fine and would be just what I need.
> 3. i would like to rename OPENRAVE_INCLUDE_DIRNAME to OPENRAVE_INCLUDE_DIR.
Sure, any kind of renames are fine. Just let me know so I can adapt the spec
file, please.
> 4. config.h.in, you removed CMAKE_INSTALL_PREFIX. what were your reasons?
> unfortunately this is needed or openrave will not be able to find the default
> plugin/data directories.
That is because I pass in absolute paths, which is required since on Fedora it
must go to /usr/lib or /usr/lib64 based on the architecture. I'd say let's make
the paths in CMakelists.txt absolute paths, so that they can be overridden
nicely by cmake parameters. The fewer places where we compose/combine paths the
better for overriding.
I didn't think enough about this so that I did actually change the default
which I did not intent.
> 5. as for the boost changes, i think we had those fixed on the recent trunk.
> from a quick glance, it looks like the current openrave trunk should compile
> fine for you..
It does on Fedora, it does not on RHEL with an older Boost version (or on
robots which simply have an older Fedora version and are a pain to upgrade
right now).
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #73 from Rosen Diankov 2011-12-13 09:23:59 EST --- > That is because I pass in absolute paths, which is required since on Fedora it > must go to /usr/lib or /usr/lib64 based on the architecture. I'd say let's > make > the paths in CMakelists.txt absolute paths, so that they can be overridden > nicely by cmake parameters. The fewer places where we compose/combine paths > the > better for overriding. that makes sense, unfortunately i think there's a debian thing where we need relative paths. given that you are not using CMAKE_INSTALL_PREFIX couldn't you add something like: cmake -DCMAKE_INSTALL_PREFIX="" ? i'll re-examine the boost patches. > > > 5. as for the boost changes, i think we had those fixed on the recent trunk. > > from a quick glance, it looks like the current openrave trunk should compile > > fine for you.. > > It does on Fedora, it does not on RHEL with an older Boost version (or on > robots which simply have an older Fedora version and are a pain to upgrade > right now). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674008
--- Comment #74 from Rosen Diankov 2011-12-13
19:31:54 EST ---
hi tim,
i committed the changes to r2909. here's my notes:
new variables:
- OPENRAVE_CMAKE_INSTALL_DIR, replaces OPENRAVE_CMAKE_DIRNAME
- OPENRAVE_INCLUDE_INSTALL_DIR - replaces OPENRAVE_INCLUDE_DIRNAME
pkg-config file now uses:
openrave${OPENRAVE_BIN_SUFFIX}.pc
openrave${OPENRAVE_BIN_SUFFIX}-core.pc
We stuck with openrave-core since that's what the library name is.
you had some typos on boost::filesystem::system_complete, it would be great if
you test on RHEL again.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #75 from Rosen Diankov 2011-12-14 08:21:17 EST --- make that r2911 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #76 from Rosen Diankov 2012-01-06 22:22:35 EST --- hi guys, any feedback on the last openrave build before we officially release 0.5.0? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #77 from Tim Niemueller 2012-01-07 06:53:08 EST --- I'll build the new version today or tomorrow as time permits. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #78 from Tim Niemueller 2012-01-11 08:16:18 EST --- I tried to build, my patch is down to two things (will create a new bug attachment): - Add @LIB_SUFFIX@ in pkg-config files (probably just an oversight) - Full path requirement in config.h.in But the F-15 sympy version is 0.7.1 and it assumes that it needs to build the local sympy version, which is not acceptable according to the Fedora guidelines. Rosen, is there an OpenRAVE version that works with this newer sympy version? The first part of the patch is simpe. For the second part I have added three new variables which contain the absolute path (including the CMAKE_INSTALL_PREFIX). I can now easily override those from the spec file and the original behavior for the config.h.in stays the same while having only one substitution variable there. Please have a look and let me know if you can apply the patches -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #79 from Rosen Diankov 2012-01-11 19:53:06 EST --- hi tim, ok, the two features you requested are now in r2949 #define OPENRAVE_PLUGINS_INSTALL_DIR "@OPENRAVE_PLUGINS_INSTALL_ABSOLUTE_DIR@" #define OPENRAVE_DATA_INSTALL_DIR "@OPENRAVE_DATA_INSTALL_ABSOLUTE_DIR@" #define OPENRAVE_PYTHON_INSTALL_DIR "@OPENRAVE_PYTHON_INSTALL_ABSOLUTE_DIR@" As for sympy, i think you'll have an easier time including a sympy 0.6.x version into fc15 rather than getting ikfast to work with 0.7.1. of course, we'd be more than happy to get patches for it, as long as 0.6.x compat is preserved. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #80 from Tim Niemueller 2012-01-12 04:55:46 EST --- Ok, I'll try the new version, but with sympy we got a problem on our hands. Since F-15 a newer version is part of the distro and we cannot just duplicate that with major effort. It might make more sense to spend that time on getting OpenRAVE fit for Sympy 0.7.1, it could be a show stopper otherwise. Since I have no experience with Sympy, what's the big deal with the new version, what needs to be changed, how much effort would it be? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #81 from Rosen Diankov 2012-01-12 07:33:27 EST --- hi tim, i understand your concerns. i don't know what changed in 0.7.1, feel free to explore this on your free time. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #82 from Tim Niemueller 2012-01-12 07:45:56 EST --- Hmm, that sounds like it would never happen? It's out there for more than half a year and it'll eventually show up in other distros as well. Have you tried it and spotted errors or is it merely a "0.6.7 works so let's don't touch it" kind of thing? Any Python/OpenRAVE guru on your side who could take a look? I don't have the time for a while... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #83 from Rosen Diankov 2012-01-12 11:16:55 EST --- you are right, eventually we'll have to add support. i plan to get around to it in 4 months -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #84 from Rosen Diankov 2012-01-20 08:38:19 EST --- openrave 0.5.0 was just officially released. several bugs were fixed in the last 48 hours, so i recommend you create new rpms with the following code: https://openrave.svn.sourceforge.net/svnroot/openrave/tags/0.5.0 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #85 from Rosen Diankov 2012-02-14 00:33:55 EST --- we're now at openrave 0.6.3 (r3104) with several exciting changes to the fparser library. As of fparser v4.4.3, openrave can now use the library without any local changes at all, meaning that it is now possible to create an official RPM of the package! http://www.programmingvision.com/fparser4.4.3_with_cmake.tgz if making a diff against the official 4.4.3 that can be found here: http://warp.povusers.org/FunctionParser/ please note the extra files that are needed: 1. CMakeLists.txt 2. modules-cmake directory 3. fpconfig.hh.in 4. fparser-config.cmake.in 5. fparser-config-version.cmake.in -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #34 from Rosen Diankov 2011-03-28 05:54:31 EDT --- Just released openrave 0.2.20 (r2199). You can see change log at: https://sourceforge.net/apps/trac/openrave/wiki/ChangeLog -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Bug 674008 depends on bug 676159, which changed state. Bug 676159 Summary: Review Request: crlibm - Correctly Rounded mathematical library https://bugzilla.redhat.com/show_bug.cgi?id=676159 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #35 from Rosen Diankov 2011-04-06 03:09:42 EDT --- hi guys, anyone know what's happening with the release? any problems left that i can help in resolving? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #36 from Tim Niemueller 2011-04-06 06:34:40 EDT --- Hi Rosen. We've used OpenRAVE successfully during the RoboCup German Open 2011 last week. It has been integrated with Fawkes to drive a Katana arm. We're preparing a demo video highlighting some of our work areas, I'll post a link once it's ready. I have to talk to my student, I think we've used openrave-0.2.20-0.2.svn2227.fc14.x86_64 in the end. It still required a little patch, due to bug #667294 (Boost was missing deprecated iostreams ctors). This should have been resolved by now and I'm going to test it later today. Please consider upgrading to non-deprecated calls in a future version. Rosen, I have currently set to use single-precision floats to speed up OpenRave a little. Do you think that has overly negative impact, or is that a good default choice for a distro package? When compiling OpenRave at around 50% it compiles openravepy_int.o, which takes more than 1.5GB of memory and is pretty heavy on the system. Any chance of splitting this up in a future release? We've been seeing some weird slowness on 32-bit systems with slow database generation, while it ran fine with the identical (besides bitness) 64-bit version. I'll discuss this with my team members tomorrow if this was an intermediate problem and if so how it was resolved. >From the packaging site I need to push updates for collada-dom (which is basically ready and built, I just didn't have the time). I have to disable assimp atm, until review bug #635511 is through. Once that is done, we get rid of the ivcon integrated library, right? After that, it leaves two libraries in there. We're clear that convexdecomposition has no standalone upstream and it got patches, therefore should stay. fparser has critical patches. Rosen, how is upstreaming going on this? Maybe you can try again and tell them that'll bring fparser to the Fedora universe as an incentive? Peter, please have a look again at this review now, I think we've reached a good state to continue, with the only things to resolve being assimp acceptance and fparser patch upstreaming and eventually ripping it out of OpenRave. What do you think? SPEC has been changed in-place, the SRPM is at http://fedorapeople.org/~timn/robotics/openrave-0.2.20-0.2.svn2227.fc14.src.rpm. It contains the patch to work with old boost. It can build with or without assimp, for the latter just install the review package. Once it has been accpted I'll add it to the BRs. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #37 from Rosen Diankov 2011-04-07 03:38:19 EDT --- Hi Tim, congratulations on a successful robocup! integration with Fawkes sounds very exciting. thanks for the heads up on the iostreams bug, it is fixed in r2278. i don't recommend single precision, it is just not worth the speedup. bugs due to single-precision inaccuracies take a long time to find. If you really need it, then just have a openrave-single rpm package along the regular one. the openravepy_int.cpp problem is known and we're trying to separate into multiple files, but it's not a high priority thing for us. if you can track down the 32bit db problem slowness, please open a ticket at the openrave trac: https://sourceforge.net/apps/trac/openrave/newticket i pinged the fparser author and he agrees it would be cool to have a fedora core rpm. we're also coming to a compromise on what to do with the local changes. however, this is not an issue worth delaying the rpm. the assimp library is used when opening model files that are not IV or VRML. for example STL, 3DS, etc. FYI, we've started a new tag called latest_stable: https://openrave.svn.sourceforge.net/svnroot/openrave/tags/latest_stable this tag is automatically updated every night if the builds/tests succeed. tar balls of the source (and eventually installers) are also updated: https://sourceforge.net/projects/openrave/files/latest_stable/ the trunk revision used is written in the commit log. whenever you do releases, please take from this tag. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #38 from Rosen Diankov 2011-04-11 06:06:50 EDT --- We've started getting all release/package related scripts inside the 'release' folder: https://openrave.svn.sourceforge.net/svnroot/openrave/trunk/release currently there is a windows installer and publishing to sourceforge. i'm not sure how the fedora core stuff works, but i would be more than happy to put a script that generates the fedora core spec, rpm files, and publishes it to fedora core. i can give you commit permissions so you can manage this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Bug 674008 depends on bug 689432, which changed state. Bug 689432 Summary: Review Request: collada-dom - COLLADA Document Object Model Library https://bugzilla.redhat.com/show_bug.cgi?id=689432 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #39 from Rosen Diankov 2011-04-21 12:34:22 EDT --- In the past week a *lot* of updates happened on the install-end of OpenRAVE. THe latest trunk version (0.3.1). Where the include files, share files, and python files are stored were completely changed: http://openrave-users-list.185357.n3.nabble.com/OpenRAVE-0-3-Official-Release-td2833265.html http://openrave-users-list.185357.n3.nabble.com/Finding-OpenRAVE-0-3-with-CMake-td2833288.html The python files are now installed in lib/python2.6/site-packages (or lib/python2.6/dist-packages) accordingly. We also generate debian source packages from cmake, which are compiled into debian binaries at: https://launchpad.net/~openrave/+archive/release The package tree looks something like: http://openrave.programmingvision.com/en/main/devel/releases.html#ubuntu-releases I think it would be possible to merge Tim's changes into the openrave cmake file to produce the RPM spec files. what do you guys think? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #41 from Rosen Diankov 2011-04-21 13:02:28 EDT --- Hi Tim, we'll be releasing 0.3.1 in the next 24 hours. once openrave can output the RPM spec files, you can always run cmake on your computer, generate the spec file, and upload it to the fedora git repository (even as a nightly cron script). no matter how you look at it, the source of the information is coming from the openrave svn repo and whenever openrave adds a new library dependency, we will simultaneously edit all the dependencies of all distros it supports. I've emailed the fparser author and we're coming to a compromise, i'll let you know when it is ok to make an rpm package of it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #40 from Tim Niemueller 2011-04-21 12:49:39 EDT --- I hope all patches are included. With changes you mean stuff from the spec file? You are welcome to add it, but the Fedora infrastructure forbids to use this to generate the packages. The spec file will be maintained and updated in the Fedora package git repository. I'll upgrade to OpenRAVE 0.3 soonish. Peter, can you have a look and do a first review? Rosen, what's the status on the fparser changes. Have they been upstreamed and released? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #42 from Rosen Diankov 2011-04-24 06:18:54 EDT --- we just released 0.3.1: https://sourceforge.net/mailarchive/forum.php?thread_name=BANLkTi%3Dibi8J9SXsuSi99R%3DWEiZd8pDx7w%40mail.gmail.com&forum_name=openrave-users we also started a page on how to install openrave on different platforms: http://openrave.programmingvision.com/en/main/install.html it would be great to add Fedora on that list. what do you think about the source rpm proposal? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #43 from Rosen Diankov 2011-07-03 04:30:31 EDT --- hi guys, i'm not sure what happened with fedora rpm packages. Just letting you know that openrave 0.4.0 was officially released yesterday. is there a chance we can get the source rpm file also put inside openrave subversion repository? thanks, rosen, -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #44 from Tim Niemueller 2011-07-03 06:16:14 EDT --- Rosen, what's the state of the fparser changes, have they been upstreamed, yet? You can of course put the SRPM file in your subversion (though my take is that binary package stuff generally does not belong in a subversion repository), but it will not be the source from which Fedora builds its packages, we want tarballs that we put on a Fedora mirror. Likewise the authoritative spec file will reside in Fedora's git repo, but you are obviously free to include a copy in your repository. I hope I find some time to package 0.4.0 next week, but it's busy atm. Peter, can you start a first review cycle so that we can straighten out rough edges? Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #46 from Rosen Diankov 2011-07-03 06:23:29 EDT --- also, where the SRPM link? thanks, -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #45 from Rosen Diankov 2011-07-03 06:23:02 EDT --- hi tim, the fparser author has been very cooperative and he released a new version that should allow openrave usage without any internal modifications. i'll email this thread once the changes are done, so hopefully we can have everything resolved in openrave 0.4.1 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #47 from Peter Lemenkov 2011-07-03 06:50:31 EDT --- Hello folks. Sorry for the hiatus - I was busy with other stuff. Tim, where is the latest src.rpm to start review with? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #48 from Rosen Diankov 2011-07-03 07:10:51 EDT --- by the way, the fparser version that is ok to package is 4.4.3 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Rich Mattes changed: What|Removed |Added CC||richmat...@gmail.com --- Comment #1 from Rich Mattes 2011-01-31 10:26:33 EST --- It looks like this package is currently using 3 bundled libraries: flann, convexdecomposition, and collada. Flann is up for review right now in bug 672440. The collada stuff appears to be "collada-dom" on sourceforge. And I'm having a hard time tracking down where the convexdecomposition library came from. It's possible to get around shipping bundled libraries under the right circumstances by filing an exception with FESCo: http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries The end of line encoding is easy to fix with sed or dos2unix. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #2 from Tim Niemueller 2011-01-31 11:35:21 EST --- Thanks for the useful comments! I'll use the flann package. New SRPM: http://fedorapeople.org/~timn/robotics/openrave-0.2.18-0.2.svn1975.fc14.src.rpm Spec file is changed in-place. Patch has been updated to include a flann system-wide installation check. The collada-dom in the OpenRAVE tree is only a sub-set of collada-dom, and the CMakeLists.txt explicitly states problems with system-wide installed versions. It only uses the static library, so no shared object is installed that could confuse other apps. I'll look into it for separate packaging. The convexdecomposition library seems to be the one from https://code.google.com/p/convexdecomposition. I'll look into it if it can be packaged separately. Again, it is linked statically into OpenRAVE and should therefore not cause confusion. OpenRAVE includes a Python wrapper. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Peter Lemenkov changed: What|Removed |Added Status|NEW |ASSIGNED CC||lemen...@gmail.com AssignedTo|nob...@fedoraproject.org|lemen...@gmail.com Flag||fedora-review? --- Comment #3 from Peter Lemenkov 2011-02-02 16:02:30 EST --- I'll review it -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Depends on||672440 --- Comment #4 from Tim Niemueller 2011-02-02 16:30:22 EST --- I have further looked into it and collada-dom is a bloody mess. The package was started for the PS3 and it looks that way. Also, the package does not build out of the box. It starts with wrong file names, obviously intended for a system which is case-insensitive, and goes on with compiler errors. I'd like to use the collada part (it's about a quarter of the collada-dom package) that is in the sub-tree. It is integrated into the OpenRAVE build system and statically linked. The convexdecomposition library is intended to be integrated into a project and does not come with an own build system config files. Therefore, again, I'd like to keep it the way it is atm. It hasn't been changed since 2009, so the chance is better to have a maintained version in OpenRAVE than to start a new upstream integrating patches that originate from OpenRAVE. Therefore I'd like to propose the current spec state as the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Rosen Diankov changed: What|Removed |Added CC||rosen.dian...@gmail.com --- Comment #5 from Rosen Diankov 2011-02-02 23:11:15 EST --- hi guys, Thanks for taking the time to create an official RPM. I'll be as helpful as possible. A couple of notes about the 3rd party libraries: 1. collada-dom has local changes, so it cannot be replaced with the official one. I have recently become the maintainer of the collada-dom package, so in the future we'll be able to create an official collada-dom rpm. 2. convexdecomposition also has local changes. I've emailed the author about them, but he seems to be busy with other things. 3. You'll notice the openrave includes both ANN and flann. ANN will be phased out. flann is mostly unmodified except some of the CMakeLists.txt files in order to get openrave plugins to link to it correctly. 4. fparser-4.3 is from this project http://warp.povusers.org/FunctionParser/ Again openrave has many local changes, I've emailed the author about them, but he is not very responsive. 5. crlibm-1.0beta4 is a high-precision math library, it is mostly unmodified except the build system. minizip, pcre-8.02, zlib, and qhull are unmodified and the openrave build system will try to use the system install versions if possible. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #6 from Rosen Diankov 2011-02-02 23:36:17 EST --- By the way, the change log should tell you what svn revisions of openrave are stable: https://sourceforge.net/apps/trac/openrave/wiki/ChangeLog Whenever you want to do a new release, check that page. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Depends on||676159 --- Comment #7 from Tim Niemueller 2011-02-08 18:32:06 EST --- Hi Rosen. Thanks for tuning in and the thorough list of packaged libraries. For packaging it would be much more pleasant if you could make tarball releases. Is that planned or could you think about it? About the libraries: collada-dom: keep as-is for now, once collada-dom is available as an updated package make an RPM, propose for inclusion and eventually use it. convexdecomposition: keep internal version, contains non-upstreamed patches, lib seems to be meant for inclusion (no build system), and is linked statically (i.e. no confusion/errant deps for other software) ann/flann: using system-wide version with my patch already fparser: keep the included version because of the local changes. If upstream becomes responsive revisit issue. crlibm: I have packaged it and it is up for review as bug #676159. minizip, pcre, zlib, qhull: I have made some changes to ensure system-version is used. I have also added statements to %prep stage to delete these directories in the source code before building. New SRPM is at http://fedorapeople.org/~timn/robotics/openrave-0.2.18-0.3.svn1975.fc14.src.rpm. Spec file has been changed in place. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Pavel Zhukov changed: What|Removed |Added Depends on|676159 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Depends on||676159 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #8 from Rosen Diankov 2011-02-09 09:00:56 EST --- i agree with you. in fact, an official tar ball is the number one item on every openrave user's list ;0) we're currently developing a proper testing framework that can track regressions, once this is done we'll immediately start publishing tarballs and releases. For now, I hope you can live with just the change log. and 0.2.19 was just released (r1996). the highlight of this release is ikfast being able to handle arbitrarily complex manipulator kinematics! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #9 from Tim Niemueller 2011-02-21 16:05:57 EST --- Blocking bugs/reviews for flann and crlibm have succeeded and are building or have been issued as an update. So I think it's time to continue with OpenRave. Peter, do you agree that the remaining, and heavily modified version of collada-dom, convexdecomposition, and fparser, which are only built and linked as static libraries, but not installed as shared libraries, should remain in OpenRave until all patches have been integrated into an active upstream? Rosen, even without full unit testing, tarballs would already be helpful and only a minimum of work ;-) I hope we can now proceed quickly and get OpenRave accepted, it's getting a more important component of robotic systems these days, and we're currently preparing Fawkes support for it that we'd like to include in the upcoming release and consequently Fedora. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #10 from Rosen Diankov 2011-02-21 18:22:47 EST --- hi tim, this is great! i'm currently working on unit testing ikfast, i just finished fixing python nose's multiprocess support so that robots taking too long can be immediately terminated without taking valuable testing time. what is the deadline on getting this into the next fedora? the past couple of weeks have seen some major improvements in openrave functionality, and it would be great to have those included. about collada, i committed my changes to the collada-dom trunk. you are willing to create a fedora package directly from trunk, then things should be ok. In that case, you would have to uncomment these lines from openrave's root CMakeLists.txt line 378: find_package(COLLADA) lines 410-418 starting at "# check for local collada compilation" fawkes is very exciting, i see you are the main dev on it ;0) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #11 from Tim Niemueller 2011-02-21 18:36:18 EST --- Rosen, can you make a collada-dom release with your changes? http://sourceforge.net/projects/collada-dom/develop says last commit was on 2011-01-15, is that the latest version you're talking about? Or is that no longer the upstream repository? Do I always have to build all the bells 'n whistles or is there an easy way to build just the libraries? What is it with the 1.4 and 1.5 in the tree, are both needed? Is there an easy way to get rid of the included external libraries, they're really annoying when packaging. Maybe you have a nicely working CMake config for collada-dom? Of we can figure those things out I'd package collada. Peter, please still consider doing the review now with the perspective to splitting of collada-dom as soon as we figured it out, but the collada buildsys looks more like meant to be built in some PS3 environment and self-contained, not to play nicely with a Linux distro. About Fawkes: thanks! A student is currently working on adding OpenRave support (topic branch on http://git.fawkesrobotics.org/fawkes.git) for use with our Katana arm. Afaik you two emailed frequently recently :-) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #12 from Rosen Diankov 2011-02-21 18:46:46 EST --- hi tim, yes those are the changes. i can have a cmake build for collada-dom committed within the week if you are willing to wait. what is the fedora deadline on package releases? there's several people from germany using the katana and openrave, what's your student's name? ;0) I'm not sure if you are subscribed to the openrave-users list, but recently they've been two major additions that will help planning with low-dof arms like the katana: 5DOF IK and move hand straight with translation IK: http://openrave.programmingvision.com/ordocs/en/openravepy-html/openravepy.examples.tutorial_ik5d-module.html http://openrave.programmingvision.com/ordocs/en/openravepy-html/openravepy.examples.movehandstraight2-module.html -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #13 from Tim Niemueller 2011-02-21 19:02:51 EST --- (In reply to comment #12) > hi tim, Hi Rosen. > yes those are the changes. i can have a cmake build for collada-dom committed > within the week if you are willing to wait. what is the fedora deadline on > package releases? There is no deadline, a package is added when it's ready. Packages can be added to the current release and future ones, so there is no need to wait half a year for the next release. It's just a matter of package maintainer patience... So yes, I'll happily wait a week to get that CMake build system if it makes packaging easier and avoids the integrated external libraries. Again, I want to get OpenRave in rather sooner than later in the current stage, which can be upgraded at any time to later known stable versions. So I request again that we go forward with the status quo and improve from there (http://www.jwz.org/doc/worse-is-better.html). Peter? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #14 from Rosen Diankov 2011-02-21 23:01:07 EST --- hi tim, i just committed the cmake build in collada-dom and bumped up its version to 2.3. I also added a *.pc file so you can use it with pkg-config. Basically you would use collada15dom as the name. In order to compile it use: mkdir build; cd build; cmake -DCMAKE_INSTALL_PREFIX=xxx ..; make install openrave r2044 now automatically recognizes the 2.3 installation. for now check out collada-dom direclty from trunk. if you see now problems, i can commit their fixes to trunk right away... maybe we can eventually do a tar ball release of collada-dom 2.3. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Bug 674008 depends on bug 672440, which changed state. Bug 672440 Summary: Review Request: flann - Fast Library for Approximate Nearest Neighbors https://bugzilla.redhat.com/show_bug.cgi?id=672440 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #15 from Peter Lemenkov 2011-02-26 04:58:52 EST --- Hello All! Sorry for the radio silence - I'm still here, and I'm still plan to review it. Will provide more lengthy reply later. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #16 from Rosen Diankov 2011-03-01 10:07:27 EST --- Hi all, openrave r2131 just added support for the Open Asset Import Library: http://assimp.sourceforge.net/ This allows users to specify robot geometry in many more formats than was previously possible: http://assimp.sourceforge.net/main_features_formats.html There is already a discussion in including it into fedora here: https://bugzilla.redhat.com/show_bug.cgi?id=635511 perhaps usage with openrave can speed things up... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #17 from Tim Niemueller 2011-03-01 11:37:40 EST --- Looks like progress on assimp might take a while. Does that eliminate the need for collada? Sorry, I haven't found a chance to look at your collada changes, yet. Will do asap. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #18 from Rosen Diankov 2011-03-01 12:31:17 EST --- collada stays. assimp is used just for extracting geometry from non-collada files, which is then combined with the openrave custom XML for kinematics. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #19 from Rosen Diankov 2011-03-06 02:48:11 EST --- hi guys, just curious what the timeframe is on this release. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #20 from Tim Niemueller 2011-03-06 05:20:38 EST --- Reviews are done when they're done. OpenRAVE is particularly problematic because it uses (and includes) so many non-standard libraries, especially the ones which not available in Fedora, yet. Instead of just packaging OpenRAVE, I'll end up packaging (and maintaining!) more like half a dozen packages in the end... I hope I'll find some time for collada soonish, but a paper deadline is ahead requiring my full attention atm. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #21 from Rosen Diankov 2011-03-06 05:39:02 EST --- ..i'm sorry, i think the only problem here was collada. of all the libraries we talked about, this is the only necessary ones. you can ignore the rest for now. i saw the 'worse is better' article you cited, that's why i thought it would be prudent to hurry up and at least get something out ;0) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Depends on||689432 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674008
--- Comment #22 from Tim Niemueller 2011-03-21 11:51:50
EDT ---
Peter: Sorry, saw your ping in crlibm request only today. Builds have been done
and updates issued. I have packaged an external collada-dom. I'll update the
OpenRAVE package at a later point to use this external library. Can you give
your opinion about the remaining integrated libraries?
Rosen: Please see the collada-dom review at bug #689432. It contains a small
patch, which uses the system-wide installed minizip if available, and which
adds ${LIB_SUFFIX} to install to /usr/lib64 on x86_64. I think it would be safe
to include it in the main package. Would you please tag the 2.3 release or
better, upload tarballs? Thanks!
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #23 from Rosen Diankov 2011-03-21 19:42:50 EDT --- hi tim, I just patched up collada-dom with your patches (r849), thanks! I'll see if i can create a tarball within the day. OpenRAVE should already detect the COLLADA external library. Also, I just patched it OpenRAVE to also use LIB_SUFFIX (r2185). was there any other patches you created for openrave? i couldn't download the sprm. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #25 from Tim Niemueller 2011-03-21 19:58:10 EDT --- Created attachment 486706 --> https://bugzilla.redhat.com/attachment.cgi?id=486706 Patch for upstreaming by Rosen -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #24 from Tim Niemueller 2011-03-21 19:57:25 EDT --- Thanks for applying. Maybe you can make it a 2.3.1 or something. I'm currently looking into assimp. Please let that be the last dependency for a while ;-) Yes, there are a few more things for OpenRAVE. I'll attach the patch to this ticket and happily drop it once you integrated it. I think there are no controversial things in there. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Depends on||689639 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Bug 674008 depends on bug 689639, which changed state. Bug 689639 Summary: Review Request: assimp - Open Asset Import Library https://bugzilla.redhat.com/show_bug.cgi?id=689639 What|Old Value |New Value Resolution||DUPLICATE Status|NEW |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #26 from Rosen Diankov 2011-03-21 21:13:14 EDT --- I have a couple of questions about your patches: 1. why are you doing if(NOT OPENRAVE_OCTAVE_INSTALL_DIR) if(NOT OPENRAVE_PLUGINS_INSTALL_DIR) 2. What happens to local installs when force-installing openravepy in the python site-packages? 3. Would it be better to do a pkg-config thing for crlibm instead of searching for the include files? 4. sympy has local changes which i'm still waiting for them to be integrated in the official releases (i've opened up issues), so don't use the default sympy over the local one in openrave. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674008
--- Comment #27 from Rosen Diankov 2011-03-21
21:36:36 EDT ---
for the *_INSTALL_DIR stuff, perhaps we can append it with "CACHE PATH". For
example:
set(OPENRAVE_OCTAVE_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/share/openrave/octave"
CACHE PATH)
i think this would still enable you to set new paths from the command line.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #28 from Rosen Diankov 2011-03-22 03:34:18 EDT --- just committed r2187 that has most of Tim Niemueller's patches. The crlibm installation search is not included since I would like to see pkg-config used. Also, the installation directories now use CACHE PATH, so they should be settable through the command line. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #29 from Tim Niemueller 2011-03-22 05:31:59 EDT --- Hi Rosen, thanks for integrating the patches. I'll update the package to the new version. So I think Question 1 is set, whatever works to set it from the outside is fine with me. To 2.: I don't really know and I hope you know a solution, maybe an option that I can set from the outside? For the package I just need that functionality. To 3: crlibm does not come with pkg-config files. Even if we added those to Fedora packages, they wouldn't exist in other distros. Hence it is the best to check for header, libraries, and function. I'd like to request that you do add my changes from the patch for the time being. To 4: Is there an upstream bug for the integration process, what is holding them back? Since sympy is included in Fedora there is no way I can use the internal version, it must be fixed upstream. What fixes did you make? I they are required consider filing a bug to get them added to the sympy (Fedora) package, but be prepared to document by mailing list conversation archives, forum or bug tracker entries that the patch is about to be accepted upstream and that the inclusion of the patch in the Fedora package is just a fast track solution for this review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #30 from Rosen Diankov 2011-03-23 01:19:21 EDT --- hi tim, Ok, i added the crlibm check back in (r2188). You should also be able to set the path openravepy installs in by setting OPENRAVEPY_INSTALL_DIR via the command line. As for sympy, the biggest problem I reported is Issue 2081: http://code.google.com/p/sympy/issues/detail?can=2&q=2081&colspec=ID%20Type%20Status%20Priority%20Milestone%20Owner%20Summary%20Stars&id=2081 Basically, it is possible for sympy to give you wrong equations, which would make the openrave ikfast solver return wrong solutions. They are still wondering how to fix it so that everyone's happy. We're looking at the problem from two different angles here: I need to make openrave 100% reliable, you need to reduce unnecessary dependencies so management is easier. I committed a temporary compromise (r2189): If you limit openrave to only sympy 0.6.7, I might have come up with a workaround where you don't have to install a local sympy (ie executing ikfast will replace the wrong function in memory). We can probably support more sympy versions, but they will require IK to be re-tested. There'a an openrave testing server now, so this might be possible in the future: http://www.openrave.org/testing/job/openrave/ If you ever decide to update the sympy version in fedora core, please email me. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #31 from Tim Niemueller 2011-03-23 06:42:37 EDT --- Rosen, thanks for you in-depth reply! It's good to see the check re-added. I'll try another build later today, hopefully without any additional patch! The sympy solution looks good. I will set an explicit dependency in the openrave package on the particular version of sympy. Although I'm not the maintainer of the package, I'd get notice of sympy updates. I'll also request watchcommits permissions to stay on it and contact you if we need to accomodate another sympy version. I'm watching the sympy issue now, it seems stalled with no reply within two months. Maybe you could ping them to move things forward. The assimp library was already work-in-progress and I just missed it. It was stalled for legal reasons, but that seems fixed. Now there is another issue that is being figured out between the assimp and irrlicht developers. Something to be learned from this odyssey: upstream early and often any changes you make, do not integrate 3rd party libs in your own source trees and stay compatible. Peter: not having assimp at the beginning means having an integrated converter library. I'd like to keep this (small) piece in there until we have assimp and can drop it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #32 from Rosen Diankov 2011-03-24 12:07:28 EDT --- just added a tarball release of collada 2.3: https://sourceforge.net/projects/collada-dom/files/Collada%20DOM/Collada%20DOM%202.3/ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #33 from Rosen Diankov 2011-03-25 02:56:16 EDT --- openrave r2195 adds a more intelligent check for sympy in case this bug is fixed in future versions. If this bug isn't fixed and using 0.6.7, it will still use the system install. If none of these conditions apply, it will install the local sympy version. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller changed: What|Removed |Added Depends on||814458 --- Comment #86 from Tim Niemueller 2012-04-19 18:27:43 EDT --- Added fparser library based on cmake stuff in OpenRAVE (missed your last post somehow). Review has been posted and waits for approval (see depending bug). Have you seen version 4.5 and that it removed the eval function? I had packaged that first before realizing that the current OpenRAVE is incompatible. I have added a tiny patch allowing it to build against system sympy 0.7.1. I have yet to run tests to check if that actually works. Rosen, do you have made any tests by now? New SRPM is at http://fedorapeople.org/~timn/robotics/openrave-0.6.4-1.fc15.src.rpm and spec has been changed in place. Once fparser has been approved and imported I can do new scratch builds and hopefully finish this review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #87 from Rosen Diankov 2012-04-19 19:41:28 EDT --- hi tim, unfortunately compatibility for sympy 0.7.1 is not that easy. like promised i'll be looking into compatibility for it shortly and within the week it should be in trunk. but keep in mind it will be officially released in openrave 0.8.0. for now, just package sympy 0.6.x, i'm sure fedora core supports offering rpms with different versions of libraries as long as they are named differently. it's not worth breaking openrave functionality. also, just did a hot fix for openrave 0.6.4 for "openrave-core.pc.in", i highly suggest you repackage the srpm with r3204 of https://openrave.svn.sourceforge.net/svnroot/openrave/tags/0.6.4 as for libfparser, the priority on it is really low. by the time openrave 0.8.0 releases, most likely fparser will have reached a different version ;0) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #88 from Rosen Diankov 2012-04-19 23:44:34 EDT --- just committed another hotfix for 0.6.4, make that r3206 of https://openrave.svn.sourceforge.net/svnroot/openrave/tags/0.6.4 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #89 from Rosen Diankov 2012-04-23 01:25:15 EDT --- hi tim, i had a question on the comment you made here: http://git.fawkesrobotics.org/fawkes.git/commitdiff/eb2deb5bd5757b16c1b68a56cc7bc815319646ae "Also, the OpenRAVE tools gave invalid flags for x86_64 (included 32bit lib dir)." can you give us the current output and expected output? by openrave tools, do you mean the openrave-config program? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #90 from Rosen Diankov 2012-04-23 01:31:33 EDT --- while i am at it, i saw this post: http://git.fawkesrobotics.org/fawkes.git/commitdiff/7488ceba5729fb35b7faab86779434acdc76cf3f you should be able to do: const Vector& trans; __env->plot3(&trans[0], ...) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #92 from Tim Niemueller 2012-05-15 07:12:14 EDT --- > can you give us the current output and expected output? by openrave tools, do > you mean the openrave-config program? The output always includes "-L/usr/lib" which is wrong on x86_64. Also, historically *-config had been used to know which flags were used to compile that particular tool and not what flags should be used to link against it. That has arguably change over the recent years, but I still prefer pkg-config whenever available. (In reply to comment #90) > you should be able to do: > > const Vector& trans; > __env->plot3(&trans[0], ...) Right, that'll be faster. I'll change it. Thanks for catching this. Is there a specific reason why there is no convenience (overridden) method to directly pass the transform? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #91 from Tim Niemueller 2012-05-15 07:08:37 EDT --- (In reply to comment #87) > hi tim, > > unfortunately compatibility for sympy 0.7.1 is not that easy. like promised > i'll be looking into compatibility for it shortly and within the week it > should > be in trunk. but keep in mind it will be officially released in openrave > 0.8.0. > for now, just package sympy 0.6.x, i'm sure fedora core supports offering rpms > with different versions of libraries as long as they are named differently. > it's not worth breaking openrave functionality. Nope, it ain't that easy and it would mean I need to file another review request. I have already done almost half a dozen for OpenRAVE, so I'd rather wait for sympy 0.7 compatibility. > also, just did a hot fix for openrave 0.6.4 for "openrave-core.pc.in", i > highly > suggest you repackage the srpm with r3204 of > > https://openrave.svn.sourceforge.net/svnroot/openrave/tags/0.6.4 I'll have a look. > as for libfparser, the priority on it is really low. by the time openrave > 0.8.0 > releases, most likely fparser will have reached a different version ;0) Well, for now I'm happy with the old fparser version. Let me know when I can upgrade. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #93 from Rosen Diankov 2012-05-15 22:47:48 EDT --- hi tim, still working on sympy 0.7.x compat and should be done soon. If openrave-config is including "-L/usr/lib", it is because some library openrave links to is including /usr/lib in its linked directories. The stringis the following: -L@CMAKE_INSTALL_PREFIX@/lib -lopenrave@OPENRAVE_LIBRARY_SUFFIX@ @OPENRAVE_BOOST_LIB_DIRS@ @Boost_THREAD_LIBRARY_RELEASE@ If you can find out which library (boost most likely?) then you can find out why it is giving wrong info. The openrave latest_stable branch now includes fparser 4.5 since it allows us to compile with LLVM/Clang As for the Vector casts, we removed the operator (float*) cast because several compiles failed because it was clashing with operator[]. For example, which function should v[0] call? I believe it was encouraging unsafe C-type array practices. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #94 from Tim Niemueller 2012-05-16 09:54:34 EDT --- (In reply to comment #93) > still working on sympy 0.7.x compat and should be done soon. Excellent, looking forward to it! > If openrave-config is including "-L/usr/lib", it is because some library > openrave links to is including /usr/lib in its linked directories. The > stringis > the following: > > -L@CMAKE_INSTALL_PREFIX@/lib -lopenrave@OPENRAVE_LIBRARY_SUFFIX@ > @OPENRAVE_BOOST_LIB_DIRS@ @Boost_THREAD_LIBRARY_RELEASE@ I assume CMAKE_INSTALL_PREFIX is /usr resulting in -L/usr/lib. > The openrave latest_stable branch now includes fparser 4.5 since it allows us > to compile with LLVM/Clang Ok, so I'll try the most recent OpenRAVE version with a system-wide installed fparser 4.5 then. > As for the Vector casts, we removed the operator (float*) cast because several > compiles failed because it was clashing with operator[]. For example, which > function should v[0] call? I believe it was encouraging unsafe C-type array > practices. I was more wondering why there is no plot3(OpenRAVE::Vector &trans, ...). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #95 from Rosen Diankov 2012-05-16 10:46:48 EDT --- added @LIB_SUFFIX@ to openrave-config.in in r3285 do you think turning this code: plot3(&v[0],..) to plot3(v,...) justifies a new function and API change? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
