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

Add desktop entry and svg logo #1411

Merged
merged 6 commits into from
Apr 6, 2022
Merged

Conversation

j-rivero
Copy link
Contributor

@j-rivero j-rivero commented Mar 25, 2022

🎉 New feature

Part of gazebo-tooling/release-tools#598

Summary

Include a desktop entry for Ignition Gazebo in upstream sources: the PR is composed by two separate files, a desktop file conformant with version 1.0 of the standard. The other file is the Ignition logo in svg that illustrate the desktop entry.

A PR in the package metadata install the files in the corresponding directories.

Test it

New packages 3.13.0~pre1 are in the prerelease repository. They can be installed and open your desktop menu, search for Ignition and, depending on the desktop manager, will appear something like:

image

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
@codecov
Copy link

codecov bot commented Mar 25, 2022

Codecov Report

Merging #1411 (59358b6) into ign-gazebo3 (1fa99f4) will increase coverage by 0.04%.
The diff coverage is n/a.

❗ Current head 59358b6 differs from pull request most recent head f89544a. Consider uploading reports for the commit f89544a to get more accurate results

@@               Coverage Diff               @@
##           ign-gazebo3    #1411      +/-   ##
===============================================
+ Coverage        77.84%   77.88%   +0.04%     
===============================================
  Files              250      250              
  Lines            14378    14378              
===============================================
+ Hits             11192    11198       +6     
+ Misses            3186     3180       -6     
Impacted Files Coverage Δ
src/SimulationRunner.cc 94.51% <0.00%> (+1.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1fa99f4...f89544a. Read the comment docs.

@chapulina
Copy link
Contributor

Oh I should have mentioned that I tried the pre-release and it worked for me 👍🏽 👍🏽

@chapulina chapulina added the OOBE 📦✨ Out-of-box experience label Mar 25, 2022
@j-rivero j-rivero force-pushed the desktop_entry_ign_gazebo3 branch from 5e1f68a to 5723a33 Compare March 31, 2022 13:47
Signed-off-by: Jose Luis Rivero <[email protected]>
@j-rivero
Copy link
Contributor Author

Tested new changes in: Build Status

3.13.0~pre3-1~focal is the prerelease repository for testing.

@j-rivero j-rivero requested a review from chapulina March 31, 2022 16:51
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.

LGTM! Let's just remember to revert the version bump before merging. We should go through a dedicated PR if we'll make a stable release with this feature.

tools/desktop/ignition-gazebo.desktop.in Outdated Show resolved Hide resolved
CMakeLists.txt Outdated
@@ -3,7 +3,7 @@ cmake_minimum_required(VERSION 3.10.2 FATAL_ERROR)
#============================================================================
# Initialize the project
#============================================================================
project(ignition-gazebo3 VERSION 3.12.0)
project(ignition-gazebo3 VERSION 3.13.0)
Copy link
Contributor

Choose a reason for hiding this comment

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

@j-rivero , I think it's cleaner if we do this in a separate PR and update the changelog as usual. It makes it easier to track the releases and then announce them, and also gives visibility to the team. How about:

  1. Reverting the version changes in this PR and merge it
  2. Opening a new PR for the stable release, with a changelog update including all the recent changes: ignition-gazebo3_3.12.0...ign-gazebo3
  3. Making a stable release

@j-rivero j-rivero requested a review from chapulina April 4, 2022 18:57
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.

🚀

@chapulina chapulina merged commit 2f7bd7e into ign-gazebo3 Apr 6, 2022
@chapulina chapulina deleted the desktop_entry_ign_gazebo3 branch April 6, 2022 00:54
@j-rivero j-rivero mentioned this pull request Apr 7, 2022
@osrf-triage
Copy link

This pull request has been mentioned on Gazebo Community. There might be relevant details there:

https://community.gazebosim.org/t/new-releases-2022-04-27-fortress-citadel/1389/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel OOBE 📦✨ Out-of-box experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants