FS#25271 - [netcfg] latest initscripts update breaks non-root actions
            Attached to Project:
            Arch Linux
            
Opened by xduugu (xduugu) - Tuesday, 26 July 2011, 13:51 GMT
Last edited by Tom Gundersen (tomegun) - Thursday, 28 July 2011, 21:40 GMT
          Opened by xduugu (xduugu) - Tuesday, 26 July 2011, 13:51 GMT
Last edited by Tom Gundersen (tomegun) - Thursday, 28 July 2011, 21:40 GMT
| 
 | Details
                    Description: The latest initscripts comes with a commit[1] that makes /etc/rc.d/functions requiring root by default when sourcing/executing it. $ netcfg --help You need to be root. [1] https://projects.archlinux.org/initscripts.git/commit/functions?id=042d197b4d989ec6461204b9d897054e5f5dd492 Solution: --- a/usr/lib/network/hooks/arch +++ b/usr/lib/network/hooks/arch @@ -1,3 +1,4 @@ +NEED_ROOT=0 . /etc/rc.conf . /etc/rc.d/functions Additional info: * netcfg 2.6.5-1 * initscripts 2011.07.1-1 | 
              This task depends upon
              
              
            
            
          
            Closed by  Tom Gundersen (tomegun)
Thursday, 28 July 2011, 21:40 GMT
Reason for closing: Fixed
Additional comments about closing: Fixed (at least for now) in testing.
          
        Thursday, 28 July 2011, 21:40 GMT
Reason for closing: Fixed
Additional comments about closing: Fixed (at least for now) in testing.
 Comment by Dan McGee (toofishes) -
            Wednesday, 27 July 2011, 15:11 GMT
            
            Comment by Dan McGee (toofishes) -
            Wednesday, 27 July 2011, 15:11 GMT
          
           
          - Field changed: Status (Unconfirmed → Assigned)
- Task assigned to Dave Reisner (falconindy), Rémy Oudompheng (remyoudompheng), Tom Gundersen (tomegun)
              This change to initscripts seems like solving a problem in the
              wrong place to me, so adding the initscripts guys too (who also
              need to fix this as you broke things).
            
           Comment by
            Dave Reisner (falconindy) -
            Wednesday, 27 July 2011, 15:22 GMT
            
            Comment by
            Dave Reisner (falconindy) -
            Wednesday, 27 July 2011, 15:22 GMT
          
           
          
              Your patch works, but no, I don't think this is a good solution.
              I'm looking back through history to see why we made this
              ridiculous change, and I'm sure I won't be happy.
            
           Comment by Tom Gundersen (tomegun) -
            Wednesday, 27 July 2011, 15:41 GMT
            
            Comment by Tom Gundersen (tomegun) -
            Wednesday, 27 July 2011, 15:41 GMT
          
           
          
              This was not intentional (at least not from my side). I'll get it
              sorted out.
            
          