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

doc: Add a .desktop and .appdata.xml file for Cockpit #1006

Closed
wants to merge 3 commits into from
Closed

doc: Add a .desktop and .appdata.xml file for Cockpit #1006

wants to merge 3 commits into from

Conversation

hughsie
Copy link
Contributor

@hughsie hughsie commented Aug 11, 2014

This allows the user to launch Cockpit from the shell and also makes it visible
to be installed in GNOME Software.

IMPORTANT: This is not anyware good enough for merging, but we can work through the FIXMEs if this is a feature you guys actually want. :)

This allows the user to launch Cockpit from the shell and also makes it visible
to be installed in GNOME Software.
@stefwalter
Copy link
Contributor

Seems like something we could merge. Installation via GNOME software isn't a primary use case, but I think it's not a bad idea.

By the way, make distcheck fails:

  ITMRG  doc/cockpit.appdata.xml
Cannot open doc/cockpit.appdata.xml: No such file or directory
make[3]: *** [doc/cockpit.appdata.xml] Error 2
make[3]: Leaving directory `/home/travis/build/cockpit-project/cockpit/cockpit-0.19/_build'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/travis/build/cockpit-project/cockpit/cockpit-0.19/_build'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/home/travis/build/cockpit-project/cockpit/cockpit-0.19/_build'
make: *** [distcheck] Error 1

[Desktop Entry]
_Name=Cockpit
_Comment=A user interface for GNU/Linux servers
# FIXME: probably need a more sexy icon here...
Copy link
Contributor

Choose a reason for hiding this comment

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

This would be the icon from http://cockpit-project.org/

@hughsie
Copy link
Contributor Author

hughsie commented Aug 11, 2014

Okay, thanks. I wonder if in the meantime Andreas could do 3-5 official screenshots of the latest code and put them up on the website somewhere? I've fixed the URL, done a 64px icon, and fixed make distcheck.

@stefwalter
Copy link
Contributor

@andreasn do you have any 16:9 screenshots of cockpit?

@andreasn
Copy link
Contributor

Will provide shortly!

@mvollmer
Copy link
Member

One issue worth thinking about would be whether or not to open the firewall for the Cockpit port when people install Cockpit for local use.

If the description makes it clear that Cockpit is also accessible remotely by default, I think we should just open it, for simplicity.

@andreasn
Copy link
Contributor

I hope I did the last one right

@hughsie
Copy link
Contributor Author

hughsie commented Aug 26, 2014

Looks good to me, thanks! Stef, do you want to collapse this down to one commit manually and pull to master?

@mvollmer
Copy link
Member

Continued in #1085.

@mvollmer mvollmer closed this Aug 27, 2014
stefwalter pushed a commit that referenced this pull request Sep 1, 2014
This allows the user to launch Cockpit from the shell and also makes it visible
to be installed in GNOME Software.

Fixes #1006
Reviewed-by: Stef Walter <stefw@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants