move assistant and webapp build-depends into main build-depends

For some reason, cabal 3.4.1.0 builds w/o the assistant and webapp,
even when the flag is explicitly turned on. Moving the build-depends from
inside the if flag section to the main build-depends somehow fixes this.

Since the webapp build deps are thus always available, there is no reason
not to build the webapp when building the assistant. So, got rid of the
webapp build flag. Kept the assistant build flag for now, since building
without it does at least still speed up the build.

Sponsored-by: Brock Spratlen on Patreon

CHANGELOG

@@ -1,3 +1,10 @@
+git-annex (10.20220823) UNRELEASED; urgency=medium
+
+  * Include the assistant and webapp when building with cabal 3.4.1.0.
+  * Merged the webapp build flag into the assistant build flag.
+
+ -- Joey Hess <id@joeyh.name>  Mon, 29 Aug 2022 15:03:04 -0400
+
 git-annex (10.20220822) upstream; urgency=medium
 
   * v8 repositories now automatically upgrade to v9, which will in turn

doc/bugs/Webapp_won__180__t_start_on_MacOS.mdwn

@@ -311,3 +311,5 @@ I had tons of luck using git-annex. I am a non-programmer person and I could eas
 I can cut and export now from every available computer in my home. I have a central server on my LAN and git-annex also helps me to back up every episode.
 My next goal is to use it for my personal documents like scans of invoices, letter and so on. I think an archive repo will help me with this. So I can have all the stuff in one big folder.
 Thanks for inventing such a nice tool
+
+> [[fixed|done]] --[[Joey]]

doc/bugs/Webapp_won__180__t_start_on_MacOS/comment_5_2c7d2e81cbe943fe9154830fdf958181._comment

@@ -0,0 +1,16 @@
+[[!comment format=mdwn
+ username="joey"
+ subject="""comment 5"""
+ date="2022-08-29T19:21:22Z"
+ content="""
+I've committed a workaround to that cabal problem, so assuming that was
+the cause of your problem, it will be fixed by the next git-annex release.
+
+If something else caused it, the next git-annex release would
+probably fail to build on homebrew. Which the homebrew developers will
+follow up on.
+
+So, I don't think this bug needs to be kept open any longer. Although you
+will need to wait, or install git-annex by another means than homebrew, in
+the meantime..
+"""]]

doc/bugs/cabal_builds_without_assistant.mdwn

@@ -10,3 +10,5 @@ I tried moving the build-depends out of the `if flag(Assistant)` section
 to the main build-depends section. It was able to install all the build
 depends, so it's not some dependency issue making it decide to turn off
 that build flag. --[[Joey]]
+
+> [[fixed|done]] by moving the dependencies into the main block. --[[Joey]]

git-annex.cabal

@@ -256,10 +256,8 @@ Extra-Source-Files:
   Utility/libkqueue.h
 
 Flag Assistant
-  Description: Enable git-annex assistant and watch command
+  Description: Enable git-annex assistant, webapp, and watch command
-
+  Default: True
-Flag Webapp
-  Description: Enable git-annex webapp
 
 Flag Pairing
   Description: Enable pairing
@@ -376,7 +374,21 @@ Executable git-annex
    aws (>= 0.20),
    DAV (>= 1.0),
    network (>= 3.0.0.0),
-   network-bsd
+   network-bsd,
+   mountpoints,
+   yesod (>= 1.4.3), 
+   yesod-static (>= 1.5.1),
+   yesod-form (>= 1.4.8),
+   yesod-core (>= 1.6.0),
+   path-pieces (>= 0.2.1),
+   warp (>= 3.2.8),
+   warp-tls (>= 3.2.2),
+   wai,
+   wai-extra,
+   blaze-builder,
+   clientsession,
+   template-haskell,
+   shakespeare (>= 2.0.11)
   CC-Options: -Wall
   GHC-Options: -Wall -fno-warn-tabs  -Wincomplete-uni-patterns
   Default-Language: Haskell2010
@@ -416,8 +428,7 @@ Executable git-annex
     Other-Modules: Utility.GitLFS
   
   if flag(Assistant) && ! os(solaris) && ! os(gnu)
-    Build-Depends: mountpoints
+    CPP-Options: -DWITH_ASSISTANT -DWITH_WEBAPP
-    CPP-Options: -DWITH_ASSISTANT
     Other-Modules:
       Assistant
       Assistant.Alert
@@ -436,6 +447,7 @@ Executable git-annex
       Assistant.Install.AutoStart
       Assistant.Install.Menu
       Assistant.MakeRemote
+      Assistant.MakeRepo
       Assistant.Monad
       Assistant.NamedThread
       Assistant.Pairing
@@ -458,6 +470,7 @@ Executable git-annex
       Assistant.Threads.Merger
       Assistant.Threads.MountWatcher
       Assistant.Threads.NetWatcher
+      Assistant.Threads.PairListener
       Assistant.Threads.ProblemFixer
       Assistant.Threads.Pusher
       Assistant.Threads.RemoteControl
@@ -469,6 +482,7 @@ Executable git-annex
       Assistant.Threads.UpgradeWatcher
       Assistant.Threads.Upgrader
       Assistant.Threads.Watcher
+      Assistant.Threads.WebApp
       Assistant.TransferQueue
       Assistant.TransferSlots
       Assistant.Types.Alert
@@ -489,53 +503,6 @@ Executable git-annex
       Assistant.Types.UrlRenderer
       Assistant.Unused
       Assistant.Upgrade
-      Command.Assistant
-      Command.Watch
-      Utility.Mounts
-      Utility.OSX
-
-    if os(linux)
-      Build-Depends: hinotify (>= 0.3.10)
-      CPP-Options: -DWITH_INOTIFY
-      Other-Modules: Utility.DirWatcher.INotify
-    else
-      if os(darwin)
-        Build-Depends: hfsevents
-        CPP-Options: -DWITH_FSEVENTS
-        Other-Modules:
-          Utility.DirWatcher.FSEvents
-      else
-        if os(windows)
-          Build-Depends: Win32-notify
-          CPP-Options: -DWITH_WIN32NOTIFY
-          Other-Modules: Utility.DirWatcher.Win32Notify
-        else
-          if (! os(solaris) && ! os(gnu) && ! os(linux))
-            CPP-Options: -DWITH_KQUEUE
-            C-Sources: Utility/libkqueue.c
-            Includes: Utility/libkqueue.h
-            Other-Modules: Utility.DirWatcher.Kqueue
-  
-    if flag(Webapp)
-      Build-Depends:
-       yesod (>= 1.4.3), 
-       yesod-static (>= 1.5.1),
-       yesod-form (>= 1.4.8),
-       yesod-core (>= 1.6.0),
-       path-pieces (>= 0.2.1),
-       warp (>= 3.2.8),
-       warp-tls (>= 3.2.2),
-       wai,
-       wai-extra,
-       blaze-builder,
-       clientsession,
-       template-haskell,
-       shakespeare (>= 2.0.11)
-      CPP-Options: -DWITH_WEBAPP
-      Other-Modules:
-        Command.WebApp
-        Assistant.Threads.WebApp
-        Assistant.Threads.PairListener
       Assistant.WebApp
       Assistant.WebApp.Common
       Assistant.WebApp.Configurators
@@ -566,10 +533,36 @@ Executable git-annex
       Assistant.WebApp.RepoList
       Assistant.WebApp.SideBar
       Assistant.WebApp.Types
-        Assistant.MakeRepo
+      Command.Assistant
+      Command.Watch
+      Command.WebApp
+      Utility.Mounts
+      Utility.OSX
       Utility.Yesod
       Utility.WebApp
     
+    if os(linux)
+      Build-Depends: hinotify (>= 0.3.10)
+      CPP-Options: -DWITH_INOTIFY
+      Other-Modules: Utility.DirWatcher.INotify
+    else
+      if os(darwin)
+        Build-Depends: hfsevents
+        CPP-Options: -DWITH_FSEVENTS
+        Other-Modules:
+          Utility.DirWatcher.FSEvents
+      else
+        if os(windows)
+          Build-Depends: Win32-notify
+          CPP-Options: -DWITH_WIN32NOTIFY
+          Other-Modules: Utility.DirWatcher.Win32Notify
+        else
+          if (! os(solaris) && ! os(gnu) && ! os(linux))
+            CPP-Options: -DWITH_KQUEUE
+            C-Sources: Utility/libkqueue.c
+            Includes: Utility/libkqueue.h
+            Other-Modules: Utility.DirWatcher.Kqueue
+  
   if flag(Dbus)
     if (os(linux))
       Build-Depends: dbus (>= 0.10.7), fdo-notify (>= 0.3)

stack.yaml

@@ -4,7 +4,6 @@ flags:
     assistant: true
     pairing: true
     torrentparser: true
-    webapp: true
     magicmime: false
     dbus: false
     debuglocks: false

standalone/linux/stack-i386ancient.yaml

@@ -4,7 +4,6 @@ flags:
     assistant: true
     pairing: true
     torrentparser: true
-    webapp: true
     magicmime: false
     dbus: false
     debuglocks: false