Skip to content

Commit 0ca811a

Browse files
committed
Use the MW_QUIBBLE_CI constant over $wgWikimediaJenkinsCI
Also don't bother setting this in the Jenkins' specific configuration ourselves, this is Quibble's/ the CI's responsibility. This should be merged together with the EntitySchema change: I28bd84d4a2d98ef581e55341e105df2c85a1bf10 Bug: T366243 Change-Id: Ie8b39c1a43118d4c5edf6f189d776a7a5c93da6e
1 parent f6d4a5d commit 0ca811a

File tree

3 files changed

+4
-7
lines changed

3 files changed

+4
-7
lines changed

Wikibase.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,15 @@
1212
if ( $wgEnableWikibaseRepo ?? true ) {
1313
wfLoadExtension( 'WikibaseRepository', __DIR__ . '/extension-repo.json' );
1414

15-
if ( $wgWikimediaJenkinsCI ?? false ) {
15+
if ( defined( 'MW_QUIBBLE_CI' ) ) {
1616
require_once __DIR__ . '/repo/config/Wikibase.ci.php';
1717
}
1818
}
1919

2020
if ( $wgEnableWikibaseClient ?? true ) {
2121
wfLoadExtension( 'WikibaseClient', __DIR__ . '/extension-client.json' );
2222

23-
if ( $wgWikimediaJenkinsCI ?? false ) {
23+
if ( defined( 'MW_QUIBBLE_CI' ) ) {
2424
require_once __DIR__ . '/client/config/WikibaseClient.ci.php';
2525
}
2626
}

build/ci-scripts/mw-apply-wb-settings.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@ function apply_repo_settings {
1919
function apply_common_before_settings {
2020
echo 'error_reporting(E_ALL| E_STRICT);' >> LocalSettings.php
2121
echo 'ini_set("display_errors", 1);' >> LocalSettings.php
22-
echo '$wgWikimediaJenkinsCI = true;' >> LocalSettings.php
22+
# For re-using the Wikimedia CI settings, pretend we're running in quibble
23+
echo 'if ( !defined( "MW_QUIBBLE_CI" ) ) define( "MW_QUIBBLE_CI", true );' >> LocalSettings.php
2324
echo '$wgShowExceptionDetails = true;' >> LocalSettings.php
2425
echo '$wgDevelopmentWarnings = true;' >> LocalSettings.php
2526
echo '$wgLanguageCode = "'$LANG'";' >> LocalSettings.php

build/jenkins/mw-apply-wb-settings.sh

-4
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,6 @@ function apply_client_settings {
3434
// mw-apply-wb-settings.php:
3535
$wgEnableWikibaseRepo = false;
3636
$wgEnableWikibaseClient = true;
37-
// $wgWikimediaJenkinsCI is usually set by Jenkins/Quibble
38-
$wgWikimediaJenkinsCI = true;
3937
PHP
4038
echo -n '?>' >> LocalSettings.php
4139
cat LocalSettings-$$.php >> LocalSettings.php
@@ -51,8 +49,6 @@ function apply_repo_settings {
5149
// mw-apply-wb-settings.php:
5250
$wgEnableWikibaseRepo = true;
5351
$wgEnableWikibaseClient = true;
54-
// $wgWikimediaJenkinsCI is usually set by Jenkins/Quibble
55-
$wgWikimediaJenkinsCI = true;
5652
PHP
5753
echo -n '?>' >> LocalSettings.php
5854
cat LocalSettings-$$.php >> LocalSettings.php

0 commit comments

Comments
 (0)