Closed daboynb closed 9 months ago
Hi @daboynb, thanks for pointing this out. Due to that fact this specific line change would break functionality for devices using other rooting methods such as KSU, I'll need to implement binary path detection as with wget
/curl
to improve compatibility.
I already have some changes to the script in the works so I'll add this in and tag you as a coauthor on that commit.
Perfect, thank you
Hi, I just want to add two things :
@daboynb Thanks for these added observations. The latest version of the script will attempt to use busybox if available which should provide much more consistent results (busybox ships with both Magisk and KSU). I've also replaced the variable-set conditional as per your second point, though I should add that your change above will always return true
because the dollar sign is missing before FORCEABI
. I'll let you know when I push the new version out. 🙂
Perfect, thank you again :)
I saw it now -> https://github.com/TheFreeman193/PIFS/pull/1/commits/3b75d2b01f841d4aea83d6333471f63e32cdea08 this was my last commit, I wrote the wrong fix on my previous post ahah
Hi, on my device I got the error unzip not found, this fixes the issue.