Put back some code that was removed for debug purposes.

symfony · ghost · Dec 4, 2017 · Dec 4, 2017 · Dec 4, 2017 · Dec 4, 2017
commit 67cc25360ec5bb7d77fe820d39bffaba9347c557
diff --git a/src/Symfony/Bridge/PhpUnit/bin/simple-phpunit b/src/Symfony/Bridge/PhpUnit/bin/simple-phpunit
@@ -231,7 +231,8 @@ if ($components) {
        class SymfonyBlacklistSimplePhpunit {}
    }
    array_splice($_SERVER['argv'], 1, 0, array('--colors=always'));
-
+    $_SERVER['argv'] = $argv;
+    $_SERVER['argc'] = ++$argc;
    include "$PHPUNIT_DIR/phpunit-$PHPUNIT_VERSION/phpunit";
 }
-Original file line number
+Diff line change
@@ -231,7 +231,8 @@ if ($components) {
             class SymfonyBlacklistSimplePhpunit {}
         }
         array_splice($_SERVER['argv'], 1, 0, array('--colors=always'));
             Copy link

  
      
    
  

  
      

  
  Member


      

  

  
    
      

      
            nicolas-grekas
  

      

      

      


        Dec 4, 2017


      
    

  


        
      
  
  
  
    

    There was a problem hiding this comment.


  

 
  
    

    Choose a reason for hiding this comment

    
      The reason will be displayed to describe this comment to others. Learn more.
    

    
      
      


  


  
    
      ok, in fact, this change introduces a bug

both $_SERVER['argv'] and $argv must be updated

this breaks the second
    
  
  


    

        
      
  
  
    
  
  
  
    
    Sorry, something went wrong.
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.


  
  


          
      
  
    
    
      
        
            
    All reactions
+        $_SERVER['argv'] = $argv;
+        $_SERVER['argc'] = ++$argc;
         include "$PHPUNIT_DIR/phpunit-$PHPUNIT_VERSION/phpunit";
     }
-          Expand Down