Merge pull request #204 from aunetx/josselinonduty/issue202

[ENH] Stop build (and ci) when a failing hunk is detected

Author: josselinonduty

Makefile

@@ -40,7 +40,12 @@ prepare: clean install_build_deps
 	@npm run prettier -- --write "$(APP_DIR)/build/*.{js,html}" --config .prettierrc.json --ignore-path /dev/null
 
 	@echo "--------------------------------"
-	@$(foreach p,$(wildcard ./patches/*), echo "Applying $(p)"; patch -p 1 -d $(APP_DIR) < $(p) && echo "Applied $(p)\n";)
+	@set -e; \
+	for p in ./patches/*; do \
+		echo "Applying $$p"; \
+		patch -p 1 -d $(APP_DIR) < $$p; \
+		echo "Applied $$p\n"; \
+	done
 
 	@echo "Append package-append.json to the package.json of the app"
 	@echo "Adds electron, elecron-builder dependencies, prod and build directives"

README.md

@@ -35,11 +35,10 @@ Other packages can be installed from your package manager, either by clicking on
 | `--start-in-tray`                                                                    | Start the app in the tray (see [patch](./patches/01-start-in-tray.patch))                                                                   |
 | `--disable-systray`                                                                  | Quit the app when the window is closed (see [patch](./patches/02-start-without-tray.patch))                                                 |
 | `--keep-kernel`                                                                      | Use the exact kernel version (see [patch](./patches/04-remove-os-information.patch)) <br/> _This feature impacts privacy._                  |
-| `--disable-features`                                                                 | Disable some features (see [patch](./patches/05-provide-metadata-mpris.patch))                                                              |
 | `--hide-offline-banner`                                                              | Hide the "Application is offline" banner that appears when using a VPN or DNS blocker (see [patch](./patches/08-hide-offline-banner.patch)) |
 | `--disable-animations`                                                               | Disable animations (see [patch](./patches/09-disable-animations.patch))                                                                     |
 | `--disable-notifications`                                                            | Disable notifications (see [patch](./patches/10-disable-notifications.patch))                                                               |
-| `--disable-hardware-acceleration`                                                    | Disable hardware acceleration (useful for systems with GPU issues) (see [patch](./patches/13-disable-hardware-acceleration.patch))                                                                          |
+| `--disable-hardware-acceleration`                                                    | Disable hardware acceleration (useful for systems with GPU issues) (see [patch](./patches/13-disable-hardware-acceleration.patch))          |
 | `--log-level`                                                                        | Set the log level (`silly`,`debug`,`verbose`,`info`,`warn`,`error`) (see [patch](./patches/06-control-log-level.patch))                     |
 | `--enable-wayland-ime` `--ozone-platform-hint=auto` `--wayland-text-input-version=3` | Enable IME keyboard support on Wayland                                                                                                      |
 
@@ -52,7 +51,7 @@ Other packages can be installed from your package manager, either by clicking on
 | `DZ_HIDE_OFFLINE_BANNER`           | `yes`,`no`                                      | Hide the "Application is offline" banner (see [patch](./patches/08-hide-offline-banner.patch)) |
 | `DZ_DISABLE_ANIMATIONS`            | `yes`,`no`                                      | Disable animations (see [patch](./patches/09-disable-animations.patch))                        |
 | `DZ_DISABLE_NOTIFICATIONS`         | `yes`,`no`                                      | Disable notifications (see [patch](./patches/10-disable-notifications.patch))                  |
-| `DZ_DISABLE_HARDWARE_ACCELERATION` | `yes`,`no`                                      | Disable hardware acceleration (see [patch](./patches/13-disable-hardware-acceleration.patch))                 |
+| `DZ_DISABLE_HARDWARE_ACCELERATION` | `yes`,`no`                                      | Disable hardware acceleration (see [patch](./patches/13-disable-hardware-acceleration.patch))  |
 | `DZ_DEVTOOLS`                      | `yes`,`no`                                      | Enable the developer console (ctrl+shift+i)                                                    |
 
 ## Building from source