Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Directories with only .gitkeep file within a plugin are not downloaded to pub cache #148002

Closed
vashworth opened this issue May 8, 2024 · 7 comments
Labels
a: build Building flutter applications with the tool found in release: 3.22 Found to occur in 3.22 has reproducible steps The issue has been confirmed reproducible and is ready to work on package flutter/packages repository. See also p: labels. platform-ios iOS applications specifically r: solved Issue is closed as solved team-ios Owned by iOS platform team tool Affects the "flutter" command-line tool. See also t: labels.

Comments

@vashworth
Copy link
Contributor

vashworth commented May 8, 2024

Steps to reproduce

  1. Create a flutter app
  2. Add a dependency for video_player_avfoundation 2.6.0
  3. flutter config --enable-swift-package-manager
  4. flutter build ios

Actual results

Will error with

xcodebuild: error: Could not resolve package dependencies:
  public headers ("include") directory path for 'video_player_avfoundation_ios' is invalid or not contained in the target

Logs

For example, this path should contain an include/.gitkeep directory

ls /path/to/.pub-cache/hosted/pub.dev/video_player_avfoundation-2.6.0/darwin/video_player_avfoundation/Sources/video_player_avfoundation_ios

https://github.com/flutter/packages/tree/a9f14bc819e635f43f1861b8da807ce03b7ce013/packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation_ios/include

Flutter Doctor output

Doctor output
<!-- Paste your output here -->
@stuartmorgan
Copy link
Contributor

Probably worth filing this at https://github.com/dart-lang/pub to explain that it's causing issues for plugins.

@vashworth
Copy link
Contributor Author

Probably worth filing this at https://github.com/dart-lang/pub to explain that it's causing issues for plugins.

Done: dart-lang/pub#4262

@darshankawar darshankawar added the in triage Presently being triaged by the triage team label May 9, 2024
@darshankawar
Copy link
Member

Was able to replicate this on master.

master flutter doctor -v

[!] Flutter (Channel master, 3.22.0-28.0.pre.17, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.22.0-28.0.pre.17 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 972dcd7c65 (2 hours ago), 2024-05-08 22:41:26 -0400
    • Engine revision 69f2d9610a
    • Dart version 3.5.0 (build 3.5.0-138.0.dev)
    • DevTools version 2.36.0-dev.5
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.
      
[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.



@darshankawar darshankawar added platform-ios iOS applications specifically tool Affects the "flutter" command-line tool. See also t: labels. a: build Building flutter applications with the tool has reproducible steps The issue has been confirmed reproducible and is ready to work on found in release: 3.22 Found to occur in 3.22 package flutter/packages repository. See also p: labels. team-ios Owned by iOS platform team and removed in triage Presently being triaged by the triage team labels May 9, 2024
@vashworth
Copy link
Contributor Author

As a temporary solution, we'll be advising plugin authors to add !.gitkeep to their .gitignore, which flutter pub publish respects

@jmagman
Copy link
Member

jmagman commented May 16, 2024

@vashworth is this issue tracking adding that documentation? Or is there something else the tooling needs to do, other than what pub needs to do in dart-lang/pub#4262 ?

@vashworth
Copy link
Contributor Author

@vashworth is this issue tracking adding that documentation? Or is there something else the tooling needs to do, other than what pub needs to do in dart-lang/pub#4262 ?

No, we can track using dart-lang/pub#4262

@darshankawar darshankawar added the r: solved Issue is closed as solved label May 22, 2024
Copy link

github-actions bot commented Jun 5, 2024

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a: build Building flutter applications with the tool found in release: 3.22 Found to occur in 3.22 has reproducible steps The issue has been confirmed reproducible and is ready to work on package flutter/packages repository. See also p: labels. platform-ios iOS applications specifically r: solved Issue is closed as solved team-ios Owned by iOS platform team tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

No branches or pull requests

4 participants