[kdenetwork-filesharing] [Bug 407845] After installing Samba, recommend rebooting

2020-03-16 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=407845

--- Comment #4 from Nate Graham  ---
It's easier to just tell the novice user that they need to reboot, so you don't
need to explain the technical details. Advanced users who understand what's
going on can log out/log-back-in manually.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenetwork-filesharing] [Bug 407845] After installing Samba, recommend rebooting

2020-03-15 Thread Jetchko Jekov
https://bugs.kde.org/show_bug.cgi?id=407845

Jetchko Jekov  changed:

   What|Removed |Added

 CC||jetchko.je...@gmail.com

--- Comment #3 from Jetchko Jekov  ---
I am sorry but that's wrong. You don't need a reboot for change in group
membership to be taken into account, you need only to perform logout/login
sequence.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenetwork-filesharing] [Bug 407845] After installing Samba, recommend rebooting

2020-03-14 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=407845

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||20.04.0
 Status|CONFIRMED   |RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/kde
   ||network-filesharing/8a104aa
   ||4fb4194708868fb58a5036ffa2e
   ||aec31d

--- Comment #2 from Nate Graham  ---
Git commit 8a104aa4fb4194708868fb58a5036ffa2eaec31d by Nate Graham.
Committed on 15/03/2020 at 01:05.
Pushed by ngraham into branch 'master'.

Recommend rebooting after installing Samba

Summary:
After Samba is installed, very frequently it will not work correctly until the
machine is
rebooted. One potential reason is when the installed package has made group
membership
changes, which only take effect after a reboot.

This patch implements a reboot recommendation along with a button to reboot the
machine.
Both are shown immediately after Samba has been installed.

Because some expert users may understand the technical details of what's going
on, the
reboot is only recommended, not required. If the window is closed and then
re-opened,
the normal Samba sharing configuration UI is displayed instead of the reboot
prompt.

Added a new function rather than using a Lambda because it may be useful for
additional
purposes too (e.g. https://bugs.kde.org/show_bug.cgi?id=407846)
FIXED-IN: 20.04.0

Test Plan:
http://s1.webmshare.com/Ry55q.webm (not uploaded to Phab due to file size
limit)

1. Remove Samba
2. Go to share a folder
3. Click "Install Samba"
4. After the installation has completed, click the "Restart" button and see
that the machine reboots

5. Remove Samba again
2. Go to share a folder
3. Click "Install Samba"
4. Close the window and re-open it instead of rebooting as recommended
5. See that the Samba sharing config UI is all there

Reviewers: #vdg, #frameworks, #dolphin, apol, sitter

Reviewed By: sitter

Subscribers: anthonyfieroni, sitter, bruns

Differential Revision: https://phabricator.kde.org/D21466

M  +25   -1samba/filepropertiesplugin/sambausershareplugin.cpp
M  +5-0samba/filepropertiesplugin/sambausershareplugin.h

https://commits.kde.org/kdenetwork-filesharing/8a104aa4fb4194708868fb58a5036ffa2eaec31d

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenetwork-filesharing] [Bug 407845] After installing Samba, recommend rebooting

2019-05-28 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=407845

--- Comment #1 from Nate Graham  ---
Patch: https://phabricator.kde.org/D21466

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenetwork-filesharing] [Bug 407845] After installing Samba, recommend rebooting

2019-05-24 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=407845

Patrick Silva  changed:

   What|Removed |Added

 CC||bugsefor...@gmx.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenetwork-filesharing] [Bug 407845] After installing Samba, recommend rebooting

2019-05-23 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=407845

Nate Graham  changed:

   What|Removed |Added

   See Also||https://bugs.kde.org/show_b
   ||ug.cgi?id=407846

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenetwork-filesharing] [Bug 407845] After installing Samba, recommend rebooting

2019-05-22 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=407845

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.