Conversation
Change the dependence to com.spark.libsparkapplist from Applist. Remove the extra plist file in /Library/MobileSubstrate/DynamicLibraries/ folder, which leads error in latest Substitute.
|
Thanks for the contribution, I will hopefully test, merge and update on repo later (I'll need to get my tweak compiling environment updated probably xd).
I remember considering updating this to use SparkAppList so that's good. Also good catch on the issue with Substitute being the first image rather than app's image, I had no chance of testing this library before contrary to Substrate and libhooker.
Hm, so it seems Substitute is the only of three that first checks for plist and then searches for dylib rather than the other way around. In fairness the existence of that file was a low-effort way to ensure a package update won't conflict with a manually modified file and override it or, worse, throw an error to the user. I'll probably need to re-think/re-check if that would even happen at all, it might be fine as-is, but otherwise a cleaner solution than before could be to only create a basic file there in postinst only if it didn't exist, and that would solve the concern in a similar way I guess... |
|
I agree with your review. I started to write some obj-c from these week for my little project, |
|
@fenfenS I’m sorry to bother you, but could you please upload the deb for your fork? It would be highly appreciated because I have no experience whatsoever with building deb files for tweaks from source… Thank You! :) |
|
Got the plist trouble in DynamicLibraries folder again. I was wrong about the bug. |
|
Was the tweak updated to ios 16 ? |
No description provided.