[Freeipa-devel] [PATCH] 0019 Use reboot from /sbin
Commit message says it all. So does the ticket. https://fedorahosted.org/freeipa/ticket/2480 -- Petr³ From aad19e793e3ea882ef5069d678f9ba739b2e6eb4 Mon Sep 17 00:00:00 2001 From: Petr Viktorin pvikt...@redhat.com Date: Fri, 2 Mar 2012 07:18:56 -0500 Subject: [PATCH] Use reboot from /sbin According to FHS, the reboot command should live in /sbin. Systems may also have a symlink in /usr/bin, but they don't have to. https://fedorahosted.org/freeipa/ticket/2480 --- ipa-client/ipa-install/ipa-client-install |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ipa-client/ipa-install/ipa-client-install b/ipa-client/ipa-install/ipa-client-install index f5c1efe0686020ce7cad79edfe19908ee3a55a30..7d405a8d7c17d8ec626d018eeda02bf234805dce 100755 --- a/ipa-client/ipa-install/ipa-client-install +++ b/ipa-client/ipa-install/ipa-client-install @@ -409,7 +409,7 @@ def uninstall(options, env, quiet=False): if not options.on_master: if user_input(Do you want to reboot the machine?, False): try: -run([/usr/bin/reboot]) +run([/sbin/reboot]) except Exception, e: emit_quiet(quiet, Reboot command failed to exceute. + str(e)) return CLIENT_UNINSTALL_ERROR -- 1.7.7.6 ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0019 Use reboot from /sbin
On Fri, Mar 02, 2012 at 01:28:56PM +0100, Petr Viktorin wrote: Commit message says it all. So does the ticket. https://fedorahosted.org/freeipa/ticket/2480 -- Petr³ Does it matter? It the UsrMoved world, both are just symlinks to systemctl.. [root@vm-146 ~]# ll /sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /sbin/reboot - ../bin/systemctl [root@vm-146 ~]# ll /usr/sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /usr/sbin/reboot - ../bin/systemctl [root@vm-146 ~]# cat /etc/redhat-release Fedora release 18 (Rawhide) ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0019 Use reboot from /sbin
On 03/02/2012 01:42 PM, Jakub Hrozek wrote: On Fri, Mar 02, 2012 at 01:28:56PM +0100, Petr Viktorin wrote: Commit message says it all. So does the ticket. https://fedorahosted.org/freeipa/ticket/2480 -- Petr³ Does it matter? It the UsrMoved world, both are just symlinks to systemctl.. [root@vm-146 ~]# ll /sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /sbin/reboot - ../bin/systemctl [root@vm-146 ~]# ll /usr/sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /usr/sbin/reboot - ../bin/systemctl [root@vm-146 ~]# cat /etc/redhat-release Fedora release 18 (Rawhide) ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel Things are different on Fedora 16: vm-084:~# ls -l /sbin/reboot lrwxrwxrwx. 1 root root 16 Mar 2 04:43 /sbin/reboot - ../bin/systemctl vm-084:~# ls -l /usr/bin/reboot ls: cannot access /usr/bin/reboot: No such file or directory vm-084:~# cat /etc/redhat-release Fedora release 16 (Verne) vm-084:~# vm-084:~# yum whatprovides /usr/bin/reboot Loaded plugins: product-id, subscription-manager Updating certificate-based repositories. usermode-1.108-1.fc16.x86_64 : Tools for certain user account management tasks Repo: Fedora-16-x86_64-Everything Matched from: Filename: /usr/bin/reboot usermode-1.108-1.fc16.x86_64 : Tools for certain user account management tasks Repo: fedora Matched from: Filename: /usr/bin/reboot Also, I expect other distros will follow FHS rather than UsrMove. -- Petr³ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0019 Use reboot from /sbin
On Fri, Mar 02, 2012 at 02:08:38PM +0100, Petr Viktorin wrote: On 03/02/2012 01:42 PM, Jakub Hrozek wrote: On Fri, Mar 02, 2012 at 01:28:56PM +0100, Petr Viktorin wrote: Commit message says it all. So does the ticket. https://fedorahosted.org/freeipa/ticket/2480 -- Petr³ Does it matter? It the UsrMoved world, both are just symlinks to systemctl.. [root@vm-146 ~]# ll /sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /sbin/reboot - ../bin/systemctl [root@vm-146 ~]# ll /usr/sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /usr/sbin/reboot - ../bin/systemctl [root@vm-146 ~]# cat /etc/redhat-release Fedora release 18 (Rawhide) ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel Things are different on Fedora 16: vm-084:~# ls -l /sbin/reboot lrwxrwxrwx. 1 root root 16 Mar 2 04:43 /sbin/reboot - ../bin/systemctl vm-084:~# ls -l /usr/bin/reboot ls: cannot access /usr/bin/reboot: No such file or directory vm-084:~# cat /etc/redhat-release Fedora release 16 (Verne) vm-084:~# vm-084:~# yum whatprovides /usr/bin/reboot Loaded plugins: product-id, subscription-manager Updating certificate-based repositories. usermode-1.108-1.fc16.x86_64 : Tools for certain user account management tasks Repo: Fedora-16-x86_64-Everything Matched from: Filename: /usr/bin/reboot usermode-1.108-1.fc16.x86_64 : Tools for certain user account management tasks Repo: fedora Matched from: Filename: /usr/bin/reboot I see. Ack Also, I expect other distros will follow FHS rather than UsrMove. ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 0019 Use reboot from /sbin
On Fri, 2012-03-02 at 14:59 +0100, Jakub Hrozek wrote: On Fri, Mar 02, 2012 at 02:08:38PM +0100, Petr Viktorin wrote: On 03/02/2012 01:42 PM, Jakub Hrozek wrote: On Fri, Mar 02, 2012 at 01:28:56PM +0100, Petr Viktorin wrote: Commit message says it all. So does the ticket. https://fedorahosted.org/freeipa/ticket/2480 -- Petr³ Does it matter? It the UsrMoved world, both are just symlinks to systemctl.. [root@vm-146 ~]# ll /sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /sbin/reboot - ../bin/systemctl [root@vm-146 ~]# ll /usr/sbin/reboot lrwxrwxrwx. 1 root root 16 Feb 29 16:25 /usr/sbin/reboot - ../bin/systemctl [root@vm-146 ~]# cat /etc/redhat-release Fedora release 18 (Rawhide) ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel Things are different on Fedora 16: vm-084:~# ls -l /sbin/reboot lrwxrwxrwx. 1 root root 16 Mar 2 04:43 /sbin/reboot - ../bin/systemctl vm-084:~# ls -l /usr/bin/reboot ls: cannot access /usr/bin/reboot: No such file or directory vm-084:~# cat /etc/redhat-release Fedora release 16 (Verne) vm-084:~# vm-084:~# yum whatprovides /usr/bin/reboot Loaded plugins: product-id, subscription-manager Updating certificate-based repositories. usermode-1.108-1.fc16.x86_64 : Tools for certain user account management tasks Repo: Fedora-16-x86_64-Everything Matched from: Filename: /usr/bin/reboot usermode-1.108-1.fc16.x86_64 : Tools for certain user account management tasks Repo: fedora Matched from: Filename: /usr/bin/reboot I see. Ack Also, I expect other distros will follow FHS rather than UsrMove. Pushed to master, ipa-2-2. Martin ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel