Skip to content
This repository has been archived by the owner on Feb 4, 2020. It is now read-only.

Use verb pattern #4

Merged
merged 15 commits into from
Aug 12, 2014
Merged

Use verb pattern #4

merged 15 commits into from
Aug 12, 2014

Conversation

wjwwood
Copy link
Contributor

@wjwwood wjwwood commented Aug 8, 2014

This is a major refactor for the verb's to use the generalized "verb pattern" now in osrf_pycommon:

osrf/osrf_pycommon#6

This pull request also refactors build and build_pkg to use the concept of BuildType and BuildActions to define build workflows for different build_type's.

Todo:

  • Use osrf_pycommon's "verb pattern":
    • build
    • build_pkg
    • list_packages
    • package_name
    • package_version
    • test
    • test_pkg
  • Use BuildAction and BuildType pattern:
    • build
    • build_pkg

As well as documentation about how to use them
to extend the supported build types for
ament_tools.
They now use the BuildType and BuildAction API's.
- Made everything verb agnostic
- Changed the entry_point name
- Moved the source for the ament_cmake build_type
- Refactored discovery and loading of build_types
out of the build_pkg verb module.
@wjwwood wjwwood mentioned this pull request Aug 12, 2014
3 tasks
wjwwood added a commit that referenced this pull request Aug 12, 2014
@wjwwood wjwwood merged commit 62d3c84 into master Aug 12, 2014
@wjwwood wjwwood deleted the use_verb_pattern branch August 12, 2014 01:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant