Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Sign in
Toggle navigation
F
Front-End
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
abdullh.alsoleman
Front-End
Commits
c93847d1
Unverified
Commit
c93847d1
authored
5 years ago
by
Jenn Magder
Committed by
GitHub
5 years ago
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove IOSDeploy from globals, hoist to XCDevice constructor (#53203)
parent
600a3a74
Changes
4
Show whitespace changes
Inline
Side-by-side
Showing
4 changed files
with
15 additions
and
12 deletions
+15
-12
context_runner.dart
packages/flutter_tools/lib/src/context_runner.dart
+7
-7
globals.dart
packages/flutter_tools/lib/src/globals.dart
+0
-2
xcode.dart
packages/flutter_tools/lib/src/macos/xcode.dart
+7
-3
xcode_test.dart
...es/flutter_tools/test/general.shard/macos/xcode_test.dart
+1
-0
No files found.
packages/flutter_tools/lib/src/context_runner.dart
View file @
c93847d1
...
...
@@ -129,13 +129,6 @@ Future<T> runInContext<T>(
FuchsiaWorkflow:
()
=>
FuchsiaWorkflow
(),
GradleUtils:
()
=>
GradleUtils
(),
HotRunnerConfig:
()
=>
HotRunnerConfig
(),
IOSDeploy:
()
=>
IOSDeploy
(
artifacts:
globals
.
artifacts
,
cache:
globals
.
cache
,
logger:
globals
.
logger
,
platform:
globals
.
platform
,
processManager:
globals
.
processManager
,
),
IOSSimulatorUtils:
()
=>
IOSSimulatorUtils
(
simControl:
globals
.
simControl
,
xcode:
globals
.
xcode
,
...
...
@@ -217,6 +210,13 @@ Future<T> runInContext<T>(
logger:
globals
.
logger
,
processManager:
globals
.
processManager
,
),
iosDeploy:
IOSDeploy
(
artifacts:
globals
.
artifacts
,
cache:
globals
.
cache
,
logger:
globals
.
logger
,
platform:
globals
.
platform
,
processManager:
globals
.
processManager
,
),
xcode:
globals
.
xcode
,
),
XcodeProjectInterpreter:
()
=>
XcodeProjectInterpreter
(
...
...
This diff is collapsed.
Click to expand it.
packages/flutter_tools/lib/src/globals.dart
View file @
c93847d1
...
...
@@ -23,7 +23,6 @@ import 'base/user_messages.dart';
import
'build_system/build_system.dart'
;
import
'cache.dart'
;
import
'fuchsia/fuchsia_sdk.dart'
;
import
'ios/ios_deploy.dart'
;
import
'ios/ios_workflow.dart'
;
import
'ios/plist_parser.dart'
;
import
'ios/simulators.dart'
;
...
...
@@ -75,7 +74,6 @@ AndroidSdk get androidSdk => context.get<AndroidSdk>();
CocoaPods
get
cocoaPods
=>
context
.
get
<
CocoaPods
>();
FlutterVersion
get
flutterVersion
=>
context
.
get
<
FlutterVersion
>();
FuchsiaArtifacts
get
fuchsiaArtifacts
=>
context
.
get
<
FuchsiaArtifacts
>();
IOSDeploy
get
iosDeploy
=>
context
.
get
<
IOSDeploy
>();
IOSSimulatorUtils
get
iosSimulatorUtils
=>
context
.
get
<
IOSSimulatorUtils
>();
IOSWorkflow
get
iosWorkflow
=>
context
.
get
<
IOSWorkflow
>();
SimControl
get
simControl
=>
context
.
get
<
SimControl
>();
...
...
This diff is collapsed.
Click to expand it.
packages/flutter_tools/lib/src/macos/xcode.dart
View file @
c93847d1
...
...
@@ -17,6 +17,7 @@ import '../build_info.dart';
import
'../convert.dart'
;
import
'../globals.dart'
as
globals
;
import
'../ios/devices.dart'
;
import
'../ios/ios_deploy.dart'
;
import
'../ios/mac.dart'
;
import
'../ios/xcodeproj.dart'
;
import
'../reporting/reporting.dart'
;
...
...
@@ -198,14 +199,17 @@ class XCDevice {
@required
Logger
logger
,
@required
Xcode
xcode
,
@required
IMobileDevice
iMobileDevice
,
@required
IOSDeploy
iosDeploy
,
})
:
_processUtils
=
ProcessUtils
(
logger:
logger
,
processManager:
processManager
),
_logger
=
logger
,
_iMobileDevice
=
iMobileDevice
,
_iosDeploy
=
iosDeploy
,
_xcode
=
xcode
;
final
ProcessUtils
_processUtils
;
final
Logger
_logger
;
final
IMobileDevice
_iMobileDevice
;
final
IOSDeploy
_iosDeploy
;
final
Xcode
_xcode
;
bool
get
isInstalled
=>
_xcode
.
isInstalledAndMeetsVersionCheck
&&
xcdevicePath
!=
null
;
...
...
@@ -359,7 +363,7 @@ class XCDevice {
artifacts:
globals
.
artifacts
,
fileSystem:
globals
.
fs
,
logger:
globals
.
logger
,
iosDeploy:
globals
.
iosDeploy
,
iosDeploy:
_
iosDeploy
,
iMobileDevice:
_iMobileDevice
,
platform:
globals
.
platform
,
));
...
...
This diff is collapsed.
Click to expand it.
packages/flutter_tools/test/general.shard/macos/xcode_test.dart
View file @
c93847d1
...
...
@@ -217,6 +217,7 @@ void main() {
logger:
logger
,
xcode:
mockXcode
,
iMobileDevice:
null
,
iosDeploy:
null
,
);
});
...
...
This diff is collapsed.
Click to expand it.
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment