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

Phoenicis PlayOnLinux #803

Closed
wants to merge 13 commits into from
Closed

Phoenicis PlayOnLinux #803

wants to merge 13 commits into from

Conversation

plata
Copy link

@plata plata commented Jan 5, 2019

see https://www.playonlinux.com/en/comments-1368.html

Note: We are currently in alpha phase. While the build is reproducible because it uses a zip file matching our released alpha 2 version, I'm not sure if this is sufficient regarding your requirements for stable applications. Nevertheless, I wanted to get your feedback so I can improve the flatpak before the final release.

"sdk-extensions": [
"org.freedesktop.Sdk.Extension.openjdk11"
],
"add-build-extensions": {
Copy link
Member

Choose a reason for hiding this comment

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

Isn't add-extensions enough here?

Copy link
Author

Choose a reason for hiding this comment

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

I'm not sure. I had this discussion here: flatpak/flatpak-builder#223. My final understanding was that I should rather use "add-build-extensions".

Copy link
Contributor

Choose a reason for hiding this comment

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

I am really not sure if the compat libs should be added as build-extensions. Try changing it to add-extensions.

Copy link
Author

Choose a reason for hiding this comment

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

It's updated.

Copy link
Contributor

Choose a reason for hiding this comment

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

@barthalion Any idea why the 32-bit compat extensions are not pulled when installing the Flatpak?

org.phoenicis.playonlinux.json Outdated Show resolved Hide resolved
@barthalion
Copy link
Member

testbot, build org.phoenicis.playonlinux

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Queued test build for org.phoenicis.playonlinux.

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Started test build 1011

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Build 1011 failed

@plata
Copy link
Author

plata commented Jan 5, 2019

I have a question regarding access: Can you give the access to the created repository to our whole organization (PhoenicisOrg) instead of only me?

@nedrichards
Copy link
Member

@plata I don't believe we can currently delegate it to an organisation from another project (I think that's a github limitiation but I'd be delighted to be proved wrong), but we can add as many usernames as you'd like.

@nedrichards
Copy link
Member

bot, build org.phoenicis.playonlinux
testbot, build org.phoenicis.playonlinux

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Queued test build for org.phoenicis.playonlinux.

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Started test build 1012

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Build 1012 failed

@plata
Copy link
Author

plata commented Jan 6, 2019

Ok. In this case please add @qparis and @madoar.

@plata
Copy link
Author

plata commented Jan 6, 2019

Any hints why GL32 is not found in the build? I can build locally and I think Steam uses the same.

@AsciiWolf
Copy link
Contributor

testbot, build org.phoenicis.playonlinux

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Queued test build for org.phoenicis.playonlinux.

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Started test build 1140

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Build 1140 successful
To test this build, install it from the testing repository:

flatpak install --user http://repo-test.flathub.org:8080/build-repo/1071/org.phoenicis.playonlinux.flatpakref

@plata
Copy link
Author

plata commented Jan 10, 2019

Ok, so the GL32 version update fixed the build.

},
"command": "phoenicis-playonlinux",
"finish-args": [
"--filesystem=home",
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the full home directory access really needed?

Copy link
Author

Choose a reason for hiding this comment

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

We use ~/.Phoenicis only. Is it possible to restrict it to that?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yup. :-) You can use --persist=.Phoenicis.

Copy link
Contributor

Choose a reason for hiding this comment

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

By the way, there was also a .org.phoenicis.javafx.JavaFXApplication directory created in home by PoL, with webview/localstorage subdirectory and webview/.lock file inside.

Copy link
Author

Choose a reason for hiding this comment

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

I've just tried this but it doesn't seem to work. I think I must adjust the directory (Java property) we are using somehow. Where is the directory mapped to inside the runtime?

Copy link
Author

Choose a reason for hiding this comment

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

It works but I don't think it is using ~/.Phoenicis. Try the following: Run Phoenicis PlayOnLinux without flatpak and install an app. Then, run Phoenicis PlayOnLinux with flatpak. You will not see the app. We don't want the apps to reside in the runtime because Phoenicis is just a mean to install them (they do not belong to it). You might want to compare it e.g. to a text editor where you can only save files inside the flatpak.

Copy link
Contributor

@AsciiWolf AsciiWolf Jan 11, 2019

Choose a reason for hiding this comment

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

One of the major points of Flatpak is that it makes apps (including their file system access) sandboxed. The text editor can use a portal to open/save files, it doesn't need full file system access. Anyway, if you really want the .Phoenicis directory to be unsandboxed, --filesystem=~/.Phoenicis should work fine.

Copy link
Author

Choose a reason for hiding this comment

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

Ok, I will try that.

Choose a reason for hiding this comment

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

Running same apps with and without flatpak will create a mess.

We don't want the apps to reside in the runtime because Phoenicis is just a mean to install them (they do not belong to it).

That doesn't makes sense. How those apps are supposed to run without the runtime?

Copy link
Author

Choose a reason for hiding this comment

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

It works.

I've noticed that the behavior is a bit strange though: You can either select a file or drag and drop it. Drag and drop accepts any file (even from directories which cannot be accessed). The file cannot be opened but that's not really obvious because the drag and drop accepts it first.

@AsciiWolf
Copy link
Contributor

It looks like that there is some problem with the Wine path, I tried installing 7-Zip and couple other apps using the Flatpak PoL and this was the result:

PoL Wine not found error

@plata
Copy link
Author

plata commented Jan 10, 2019

Is that with the current manifest or with --persist=.Phoenicis?

@AsciiWolf
Copy link
Contributor

With the current manifest.

@plata
Copy link
Author

plata commented Jan 10, 2019

Just did a clean build and installed and tested Notepad++. I remember having this issue before as well when 32bit was not working correctly. Do you have org.freedesktop.Platform.Compat.i386 installed?

@AsciiWolf
Copy link
Contributor

AsciiWolf commented Jan 10, 2019

Hmm, you're right. I did not have the compat runtime installed. It doesn't seem to be installed automatically when installing from flatpakref. PoL now works fine after installing the 32-bit compat runtime.

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Started test build 1252

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Build 1252 successful
To test this build, install it from the testing repository:

flatpak install --user http://repo-test.flathub.org:8080/build-repo/1180/org.phoenicis.playonlinux.flatpakref

@barthalion
Copy link
Member

Is it ready to merge? Looks like all reported issues are related to POL itself, not flatpak specifically.

@plata
Copy link
Author

plata commented Jan 21, 2019

@LeandroStanger I've added cabextract. Adobe Reader is working now.

@barthalion I think it can be merged. We will probably find additional dependencies required by certain applications but I guess that's ok.

@AsciiWolf
Copy link
Contributor

testbot, build org.phoenicis.playonlinux

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Queued test build for org.phoenicis.playonlinux.

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Started test build 1443

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Build 1443 successful
To test this build, install it from the testing repository:

flatpak install --user http://repo-test.flathub.org:8080/build-repo/1369/org.phoenicis.playonlinux.flatpakref

@plata
Copy link
Author

plata commented Jan 22, 2019

@LeandroStanger we do not have a newer release currently.

@Erick555
Copy link

@plata could you explain how this project works? Is it possible to install and run wine apps through it in flatpak environment?

@plata
Copy link
Author

plata commented Jan 22, 2019

@Erick555 yes. If you know PlayOnLinux, Phoenicis PlayOnLinux is the next version.

@Erick555
Copy link

Erick555 commented Jan 22, 2019

Great. So --filesystem=~/.Phoenicis is needed only when user want to share it with Phoenicis installed on host and/or run wine apps directly on host, right?

@plata
Copy link
Author

plata commented Jan 22, 2019

Well, technically the applications are installed to ~/.Phoenicis but executed in the flatpak. Originally, I installed the applications inside the flatpak directly but that turned out to be rather strange (e.g. if you want to run a Phoenicis which got installed via .deb or so).

@aiamuzz
Copy link

aiamuzz commented Jan 22, 2019

@plata ... i was trying to test this ... the installation went fine ... but the applications inside (adobe, notepad++ ) just didn't install successfully ... throws up error ...

would you entertain errors here on this PR or you want me to wait until the repo gets created and post the issue there ?

@plata
Copy link
Author

plata commented Jan 22, 2019

@aiamuzz it's probably the best if you open a new issue in https://github.com/PhoenicisOrg/phoenicis. If we find that there's an error in the flatpak, I can fix it here as well but we won't bloat this PR.

@aiamuzz
Copy link

aiamuzz commented Jan 22, 2019

@aiamuzz it's probably the best if you open a new issue in https://github.com/PhoenicisOrg/phoenicis. If we find that there's an error in the flatpak, I can fix it here as well but we won't bloat this PR.

i am testing this flapak build ... that repo is for the normal build isn't it ... i thought this issue could be with the flatpak'ing that's underway here ...

anyways ... i think i'll wait until the flathub repo is created for this application and post the issue there ...

@plata
Copy link
Author

plata commented Jan 22, 2019

The repo is for the software in general. If you have an issue with Phoenicis, it's perfectly fine to open an issue there. Especially if you're not completely sure if it's even related to the flatpak.

@barthalion
Copy link
Member

bot, build org.phoenicis.playonlinux
testbot, build org.phoenicis.playonlinux

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Queued test build for org.phoenicis.playonlinux.

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Started test build 1498

@flathubbot
Copy link

NOTE: This comment was posted by the test instance of buildbot

Build 1498 successful
To test this build, install it from the testing repository:

flatpak install --user http://repo-test.flathub.org:8080/build-repo/1422/org.phoenicis.playonlinux.flatpakref

@barthalion
Copy link
Member

Repository has been created: https://github.com/flathub/org.phoenicis.playonlinux

@plata, should someone besides you get write access to the repository above?

Thanks!

@barthalion barthalion closed this Jan 23, 2019
@plata
Copy link
Author

plata commented Jan 23, 2019

Thank you!

Yes, please give write access also to @qparis and @madoar.

@plata plata deleted the phoenicis branch January 23, 2019 18:31
@barthalion
Copy link
Member

Invitations sent.

@plata
Copy link
Author

plata commented Jan 23, 2019

Thanks.

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.

None yet

10 participants