Merge pull request #11

Add Linux AppImage build to workflow, update packages and replace file_selector and lints with their successors, Fix Lint warnings

.github/workflows/deploy.yml

@@ -18,48 +18,73 @@ jobs:
     runs-on: windows-2019
 
     steps:
-
-    - name: Checkout
-      uses: actions/checkout@v2
-
-    - name: Setup java
-      uses: actions/setup-java@v1
-      with:
-        java-version: '12.x'
-
-    - name: Setup flutter
-      uses: subosito/flutter-action@v1
-
-    - name: Enable desktop support	
-      run: flutter config --enable-windows-desktop
+      - name: Checkout
+        uses: actions/checkout@v2
 
-    - name: Doctor
-      run: flutter doctor -v
+      - name: Setup java
+        uses: actions/setup-java@v1
+        with:
+          java-version: '12.x'
+
+      - name: Setup flutter
+        uses: subosito/flutter-action@master
+
+      - name: Enable desktop support	
+        run: flutter config --enable-windows-desktop
+
+      - name: Doctor
+        run: flutter doctor -v
+
+      - name: Get dependencies
+        run: flutter pub get
 
-    - name: Get dependencies
-      run: flutter pub get
-
-    - name: Build windows
-      run: flutter build windows
+      - name: Build windows
+        run: flutter build windows
+
+      - name: Build apk
+        run: flutter build apk
+
+      - name: Upload apk artifacts
+        uses: actions/upload-artifact@v1
+        with:
+          name: Youtube Downloader.apk
+          path: ./build/app/outputs/flutter-apk/app-release.apk
+
+      - name: Compress Windows App
+        run: .\tool\compress.ps1
+        shell: powershell
+
+
+      - name: Upload windows zip artifacts
+        uses: actions/upload-artifact@v1
+        with:
+          name: Youtube Downloader - Windows.zip
+          path: Youtube Downloader.zip
+
+  build-appimage:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Setup flutter
+        uses: subosito/flutter-action@master
 
-    - name: Build apk
-      run: flutter build apk
+      - name: Install dependencies
+        run: sudo apt-get update -y && sudo apt-get install -y ninja-build libgtk-3-dev
 
-    - name: Upload apk artifacts
-      uses: actions/upload-artifact@v1
-      with:
-        name: Youtube Downloader.apk
-        path: ./build/app/outputs/flutter-apk/app-release.apk
+      - name: Enable desktop support	
+        run: flutter config --enable-linux-desktop
 
-    - name: Compress Windows App
-      run: .\tool\compress.ps1
-      shell: powershell
+      - name: Build Flutter app
+        run: flutter build linux
 
+      - name: Build AppImage
+        run: ./create_appimage.sh
 
-    - name: Upload windows zip artifacts
-      uses: actions/upload-artifact@v1
-      with:
-        name: Youtube Downloader - Windows.zip
-        path: Youtube Downloader.zip
+      - name: Upload linux appimage to artifacts
+        uses: actions/upload-artifact@v1
+        with:
+          name: Youtube Downloader - Linux.AppImage
+          path: youtube_downloader_flutter-x86_64.AppImage
 
 

.gitignore

@@ -8,6 +8,8 @@
 .buildlog/
 .history
 .svn/
+AppDir/
+*.AppImage
 
 # IntelliJ related
 *.iml

analysis_options.yaml

@@ -1 +1 @@
-include: package:lint/analysis_options.yaml
+include: package:flutter_lints/flutter.yaml

create_appimage.sh

@@ -0,0 +1,42 @@
+#!/bin/sh
+
+# Create Application Directory
+mkdir -p AppDir
+
+# Create AppRun file(required by AppImage)
+echo '#!/bin/sh
+
+cd "$(dirname "$0")"
+exec ./youtube_downloader_flutter' > AppDir/AppRun
+sudo chmod +x AppDir/AppRun
+
+# Copy All build files to AppDir
+cp -r build/linux/x64/release/bundle/* AppDir
+
+## Add Application metadata
+# Copy app icon
+sudo mkdir -p AppDir/usr/share/icons/hicolor/256x256/apps/
+cp android/app/src/main/res/mipmap-xxhdpi/ic_launcher.png AppDir/youtube_downloader_flutter.png
+sudo cp AppDir/youtube_downloader_flutter.png AppDir/usr/share/icons/hicolor/256x256/apps/youtube_downloader_flutter.png
+
+sudo mkdir -p AppDir/usr/share/applications
+
+# Either copy .desktop file content from file or with echo command
+# cp assets/youtube_downloader_flutter.desktop AppDir/youtube_downloader_flutter.desktop
+
+echo '[Desktop Entry]
+Version=1.0
+Type=Application
+Name=Youtube downloader flutter
+Icon=youtube_downloader_flutter
+Exec=youtube_downloader_flutter %u
+StartupWMClass=youtube_downloader_flutter
+Categories=Utility;' > AppDir/youtube_downloader_flutter.desktop
+
+# Also copy the same .desktop file to usr folder
+sudo cp AppDir/youtube_downloader_flutter.desktop AppDir/usr/share/applications/youtube_downloader_flutter.desktop
+
+## Start build
+test ! -e appimagetool-x86_64.AppImage && curl -L https://github.com/AppImage/AppImageKit/releases/download/continuous/appimagetool-x86_64.AppImage -o appimagetool-x86_64.AppImage
+sudo chmod +x appimagetool-x86_64.AppImage
+ARCH=x86_64 ./appimagetool-x86_64.AppImage AppDir/ youtube_downloader_flutter-x86_64.AppImage

lib/main.dart

@@ -23,15 +23,17 @@ void main() {
       SetProcessDpiAwareness(1);
     }
   }
-  runApp(MyApp());
+  runApp(const MyApp());
 }
 
 class MyApp extends StatelessWidget {
+  const MyApp({Key? key}) : super(key: key);
+
   @override
   Widget build(BuildContext context) {
     return ProviderScope(
       observers: [MainObserver()],
-      child: AppInit(),
+      child: const AppInit(),
     );
   }
 }
@@ -40,24 +42,26 @@ class MainObserver implements ProviderObserver {
   @override
   void didAddProvider(
       ProviderBase provider, Object? value, ProviderContainer container) {
-    print('Added: $provider : $value(${value.runtimeType})');
+    debugPrint('Added: $provider : $value(${value.runtimeType})');
   }
 
   @override
   void didDisposeProvider(ProviderBase provider, ProviderContainer container) {
-    print('Disposed: $provider');
+    debugPrint('Disposed: $provider');
   }
 
   @override
   void didUpdateProvider(ProviderBase provider, Object? previousValue,
       Object? newValue, ProviderContainer container) {
-    print('Update: $provider : $newValue');
+    debugPrint('Update: $provider : $newValue');
   }
 }
 
 class AppInit extends HookConsumerWidget {
   static final scaffoldKey = GlobalKey<ScaffoldMessengerState>();
 
+  const AppInit({Key? key}) : super(key: key);
+
   @override
   Widget build(BuildContext context, WidgetRef ref) {
     final fetched = useState<bool>(false);

lib/src/models/download_manager.dart

@@ -305,7 +305,7 @@ class DownloadManagerImpl extends ChangeNotifier implements DownloadManager {
 
     process.stdout.listen((event) {
       final data = utf8.decode(event);
-      print('OUT: $data');
+      debugPrint('OUT: $data');
 
       final timeStr = RegExp(r'out_time_ms=(\d+)').firstMatch(data)?.group(1);
       if (timeStr == null) {
@@ -578,7 +578,7 @@ class SingleTrack extends ChangeNotifier {
 
   void cancelDownload() {
     if (_cancelCallback == null) {
-      print('Tried to cancel an uncancellable video');
+      debugPrint('Tried to cancel an uncancellable video');
       return;
     }
     _cancelCallback!();

lib/src/search_bar.dart

@@ -6,7 +6,7 @@ import 'package:youtube_downloader_flutter/src/widgets/search_view/search_result
 import 'widgets/search_view/suggestion_list.dart';
 
 class SearchBar extends HookWidget {
-  const SearchBar();
+  const SearchBar({Key? key}) : super(key: key);
 
   @override
   Widget build(BuildContext context) {

lib/src/services/search_service.dart

@@ -19,10 +19,10 @@ abstract class SearchService extends ChangeNotifier {
   factory SearchService(YoutubeExplode yt, String query) {
     final cid = ChannelId.parseChannelId(query);
     if (cid != null) {
-      print('Channel!');
+      debugPrint('Channel!');
       return _ChannelSearchServiceImpl(yt, cid);
     }
-    print('Video!');
+    debugPrint('Video!');
     return _VideoSearchServiceImpl(yt, query);
   }
 }