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

require latest uncrustify version from source #20

Closed
tfoote opened this issue Apr 11, 2015 · 5 comments
Closed

require latest uncrustify version from source #20

tfoote opened this issue Apr 11, 2015 · 5 comments
Assignees

Comments

@tfoote
Copy link
Member

tfoote commented Apr 11, 2015

We need the latest version 0.61. This should be documented in our install instructions: https://github.com/ros2/examples/wiki as well as in our CI jobs.

@tfoote
Copy link
Member Author

tfoote commented Apr 11, 2015

Actually we need features more recent than the most recent 0.61, only in trunk.

Including but not limited to:

'string_replace_tab_chars'
'disable_processing_cmt'
'enable_processing_cmt'
'indent_class_on_colon'
'indent_shift'
'indent_min_vbrace_open'
'indent_vbrace_open_on_tabstop'
'sp_after_mdatype_commas'
'sp_before_mdatype_commas'
'sp_between_mdatype_commas'
'sp_cmt_cpp_doxygen'
'sp_between_new_paren'
'nl_after_label_colon'

Upstream source is here:
https://github.com/bengardner/uncrustify

@dirk-thomas dirk-thomas self-assigned this Apr 14, 2015
@tfoote
Copy link
Member Author

tfoote commented Apr 14, 2015

We need >0.61 We'll create an ament package to build the newest version and make it available on all 3 platforms.

@tfoote tfoote added the ready Work is about to start (Kanban column) label Apr 14, 2015
@dirk-thomas dirk-thomas added in progress Actively being worked on (Kanban column) and removed ready Work is about to start (Kanban column) labels Apr 14, 2015
@dirk-thomas dirk-thomas changed the title document uncrustify requirements and intall procedures require latest uncrustify version from source Apr 14, 2015
@dirk-thomas
Copy link
Contributor

I have forked the uncrustify repo (https://github.com/ament/uncrustify) and created a package.xml as well as CMakeLists.txt file to invoke configure && make for the package and install the binary in the bin folder.

The referenced PR will add the new repo to the ros2.repos file.

The latest test results (using this custom repos list) show that the uncrustify checks do pass now: http://54.183.26.131:8080/view/ros2/job/ros2_batch_ci_osx/88/testReport/

I could not yet get it to work on Windows. But the PR should be merged to make it work on the other platforms. I will continue to try to build it on Windows...

@dirk-thomas
Copy link
Contributor

Waiting on Windows build to confirm that the latest changes (ament/uncrustify@96f42bc) actually make the uncrustify tests pass on Windows (ros2/ros2#34 (comment)).

@dirk-thomas
Copy link
Contributor

After more debugging the latest build (http://54.183.26.131:8080/view/ros2/job/ros2_batch_ci_windows/166/) actually builds uncrustify successfully. The corresponding tests still fail but it looks like it is unrelated to the uncrustify binary itself.

@dirk-thomas dirk-thomas removed the in progress Actively being worked on (Kanban column) label Apr 17, 2015
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

No branches or pull requests

2 participants