Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added using Substitute as an option #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

syto203
Copy link

@syto203 syto203 commented Feb 23, 2021

No description provided.

@syto203 syto203 force-pushed the Substitute branch 2 times, most recently from b24a641 to f383efb Compare February 23, 2021 09:18
@Randomblock1
Copy link

This works really well, but I've found an issue with it. When using ./install to download Substitute, switching to Substrate, and then back to Substitute, it won't put Cydia Substrate Framework in the IPA's Frameworks. In modules/lib/, Cydia Substrate Framework is named CydiaSubstrate_0.9.6301.framework instead of just CydiaSubstrate.framework, so something is not working right in the install script.

I think this is because the install script starts permanently removing files around here.
It would be much better if it used symbolic links instead to preserve every file. For example, instead of ipa_substitute.sh completely replacing and deleting ipa.sh, ipa.sh is just a symbolic link to either the new ipa script or the old one (which would be renamed to ipa_substrate.sh). This way, switching between the two doesn't delete any files, just changes links.

That being said, git reset --hard to reset the repository and then just running ./install again fixes it.

@syto203
Copy link
Author

syto203 commented Mar 12, 2021

Sorry I was half asleep when I submitted this hence the multiple pushes. I didn’t take into account someone juggling between both but the fix will be to remove the Cydiasubstrate.framework and download and use the new one. I’ll modify the script soon.

also removed downloaded binaries so you always update to latest if you run the installer.

Signed-off-by: syto203 <26351342+syto203@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants