search

sailfishos-patches / patchmanager

Patchmanager for SailfishOS
https://openrepos.net/content/patchmanager/patchmanager
Other
21 stars 22 forks source link

Update testcases to include more patch variants. #429

Open nephros opened 1 year ago

nephros commented 1 year ago

Update the test cases with the information gathered in #426:

Either of the styles should work, with "lib" being either lib or lib64, and either resulting patch should apply on the other system (naturally).

Looking at my personal collection of installed patches, we have these styles of patch beginnings we should be able to deal with:

--- /usr/...
+++ /usr/...
--- a/usr/...
+++ b/usr/...
--- orig/usr/...
+++ patch/usr/...
--- jolla/usr/...
+++ developername/usr/...

And special case variants of the above for new files:

--- /dev/null
+++ {user,b,patch}/usr/...

Originally posted by @nephros in https://github.com/sailfishos-patches/patchmanager/issues/426#issuecomment-1509761534

nephros commented 1 year ago

... also, actually test stuff using the test cases ;))

nephros commented 1 year ago

Actually the cases are already there (e.g. 32-to-64/ and 32-to-64-alt), but the evil thing about #426 is, the bug will not appear if the files to be patched have already been created previously by an unaffected patch.