Skip to content

Remove Zebra#58

Closed
imnltsa wants to merge 5 commits into
roothide:mainfrom
imnltsa:nozebra
Closed

Remove Zebra#58
imnltsa wants to merge 5 commits into
roothide:mainfrom
imnltsa:nozebra

Conversation

@imnltsa
Copy link
Copy Markdown
Collaborator

@imnltsa imnltsa commented Jan 6, 2024

Zebra doesn't really fit in the RootHide Bootstrap unless properly patched for FULL rootless repository support. Installing RootHide Patcher through Zebra gives a bunch of errors, and I think it's just too much for people (a lot have asked how to fix it). It's just better to use the patched Sileo that gives conversion warnings.

This PR may be hacky though, it was done manually, I don't know how much it will mess up the .pbxproj since it wasn't edited through Xcode (still builds regardless).

Built, installed, and tested on real hardware. Zebra is now no longer a default application. Seems to work fine with icon cache too.

Copy link
Copy Markdown

@znexo znexo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, Zebra doesn’t work entirely, and many have to stick to using Sileo. All that was done was the removal of the deb and the installation process of it.

Copy link
Copy Markdown

@boat1337 boat1337 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

zebra can suck on my left nut

Copy link
Copy Markdown

@null-socks null-socks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add cydia pls

@imnltsa
Copy link
Copy Markdown
Collaborator Author

imnltsa commented Jan 6, 2024

Agreed, Zebra doesn’t work entirely, and many have to stick to using Sileo. All that was done was the removal of the deb and the installation process of it.

Yeah, though what I'm not sure on is if the .pbxproj file is now corrupted. Still builds and works fine though.

@imnltsa
Copy link
Copy Markdown
Collaborator Author

imnltsa commented Jan 6, 2024

A good alternative to viewing rootful repositories is Postbox from https://postbox.news/

@imnltsa imnltsa added the enhancement New feature or request label Jan 8, 2024
@roothide
Copy link
Copy Markdown
Collaborator

roothide commented Jan 8, 2024

In the public release version we should add an option for users to choose whether to install Zebra/Sileo

@imnltsa imnltsa mentioned this pull request Jan 9, 2024
@haxi0
Copy link
Copy Markdown
Contributor

haxi0 commented Jan 15, 2024

yea btw i think this is the right decision to remove zebra, it kinda works but buggy. anyways sileo is better modified for roothide

@roothide
Copy link
Copy Markdown
Collaborator

yea btw i think this is the right decision to remove zebra, it kinda works but buggy. anyways sileo is better modified for roothide

kirb won't be happy, maybe we can add an option to let the user decide to install sileo/Zebra?

@boat1337
Copy link
Copy Markdown

yea btw i think this is the right decision to remove zebra, it kinda works but buggy. anyways sileo is better modified for roothide

kirb won't be happy, maybe we can add an option to let the user decide to install sileo/Zebra?

Zebra being bundled would only be a good idea if it was properly patched for both rootless and roothide, which it isn't.

@imnltsa
Copy link
Copy Markdown
Collaborator Author

imnltsa commented Jan 16, 2024

yea btw i think this is the right decision to remove zebra, it kinda works but buggy. anyways sileo is better modified for roothide

kirb won't be happy, maybe we can add an option to let the user decide to install sileo/Zebra?

Zebra being bundled would only be a good idea if it was properly patched for both rootless and roothide, which it isn't.

This is exactly it. I think it's a little too much effort than its worth, needs to properly support both rootless repositories and roothide patching. Currently, you can't even install roothide Patcher with it.

@wwg135
Copy link
Copy Markdown
Contributor

wwg135 commented Jan 20, 2024

agree to delete zebra!!

@imnltsa
Copy link
Copy Markdown
Collaborator Author

imnltsa commented Jan 27, 2024

Duplicate of #235

@imnltsa imnltsa marked this as a duplicate of #235 Jan 27, 2024
@imnltsa imnltsa closed this Jan 27, 2024
@imnltsa imnltsa mentioned this pull request Jan 27, 2024
@imnltsa imnltsa deleted the nozebra branch January 27, 2024 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants