Skip to content

ROCK_BUNDLE_PATH should only define super folder of bundles#16

Merged
doudou merged 1 commit into
rock-core:masterfrom
maltewi:bundle_path
Jan 26, 2015
Merged

ROCK_BUNDLE_PATH should only define super folder of bundles#16
doudou merged 1 commit into
rock-core:masterfrom
maltewi:bundle_path

Conversation

@maltewi
Copy link
Copy Markdown
Contributor

@maltewi maltewi commented Nov 10, 2014

ROCK_BUNDLE_PATH should only contain higher-level folder where bundles can be found as subfolders. It should not contain the bundle folders themselves.

…s can be found as subfolders. It should not contain the bundle folders themselves.
@doudou
Copy link
Copy Markdown
Member

doudou commented Nov 10, 2014

Actually, it could by design. The idea was that not all bundles might end up in a "bundle" folder.

I am open for discussion there, though. What was your problem ?

@maltewi
Copy link
Copy Markdown
Contributor Author

maltewi commented Nov 11, 2014

I had two problems (possibly related):

  1. Errors from another bundle leak into the current one.
    E.g. put some random text into models/actions/main.rb of bundle 'B'. It
    will break a syskit instanciate command in another bundle ('A') which
    has no dependency on bundle 'B'.

  2. Searching for files behaves not as expected.
    I would expect that Roby.app.find_files_in_dirs is limited to the
    loaded bundles, but is searches within all folders defined in
    ROCK_BUNDLE_PATH.

@doudou
Copy link
Copy Markdown
Member

doudou commented Nov 13, 2014

Should definitely not happen. My guess is that modifying the ROCK_BUNDLE_PATH the way you want actually simply removes the problematic bundles from the search patch altogether.

@maltewi
Copy link
Copy Markdown
Contributor Author

maltewi commented Jan 26, 2015

The probem is occuring to other people as well. The suggested solution so far worked for them. Maybe merge for now and, if the solution is not quite perfect, open an issue for improvements?

doudou added a commit that referenced this pull request Jan 26, 2015
ROCK_BUNDLE_PATH should only define super folder of bundles
@doudou doudou merged commit 83ec42a into rock-core:master Jan 26, 2015
@doudou
Copy link
Copy Markdown
Member

doudou commented Jan 26, 2015

Agreed and done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants