v3.1 - (mostly) fix diff functionality

earthlng · web-flow · commit ada31d4f504d · 2021-06-06T18:01:56.000Z
see #1188 this should fix the issue that "All prefs after a multi-line comment declaration, on a single line, are deleted with the remove_comments function from the updater."
diff --git a/updater.sh b/updater.sh
@@ -2,7 +2,7 @@
 
 ## arkenfox user.js updater for macOS and Linux
 
-## version: 3.0
+## version: 3.1
 ## Author: Pat Johnson (@overdodactyl)
 ## Additional contributors: @earthlng, @ema-pe, @claustromaniac
 
@@ -236,7 +236,7 @@ add_override () {
 }
 
 remove_comments () { # expects 2 arguments: from-file and to-file
-  sed -e 's/^[[:space:]]*\/\/.*$//' -e '/^\/\*/,/\*\//d' -e '/^[[:space:]]*$/d' -e 's/);[[:space:]]*\/\/.*/);/' "$1" > "$2"
+  sed -e 's/^[[:space:]]*\/\/.*$//' -e '/^\/\*.\+\*\/[[:space:]]*$/d' -e '/^\/\*/,/\*\//d' -e '/^[[:space:]]*$/d' -e 's/);[[:space:]]*\/\/.*/);/' "$1" > "$2"
 }
 
 # Applies latest version of user.js and any custom overrides

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`
`3`	`3`	`## arkenfox user.js updater for macOS and Linux`
`4`	`4`
`5`		`-## version: 3.0`
	`5`	`+## version: 3.1`
`6`	`6`	`## Author: Pat Johnson (@overdodactyl)`
`7`	`7`	`## Additional contributors: @earthlng, @ema-pe, @claustromaniac`
`8`	`8`
`@@ -236,7 +236,7 @@ add_override () {`
`236`	`236`	`}`
`237`	`237`
`238`	`238`	`remove_comments () { # expects 2 arguments: from-file and to-file`
`239`		`- sed -e 's/^[[:space:]]\/\/.$//' -e '/^\/\/,/\\//d' -e '/^[[:space:]]$/d' -e 's/);[[:space:]]\/\/.*/);/' "$1" > "$2"`
	`239`	`+ sed -e 's/^[[:space:]]\/\/.$//' -e '/^\/\.\+\\/[[:space:]]$/d' -e '/^\/\/,/\\//d' -e '/^[[:space:]]$/d' -e 's/);[[:space:]]\/\/./);/' "$1" > "$2"`
`240`	`240`	`}`
`241`	`241`
`242`	`242`	`# Applies latest version of user.js and any custom overrides`