Date: Thu, 20 Nov 2008 01:27:58 GMT From: Joe <fbsd1@a1poweruser.com> To: freebsd-gnats-submit@FreeBSD.org Subject: ports/129002: add patch to x11/slim-1.3.1 Message-ID: <200811200127.mAK1Rw8U015609@www.freebsd.org> Resent-Message-ID: <200811200130.mAK1U1wY066108@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 129002 >Category: ports >Synopsis: add patch to x11/slim-1.3.1 >Confidential: no >Severity: non-critical >Priority: medium >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: maintainer-update >Submitter-Id: current-users >Arrival-Date: Thu Nov 20 01:30:01 UTC 2008 >Closed-Date: >Last-Modified: >Originator: Joe >Release: 7.1 >Organization: >Environment: >Description: On the developers website there is a patch i would like added to x11/slim-1.3.1 http://developer.berlios.de/patch/?func=detailpatch&patch_id=2283&group_id=2663 [ Patch #2283 ] Add a variable to run shutdown commands without root pass. This is the contents of the patch file From: Nicolas Pierron <nicolas.b.pierron+berlios@gmail.com> Subject: Add a variable to run shutdown commands without root password. URL: http://svn.berlios.de/svnroot/repos/slim/trunk ChangeLog: 2007-12-16 Nicolas Pierron <nicolas.pierron@lrde.epita.fr> Add a variable to run system command without root password. * app.cpp: Add the test for reboot, halt and suspend. * cfg.cpp: Add the new variable with the default value set to false. * slim.conf: Add an example of the command. --- app.cpp | 5 +++++ cfg.cpp | 1 + slim.conf | 5 +++++ 3 files changed, 11 insertions(+) Index: slim.conf =================================================================== --- slim.conf (revision 150) +++ slim.conf (working copy) @@ -10,6 +10,11 @@ console_cmd /usr/X11R6/bin/xterm -C -fg white -bg black +sb -T "Console login" -e /bin/sh -c "/bin/cat /etc/issue; exec /bin/login" #suspend_cmd /usr/sbin/suspend +# Let normal users have access to systems commands. If the value is true, +# then the root password is requiered to start a system command. +# Valid values: true|false +# root_password false + # Full path to the xauth binary xauth_path /usr/X11R6/bin/xauth Index: cfg.cpp =================================================================== --- cfg.cpp (revision 150) +++ cfg.cpp (working copy) @@ -36,6 +36,7 @@ options.insert(option("login_cmd","exec /bin/bash -login ~/.xinitrc %session")); options.insert(option("halt_cmd","/sbin/shutdown -h now")); options.insert(option("reboot_cmd","/sbin/shutdown -r now")); + options.insert(option("root_password","true")); options.insert(option("suspend_cmd","")); options.insert(option("sessionstart_cmd","")); options.insert(option("sessionstop_cmd","")); Index: app.cpp =================================================================== --- app.cpp (revision 150) +++ app.cpp (working copy) @@ -407,6 +407,11 @@ case Panel::Console: cerr << APPNAME << ": Got a special command (" << LoginPanel->GetName() << ")" << endl; return true; // <--- This is simply fake! + case Panel::Suspend: + case Panel::Halt: + case Panel::Reboot: + if (cfg->getOption("root_password") == "false") + return true; default: break; }; >How-To-Repeat: >Fix: Patch attached with submission follows: From: Nicolas Pierron <nicolas.b.pierron+berlios@gmail.com> Subject: r???: Add a variable to run shutdown commands without root password. URL: http://svn.berlios.de/svnroot/repos/slim/trunk ChangeLog: 2007-12-16 Nicolas Pierron <nicolas.pierron@lrde.epita.fr> Add a variable to run system command without root password. * app.cpp: Add the test for reboot, halt and suspend. * cfg.cpp: Add the new variable with the default value set to false. * slim.conf: Add an example of the command. --- app.cpp | 5 +++++ cfg.cpp | 1 + slim.conf | 5 +++++ 3 files changed, 11 insertions(+) Index: slim.conf =================================================================== --- slim.conf (revision 150) +++ slim.conf (working copy) @@ -10,6 +10,11 @@ console_cmd /usr/X11R6/bin/xterm -C -fg white -bg black +sb -T "Console login" -e /bin/sh -c "/bin/cat /etc/issue; exec /bin/login" #suspend_cmd /usr/sbin/suspend +# Let normal users have access to systems commands. If the value is true, +# then the root password is requiered to start a system command. +# Valid values: true|false +# root_password false + # Full path to the xauth binary xauth_path /usr/X11R6/bin/xauth Index: cfg.cpp =================================================================== --- cfg.cpp (revision 150) +++ cfg.cpp (working copy) @@ -36,6 +36,7 @@ options.insert(option("login_cmd","exec /bin/bash -login ~/.xinitrc %session")); options.insert(option("halt_cmd","/sbin/shutdown -h now")); options.insert(option("reboot_cmd","/sbin/shutdown -r now")); + options.insert(option("root_password","true")); options.insert(option("suspend_cmd","")); options.insert(option("sessionstart_cmd","")); options.insert(option("sessionstop_cmd","")); Index: app.cpp =================================================================== --- app.cpp (revision 150) +++ app.cpp (working copy) @@ -407,6 +407,11 @@ case Panel::Console: cerr << APPNAME << ": Got a special command (" << LoginPanel->GetName() << ")" << endl; return true; // <--- This is simply fake! + case Panel::Suspend: + case Panel::Halt: + case Panel::Reboot: + if (cfg->getOption("root_password") == "false") + return true; default: break; }; >Release-Note: >Audit-Trail: >Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200811200127.mAK1Rw8U015609>