Skip to content
This repository has been archived by the owner. It is now read-only.

Fix for issue #102: allow user to set mongodb db location #103

Merged

Conversation

@gavanderhoorn
Copy link
Member

@gavanderhoorn gavanderhoorn commented Jan 18, 2015

As per subject.

I'm not too happy with the repeated <arg .. /> then <arg .. /> in both default_warehouse_db.launch and demo.launch, but I saw no other way to make this bw compatible.

…os-planning#102.

Also update generated demo.launch accordingly.

The default directory could be located on a non-writable file system, leading
to crashes of the mongodb wrapper script. This change allows the user to specify
an alternative location using the 'db_path' argument.
@mikeferguson
Copy link

@mikeferguson mikeferguson commented Jan 18, 2015

+1

Loading

isucan added a commit that referenced this issue Jan 21, 2015
Fix for issue #102: allow user to set mongodb db location
@isucan isucan merged commit 23a0868 into ros-planning:indigo-devel Jan 21, 2015
@gavanderhoorn gavanderhoorn deleted the issue102_cfgrble_db_path branch Jan 21, 2015
gavanderhoorn added a commit to gavanderhoorn/fanuc that referenced this issue Jan 21, 2015
…ustrial#142.

These changes prevent the MoveIt mongodb wrapper script from running unless
the user explicitly requests it to be enabled by setting `db:=true` when
starting `demo.launch` or `moveit_planning_execution.launch`.

Additionally, when starting the wrapper script from locations that are
non-writable for the current user, the `db_path` launch file argument may
be used to specify an alternate location for the database.

Specifying an alternative location is required when using MoveIt Configuration
packages that have been released as Debians (as those are typically installed
into non-writable directories).

Essentially a backport of the fix in ros-planning/moveit_setup_assistant#103.
gavanderhoorn added a commit to gavanderhoorn/abb that referenced this issue Jan 21, 2015
…ustrial#58.

Released MoveIt configuration packages are installed in non-writable
locations most of the time. Starting `demo.launch` or any other launch
file that starts the mongodb wrapper script results in a 'Permission
denied' error, as it cannot create the database in those directories.

These changes allow the user to configure an alternative location
for the mongodb database by using the `db_path` argument.

Essentially a backport of the fix in ros-planning/moveit_setup_assistant#103.

These fixes have been applied to both the new and the deprecated MoveIt
configuration packages.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants