minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path"
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Tgr
	Jan 2 2019, 10:11 AM

Description

E.g. https://integration.wikimedia.org/ci/job/mediawiki-quibble-composertest-php70-docker/10986/console :

01:17:02 INFO:test.run_composer_test:Running composer test
...
01:17:12 > composer minus-x 'includes/DefaultSettings.php' 'includes/auth/AbstractPasswordPrimaryAuthenticationProvider.php' 'includes/password/UserPasswordPolicy.php' 'includes/user/User.php' 'tests/phpunit/includes/auth/AbstractPasswordPrimaryAuthenticationProviderTest.php' 'tests/phpunit/includes/auth/LocalPasswordPrimaryAuthenticationProviderTest.php' 'tests/phpunit/includes/password/UserPasswordPolicyTest.php'
01:17:12 > minus-x check 'includes/DefaultSettings.php' 'includes/auth/AbstractPasswordPrimaryAuthenticationProvider.php' 'includes/password/UserPasswordPolicy.php' 'includes/user/User.php' 'tests/phpunit/includes/auth/AbstractPasswordPrimaryAuthenticationProviderTest.php' 'tests/phpunit/includes/auth/LocalPasswordPrimaryAuthenticationProviderTest.php' 'tests/phpunit/includes/password/UserPasswordPolicyTest.php'
01:17:12 
01:17:12                                                             
01:17:12   Too many arguments, expected arguments "command" "path".  
01:17:12                                                             
01:17:12 
01:17:12 check <path>
01:17:12 
01:17:12 Script minus-x check handling the minus-x event returned with error code 1
01:17:12 Script composer minus-x handling the test event returned with error code 1

Apparently some CI code calls composer test specifically on the changed files, and minus-x gets confused from the extra arguments.

Details

	Subject	Repo	Branch	Lines +/-
	Revert "build: Run minus-x to check for executable files that shouldn't be"	mediawiki/core	master	+2 -15

Customize query in gerrit

Related Objects

Mentioned In: T212746: Run minus-x on mediawiki/core for all files
Mentioned Here: T188022: Add CI entry point to run MinusX on mediawiki/core
T212746: Run minus-x on mediawiki/core for all files
T175794: Add MinusX to MediaWiki extensions and PHP library repos

Event Timeline

Tgr created this task.Jan 2 2019, 10:11 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 2 2019, 10:11 AM

Possibly caused by https://gerrit.wikimedia.org/r/c/mediawiki/core/+/402166 / T175794: Add MinusX to MediaWiki extensions and PHP library repos.

Change 481834 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] Revert "build: Run minus-x to check for executable files that shouldn't be"

https://gerrit.wikimedia.org/r/481834

gerritbot added a project: Patch-For-Review.Jan 2 2019, 10:19 AM

Change 481834 merged by jenkins-bot:
[mediawiki/core@master] Revert "build: Run minus-x to check for executable files that shouldn't be"

https://gerrit.wikimedia.org/r/481834

ReleaseTaggerBot added a project: MW-1.33-notes (1.33.0-wmf.12; 2019-01-08).Jan 2 2019, 11:00 AM

Umherirrender mentioned this in T212746: Run minus-x on mediawiki/core for all files.Jan 2 2019, 11:52 AM

It seems MinusX can only work on one directory and not of a list of one or more files as given by CI

> composer minus-x 'includes/logging/LogPager.php'
> minus-x check 'includes/logging/LogPager.php'
MinusX
======
Processing /workspace/src/includes/logging/LogPager.php...

In CheckCommand.php line 236:
                                                                               
  RecursiveDirectoryIterator::__construct(/workspace/src/includes/logging/Log  
  Pager.php): failed to open dir: Not a directory                              
                                                                               

check <path>

So the question is, if MinusX needs support for a list of files over the arguments or does mediawiki/core needs a new CI job to always run over the whole repo, see T212746 for the other issue. Than MinusX does not need adjustment.

Jdforrester-WMF removed a project: Patch-For-Review.Jan 2 2019, 2:45 PM

Jdforrester-WMF subscribed.

Nikerabbit subscribed.Jan 3 2019, 8:55 AM

That got fixed by reverting the addition of minus-x (which was https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/402166/ ).

I have described arguments being passed to composer test on T188022#4847735 which I reopened.

minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path"Closed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path"
Closed, ResolvedPublic
Actions