<br><br><div class="gmail_quote">On Fri, May 20, 2011 at 4:29 PM, Sascha Silbe <span dir="ltr"><<a href="mailto:silbe@activitycentral.com">silbe@activitycentral.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Excerpts from Rafael Ortiz's message of Thu May 19 18:52:59 +0200 2011:<br>
<div class="im"><br>
> Temporaly Disabling become root button on terminal activity<br>
> avoiding problems reported SL#2756 and #2597, also in merge request on gitorious<br>
<br>
</div>CC'ing Gary and Walter to discuss the UI side of this.<br>
As discussed in both tickets [1,2] we might want to get rid of this<br>
button permanently - at least in its current form. It's dangerous<br>
(kills the currently running process(es)) and just a shorthand for<br>
"su -".<br>
<br>
While getting rid of the button without any replacement would make root<br>
access less easy to discover, I'd argue that without a minimal<br>
introduction the shell functionality is equally "hard" to discover. The<br>
same minimal introduction (/etc/motd?) could mention "sudo -i".<br>
<div class="im"><br>
<br></div></blockquote><div><br></div><div>Thanks for the review Sasha, I also think we will need further introductions on how to become root, although at this moment I don't know how to add this from the activity itself without having a mod on the os, IIRC olpc's term had this info along with a ''welcome children of the world'' intro , although dextrose don't. (I might be wrong though), ideas welcomed. </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">
[terminal.py]<br>
> @@ -86,17 +86,17 @@ class TerminalActivity(activity.Activity):<br>
> tab_toolbar_button.show()<br>
><br>
> # Add a button that will be used to become root easily.<br>
> - root_button = ToolButton('activity-become-root')<br>
> - root_button.set_tooltip(_('Become root'))<br>
[...]<br>
</div>> + #root_button = ToolButton('activity-become-root')<br>
<div class="im">> + #root_button.set_tooltip(_('Become root'))<br>
</div>[...]<br>
<br>
Please just remove the lines. If we want to bring them back later, we<br>
can just revert the patch. That's what a VCS is for after all.<br>
<br></blockquote><div>Right!. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Sascha<br>
<br>
[1] <a href="https://bugs.sugarlabs.org/ticket/2597" target="_blank">https://bugs.sugarlabs.org/ticket/2597</a><br>
[2] <a href="https://bugs.sugarlabs.org/ticket/2756" target="_blank">https://bugs.sugarlabs.org/ticket/2756</a><br>
<font color="#888888">--<br>
<a href="http://sascha.silbe.org/" target="_blank">http://sascha.silbe.org/</a><br>
<a href="http://www.infra-silbe.de/" target="_blank">http://www.infra-silbe.de/</a><br>
</font></blockquote></div><br>