-
Notifications
You must be signed in to change notification settings - Fork 23
Add crash reporting #240
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
Add crash reporting #240
Conversation
|
Managed to find and resolve #239 by using the crash reporting! |
|
Building in the toltec toolchain is currently blocked by toltec-dev/toolchain#13 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 527177 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 527375 lines exceeds the maximum allowed for the inline comments feature.
|
Code Climate has analyzed commit f7e80f1 and detected 135 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
ContributorsCommit-Lint commandsYou can trigger Commit-Lint actions by commenting on this PR:
|
318c7ae to
02724ff
Compare
|
@LinusCDE would you be willing to test again? The build is now working, so you should be able to get ipk files to simplify the install process. |
|
Will look to do so in the next few days. |
@LinusCDE thanks! |
|
@LinusCDE did you get a chance to look at this? |
Not yet. Gotta look into some Linux Container/Namespace books the coming weeks anyway and also wanted to look into some new progression fantasy book to read. Was caught up this week, syncing my new backup yubikey and finally did all the openpgp+ssh madness with it (also on my smartphone) as well as setting up gpg signing all commits by default. |
|
Installed oxide from the lasted built packages here: Using fw version 2.12.1.527 and had some later packages installed already, so I needed to add |
|
@LinusCDE that's very odd. I'm also running 2.12.1.527 on my rM1, but I'm not having any issues. This might be an issue with the toltec toolchain? I develop with the reMarkable supplied toolchain. Could you try removing all the packages and installing fresh? Maybe it failed to install liboxide properly? @matteodelabre any ideas? |
|
@matteodelabre poke? |
|
@LinusCDE did you get a chance to try removing and reinstalling fresh yet? |
|
Will look to do so tomorrow. I did do a full reinstall of oxide which I thought was enough. But should do a toltec reinstallation anyway. |
|
Did update my firmware and rather then reenable, wiped and reinstalled toltec fresh with the instructions from the website. Afterwards I installed all ipkg files from the newest build here and removed I also set Seems the file specified is not packaged yet. |
|
@LinusCDE looks like I was missing the liboxide package in the build. This should include it. |
|
The "liblib" in question is installed, but a symlink to [...].2 is missing. Adding the symlink manually fixes it for now. Started without any config (removed Oxide asks for the kind of telemetry, then it asks whether I want a pin. After I select "no pin" or a pin (and then successfully enter one twice), the app freezes forever. I cant access the task manager (swipe from bottom) and |
|
After starting tarnish again, the launcher properly boots (the pin decision was saved properly) and everything works again. But initial launch currently doesn't work without ssh access and some medium knowledge. Here is the log again for the first launch until freeze: https://haste.cosmos-ink.net/silucyheky.log Notably the freeze doesn't log anything beforehand and the last "No pin" message actually appeared with the pin setup dialog. I selected no pin in this instance the the app froze while clicking that. |
|
Interesting, I'll try the entire first setup again and poke at fixing the build to include the links. |
|
@LinusCDE This should fix the missing symlinks. I'm not sure what is causing the freeze for you, as I'm unable to replicate it. Could it be a rM2 only issue? |


Fix #238
Fix #239
Fix #243
Fix #242
Fix #244
Fix #246
Create a shared library to help reduce code size, and provide another API option for people developing applications.