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

sync up with gbp stuff #25

Closed
wants to merge 19 commits into from
Closed

sync up with gbp stuff #25

wants to merge 19 commits into from

Conversation

ethanrublee
Copy link
Contributor

This is still a work in progress. I have a few fixes related to python 2.6 version.
Also a bit of cmake for making a dist tarball and gbp. Expect gbp stuffs to change a lot as i figure out the work flow.

@ethanrublee
Copy link
Contributor Author

marking invalid.

dirk-thomas pushed a commit that referenced this pull request Jan 28, 2019
* Create env.bat and setup.bat in build_cmake_package for Windows (#9)

* Skip the drive letter if run on Windows (#10)

* Skip the drive letter if run on Windows

* Remove unnecessary check for win32

* Provide top level devel space environment and setup bat files for Windows (#11)

* use os.pathsep to join path components

* revert whitespace change

* Fix path backslashes in catkin_package.cmake (#25)

* Update catkin_package.cmake

* Update catkin_package.cmake

* Update builder.py

* Normalize the path from subs['cmake_prefix_path']

* Revert the unrelated changes from my bad merge in the previous commit.

Revert the unrelated changes from my bad merge in the previous commit to keep this PR clean.

* use underscope to exclude drive letter, and clarify there's no effect on Linux (#31)

* Define string templates in separate steps and update comment (#32)

* Create env.bat and setup.bat in build_cmake_package for Windows (#9)

* define string templates in a separate step

* change script writer function names

* fix indentation for script template

* fix env script name typo (#33)

* minor nitpicks
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.

1 participant