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

fix boo#898861: Duplicated entry for Service Manager in Yast2 #9

Merged
merged 2 commits into from
Nov 4, 2014
Merged

fix boo#898861: Duplicated entry for Service Manager in Yast2 #9

merged 2 commits into from
Nov 4, 2014

Conversation

super7ramp
Copy link
Contributor

YaST2 control center (Qt) displays a duplicate entry for Services manager. It is because there are actually two .desktop files for this module, for backward compatibility, but only one should be displayed. The other one has a Hidden flag and should be ignored.

This change makes YaST2 control center ignore desktop file with Hidden flag activated.

YaST2 control center (Qt) displays a duplicate entry for Services manager. It is because there are actually two .desktop files for this module, for backward compatibility, but only one should be displayed. The other one has a Hidden flag and should be ignored.

This change makes YaST2 control center ignore desktop file with Hidden flag activated.
@jreidinger
Copy link
Member

Change loooks good, just please add changes to changelog and increase version.

@super7ramp
Copy link
Contributor Author

Changelog updated and version increased to 3.1.4.

@jreidinger
Copy link
Member

thanks for your patch

jreidinger added a commit that referenced this pull request Nov 4, 2014
fix boo#898861: Duplicated entry for Service Manager in Yast2
@jreidinger jreidinger merged commit a93526b into yast:master Nov 4, 2014
@super7ramp
Copy link
Contributor Author

You're welcome :)

@super7ramp super7ramp deleted the fixing-boo#898861 branch November 7, 2014 05:18
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.

None yet

2 participants