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

ign -> gz CLI Migration : gz-tools #94

Merged
merged 10 commits into from
Jun 21, 2022
Merged

ign -> gz CLI Migration : gz-tools #94

merged 10 commits into from
Jun 21, 2022

Conversation

methylDragon
Copy link
Contributor

@github-actions github-actions bot added the 🌱 garden Ignition Garden label Jun 12, 2022
@methylDragon methylDragon marked this pull request as ready for review June 13, 2022 05:37
@methylDragon methylDragon force-pushed the cli_migration branch 4 times, most recently from 567a708 to 5c77b63 Compare June 13, 2022 06:52
@methylDragon
Copy link
Contributor Author

The CMakeLists.txt changes are quite scuffed.

I needed to contort it because execute_process isn't expanding the $ generator expression...

@chapulina chapulina added the ign to gz Renaming Ignition to Gazebo. label Jun 13, 2022
src/gz.in Outdated Show resolved Hide resolved
src/gz.in Outdated Show resolved Hide resolved
etc/gz.bash_completion.sh Show resolved Hide resolved
Signed-off-by: methylDragon <[email protected]>
@methylDragon methylDragon force-pushed the cli_migration branch 2 times, most recently from 285a312 to 373ebf1 Compare June 16, 2022 21:20
Signed-off-by: methylDragon <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Windows tick-tock was failing, I tried various things without success, they're documented on the commit history of this PR. We don't need to block on Windows tick-tock, since Windows support is very limited at the moment anyway (see #7). So let's proceed as is.

@chapulina chapulina merged commit 30fa4d9 into main Jun 21, 2022
@chapulina chapulina deleted the cli_migration branch June 21, 2022 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱 garden Ignition Garden ign to gz Renaming Ignition to Gazebo.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants