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

Removed unneeded permissions and disabled the ogra option #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bhdouglass
Copy link
Contributor

I removed the unnecessary permissions, disabled the ogra option (since Oxide isn't available on pinephone and is depricated), updated the clickabe.json to use the latest syntax, and removed reference to the OpenStore in the description field.

I ran into an error when testing this: *** buffer overflow detected ***: /usr/lib/qt5/bin/qmlscene terminated with and without my changes, so I'm not sure what was up with that as the app from the OpenStore works fine.

@cibersheep
Copy link
Collaborator

If I remember correctly, the files needed to build the .click are written from the stdout.
Probably the lib is expecting some var that is not anymore with the removal of ogra container?

https://github.com/Jujuyeh/webapp-creator/blob/master/webapp-creator/backend/modules/Webapp_Creator/mytype.cpp#L36

@bhdouglass
Copy link
Contributor Author

@cibersheep but I only removed the UI element to select it, so I wouldn't think that would be a problem. Also I tried with and without my changes both times lead to the same crash.

@cibersheep
Copy link
Collaborator

Oh, I see. Sorry.
A hidden until now bug, then?

@bhdouglass
Copy link
Contributor Author

Maybe? Hopefully @Jujuyeh has some better insight here.

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