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

Add com.basiomeuspuga.Lector.json #891

Closed
wants to merge 1 commit into from

Conversation

BasioMeusPuga
Copy link

Lector is a Qt based ebook reader with support for epub, pdf, fb2, mobi, azw / azw3 / azw4, cbr / cbz

See https://github.com/BasioMeusPuga/Lector

"rename-desktop-file":"lector.desktop",
"rename-appdata-file":"lector.appdata.xml",
"finish-args":[
"--filesystem=host",
Copy link
Member

Choose a reason for hiding this comment

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

what do you need host permissions for? That seems uncomfortably broad.

Copy link
Author

Choose a reason for hiding this comment

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

I imagine books can be located anywhere in the filesystem. I myself keep my books stored in an NTFS filesystem shared between Windows and Linux that's mounted outside my home directory. That said, is it better to go with "--filesystem=host:ro"?

Copy link
Member

Choose a reason for hiding this comment

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

It would be better to use xdg-desktop-portal but I'm not actually sure what it takes for Qt. @aleixpol any pointers?

Choose a reason for hiding this comment

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

That I know of, it's not possible to open directories using portals.

@nedrichards
Copy link
Member

bot, build com.basiomeuspuga.Lector

@flathubbot
Copy link

Queued test build for com.basiomeuspuga.Lector.

@flathubbot
Copy link

Started test build 799

@@ -0,0 +1,143 @@
{
"app-id":"com.basiomeuspuga.Lector",
Copy link
Member

Choose a reason for hiding this comment

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

this domain name doesn't exist? com.github.BasioMeusPuga.Lector might be better (or io.github.BasioMeusPuga.Lector if you have a website there) a bit more on app-id selection http://docs.flatpak.org/en/latest/conventions.html#application-ids

Copy link
Author

Choose a reason for hiding this comment

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

Understood. This is my first time building one of these. Thank you for being patient.

@flathubbot
Copy link

Build 799 failed

@nedrichards
Copy link
Member

So the appdata doesn't pass the https://github.com/flathub/flathub/wiki/AppData-Guidelines listed above on x86-64, and you're trying to load x86-64 only python wheels on the other architectures which is why they fail.

@BasioMeusPuga
Copy link
Author

I guess I'm going to go into the documentation a little. I'll get back when I can figure out how to get some of these packages to compile from source in the manifest.

@nedrichards
Copy link
Member

if it's impossible you can filter architectures but we only do that was a last resort

@flathubbot
Copy link

This PR hasn't received any updates in a year and will be automatically closed in 14 days. Feel free to re-open it if you plan to continue working on that pull request or think it deserves attention from Flathub admins

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants