catkin_create_pkg use package.xml format 2 (#191)
* Switch to package.xml format 2
add format attribute
update depend tag documentation in template
* package_templates.py: update depend tag logic
Replace run_depend with exec_depend
Use more concise depend when possible
* test_templates: expect format 2
* generate build_export_depend tags too
* update order of commented dependency types, fix comment
* better example for doc dependency
[improve] Print package name when maintainer entry is missing. (#189)
* [improve] Print package name when maintainer entry is missing.
E.g. when `rosdep install` hits a package where maintainer entry is missing, it only prints the error message without telling the package name. With this PR package name is printed too.
BEFORE this PR:
```
$ rosdep install -r -y --from-paths src --ignore-src
ERROR: Rosdep experienced an error: Package must declare at least one maintainer
Please go to the rosdep page [1] and file a bug report with the stack trace below.
[1] : http://www.ros.org/wiki/rosdep
rosdep version: 0.11.7
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/rosdep2/main.py", line 137, in rosdep_main
exit_code = _rosdep_main(args)
File "/usr/lib/python2.7/dist-packages/rosdep2/main.py", line 383, in _rosdep_main
return _package_args_handler(command, parser, options, args)
File "/usr/lib/python2.7/dist-packages/rosdep2/main.py", line 436, in _package_args_handler
pkgs = find_catkin_packages_in(path, options.verbose)
File "/usr/lib/python2.7/dist-packages/rosdep2/catkin_packages.py", line 33, in find_catkin_packages_in
packages = find_packages(path)
File "/usr/lib/python2.7/dist-packages/catkin_pkg/packages.py", line 87, in find_packages
packages = find_packages_allowing_duplicates(basepath, exclude_paths=exclude_paths, exclude_subspaces=exclude_subspaces, warnings=warnings)
File "/usr/lib/python2.7/dist-packages/catkin_pkg/packages.py", line 144, in find_packages_allowing_duplicates
path_parsed_packages, warnings_lists = zip(*pool.map(parser, data))
File "/usr/lib/python2.7/multiprocessing/pool.py", line 251, in map
return self.map_async(func, iterable, chunksize).get()
File "/usr/lib/python2.7/multiprocessing/pool.py", line 558, in get
raise self._value
InvalidPackage: Package must declare at least one maintainer
```
AFTER this PR:
```
$ rosdep install -r -y --from-paths src --ignore-src
ERROR: Rosdep experienced an error: Package 'aubo_trajectory_filters' must declare at least one maintainer
Please go to the rosdep page [1] and file a bug report with the stack trace below.
[1] : http://www.ros.org/wiki/rosdep
rosdep version: 0.11.7
Traceback (most recent call last):
File "/home/noodlemachine/ROS/ncws/src/ros-infrastructure/rosdep/src/rosdep2/main.py", line 137, in rosdep_main
exit_code = _rosdep_main(args)
File "/home/noodlemachine/ROS/ncws/src/ros-infrastructure/rosdep/src/rosdep2/main.py", line 384, in _rosdep_main
return _package_args_handler(command, parser, options, args)
File "/home/noodlemachine/ROS/ncws/src/ros-infrastructure/rosdep/src/rosdep2/main.py", line 437, in _package_args_handler
pkgs = find_catkin_packages_in(path, options.verbose)
File "/home/noodlemachine/ROS/ncws/src/ros-infrastructure/rosdep/src/rosdep2/catkin_packages.py", line 33, in find_catkin_packages_in
packages = find_packages(path)
File "/home/noodlemachine/ROS/ncws/src/ros-infrastructure/catkin_pkg/src/catkin_pkg/packages.py", line 86, in find_packages
packages = find_packages_allowing_duplicates(basepath, exclude_paths=exclude_paths, exclude_subspaces=exclude_subspaces, warnings=warnings)
File "/home/noodlemachine/ROS/ncws/src/ros-infrastructure/catkin_pkg/src/catkin_pkg/packages.py", line 143, in find_packages_allowing_duplicates
path_parsed_packages, warnings_lists = zip(*pool.map(parser, data))
File "/usr/lib/python2.7/multiprocessing/pool.py", line 251, in map
return self.map_async(func, iterable, chunksize).get()
File "/usr/lib/python2.7/multiprocessing/pool.py", line 558, in get
raise self._value
InvalidPackage: Package 'aubo_trajectory_filters' must declare at least one maintainer
```
* use " outside of '
"--skip-merge" does not change any outputs (#185)
* "--skip-merge" does not change any outputs
`catkin_generate_changelog` outputs changelog with what we called with `--skip-merge` option, and reason is that `git show` does not show files for merged commit, we need '-m' option
```
k-okada@p40-yoga:/tmp/jsk_3rdparty$ git show 6a0bfaaf29bcc9a8e07e6edfccd230ad7f003070 --name-only --format=format:""
k-okada@p40-yoga:/tmp/jsk_3rdparty$ git show 6a0bfaaf29bcc9a8e07e6edfccd230ad7f003070
commit 6a0bfaaf29bcc9a8e07e6edfccd230ad7f003070
Merge: 5f6888c 54ea9bd
Author: Kei Okada <email address hidden>
Date: Fri Jul 7 11:32:32 2017 +0900
Merge pull request #114 from k-okada/add_unzip
add unzip to build_depend
```
* use --first-parent, instead of -m