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

New Floorp-private-components repository #62

Closed
christoph-heiss opened this issue Mar 19, 2024 · 10 comments
Closed

New Floorp-private-components repository #62

christoph-heiss opened this issue Mar 19, 2024 · 10 comments

Comments

@christoph-heiss
Copy link

Hi!

I'm maintaining the Floorp package for NixOS.
While updating Floorp to the newly released v11.11.0, I noticed - since it failed the build - that a new Floorp-private-components repository/submodule was introduced as part of this repository.

What does it contain? Is it required to build Floorp?
If (esp. the latter) is true, than I will go ahead and drop the Floorp package for the next release of NixOS, since I'm not willing and never will support proprietary software.

@CutterKnife
Copy link

Sure, it is a proprietary part of Floorp. But AFAIK it should be nothing or not used yet.

The team is working on new features for the release of Floorp 12. They will be using a different repository with a different license for some of its new features. Maybe it will be available by the time it is released.

@surapunoyousei
Copy link
Member

We are in the process of implementing it so that it can be an open source project, but it will take time so do what you want.

I am the only maintainer of Floorp, so I need time.

@stefanwimmer128
Copy link

I maintain my own personal fork of Floorp, since I'm adding various patches. Is there a way for me to include these private components?

@christoph-heiss
Copy link
Author

Sure, it is a proprietary part of Floorp. But AFAIK it should be nothing or not used yet.

Well, than Floorp should to be explicitly marked as proprietary, so users know what they are downloading.
If you do not see the issue & impact of this - that's fine, it is your project after all.

But this change effectively prevents anyone from building Floorp. Be it in an automated fashion in distro packaging or individual users like @stefanwimmer128.
Simply due to the forced dependency via a git submodule on this proprietary compontent - which means this repository (which is needed for building of course) cannot be checked out anymore.

In any case, you have shattered my trust and further my willingness to package this for NixOS, other than it being literally impossible now. All users of Floorp on NixOS (of which there are definitely a few) cannot receive updates for Floorp anymore.
Please understand the impact of this.

I'll go ahead and deprecate the package for NixOS and hopefully be able to drop this with the upcoming release.

@stefanwimmer128
Copy link

Well 1. you can definitely still build it without the proprietary components. The AUR-floorp package, the firedragon-fork and my fork do work without those components, those features are just missing, but the browser works.

But 2. I can at least ask.

@CutterKnife
Copy link

CutterKnife commented Mar 20, 2024

As already mentioned, this is not permanent. It will be open source at the time of release.

Due to the rude behavior of a project that is forking Floorp, they have been forced to make changes to protect some of their assets. This means that some file structures will change, but they assure you that the parts included in the actual Floorp will continue to be open source.

@surapunoyousei
Copy link
Member

surapunoyousei commented Mar 21, 2024

Sure, it is a proprietary part of Floorp. But AFAIK it should be nothing or not used yet.

Well, than Floorp should to be explicitly marked as proprietary, so users know what they are downloading. If you do not see the issue & impact of this - that's fine, it is your project after all.

But this change effectively prevents anyone from building Floorp. Be it in an automated fashion in distro packaging or individual users like @stefanwimmer128. Simply due to the forced dependency via a git submodule on this proprietary compontent - which means this repository (which is needed for building of course) cannot be checked out anymore.

In any case, you have shattered my trust and further my willingness to package this for NixOS, other than it being literally impossible now. All users of Floorp on NixOS (of which there are definitely a few) cannot receive updates for Floorp anymore. Please understand the impact of this.

I'll go ahead and deprecate the package for NixOS and hopefully be able to drop this with the upcoming release.

I recommend that it be removed.

We have addressed this by granting access to maintainers such as FireDragon, but if you only maintain open source project, you may want to turn it off.

I respect your decision.

PS: It may have been annoying to you, but please don't let it interfere with my decision. I have been creating browsers for several years and I am not making any money. It is completely a hobby.

@surapunoyousei
Copy link
Member

surapunoyousei commented Mar 21, 2024

I maintain my own personal fork of Floorp, since I'm adding various patches. Is there a way for me to include these private components?

Add permission to you. But, be careful store Floorp code

@stefanwimmer128
Copy link

Thanks!

@dominichayesferen
Copy link

dominichayesferen commented Mar 24, 2024

Aftermath context for those who may end up in here:
https://blog.ablaze.one/4125/2024-03-11/

https://github.com/Floorp-Projects/Floorp-private-components

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

No branches or pull requests

5 participants