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

rename yast-sshd to yast-ssh-server #58

Merged
merged 1 commit into from
Oct 7, 2015
Merged

Conversation

bmwiedemann
Copy link
Contributor

it was already renamed in some other places (e.g. icons/hicolor/scalable/apps/yast-ssh-server.svg )
and this makes it consistent

This is needed for yast/skelcd-control-openSUSE#35

it was already renamed in some other places
and this makes it consistent
- needed for yast/skelcd-control-openSUSE#35
@jreidinger
Copy link
Member

@kobliha do you know if it still make sense? and I am not sure if it can be changed this way without change in other respective places.

@bmwiedemann
Copy link
Contributor Author

as far as I saw, the only usage is dropped in the first commit of yast/skelcd-control-openSUSE#35
and it is already half-broken atm, because some of the files already come with the yast-ssh-server name and some dont

@jreidinger
Copy link
Member

ok, so lets merge it

jreidinger added a commit that referenced this pull request Oct 7, 2015
rename yast-sshd to yast-ssh-server
@jreidinger jreidinger merged commit 75fad49 into yast:master Oct 7, 2015
@kobliha
Copy link
Member

kobliha commented Oct 7, 2015

There is no Yast SSH* anymore. But the icon is IIRC used in some installation dialog... This will probably break it...

@kobliha
Copy link
Member

kobliha commented Oct 7, 2015

@bmwiedemann @jreidinger OK, checked, you have just broken several places in Yast (SP1): https://github.com/search?q=yast-sshd+user%3Ayast&ref=searchresults&type=Code&utf8=%E2%9C%93

@jreidinger
Copy link
Member

as this change do not bump version it do not make SP1

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

3 participants