Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[modules] autoload modules from an other module #1728

Merged
merged 1 commit into from
Jun 15, 2016
Merged

Conversation

gautierhattenberger
Copy link
Member

@gautierhattenberger gautierhattenberger commented Jun 15, 2016

This is not a dependency mechanism. It is just equivalent to adding modules directly in an airframe file.

This should help to solve the issue in gps_modules branch in particular (#1625).

@flixr
Copy link
Member

flixr commented Jun 15, 2016

Great, that helps a lot!
So it looks like you should add a <autoload name="gps_nps"/> element to the module xml, but I'm wondering if it wouldn't make more sense to use the same syntax as for <depends>foo,bar</depends>

@flixr
Copy link
Member

flixr commented Jun 15, 2016

Hm.. I guess this way it is consistent with how modules are loaded otherwise and probably better to keep the parsing the same here (and it's already done ;-))

@flixr flixr merged commit 5b44eb8 into master Jun 15, 2016
@flixr flixr deleted the autoload_modules branch June 15, 2016 19:25
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.

None yet

2 participants