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

Added a shipping readme that contains information about packaging #11937

Merged
merged 4 commits into from
Oct 21, 2024

Conversation

dragotin
Copy link
Contributor

This is hopefully useful information for downstream packagers.

@mmattel
Copy link
Contributor

mmattel commented Oct 17, 2024

This true for the upcoming and subsequent releases an should be noted as such.

PACKAGING.md Outdated

Here are some hints for packager of the linux distributions which we kindly ask to change their packages in the distros to help us providing the best ownCloud client integration experience:

The shell integrations are available for KDE Dolphin [1], Nautilus and Caja [1]. While KDE Dolphin needs compiled code, the latter two are using python. [ADD LINKS].
Copy link
Member

Choose a reason for hiding this comment

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

@dragotin which links are you expecting here? can you add them or at least point me in some direction? THX 😍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have pushed a commit that adds the links to the source repos of the discussed extensions. I hope that is sufficient.

Copy link
Member

Choose a reason for hiding this comment

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

THX a lot

@DeepDiver1975
Copy link
Member

DeepDiver1975 commented Oct 18, 2024

This true for the upcoming and subsequent releases an should be noted as such.

what is you expectation? @mmattel

to me it is valid on the branch(es) where this file is contained to state the current status.
older and possible future branches not holding this file are not affected - so we might want to backport this change if it applies ....

@mmattel
Copy link
Contributor

mmattel commented Oct 18, 2024

what is you expectation?

as discussed in online, just add a remark that this is available with +6.0

PACKAGING.md Show resolved Hide resolved

## Integration Problems

The ownCloud desktop client gets shipped as AppImage because it is effortless to create, independent from the underlying distro and easy to install, as long as the distribution is not providing a package on their own which is frequently updated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The ownCloud desktop client gets shipped as AppImage because it is effortless to create, independent from the underlying distro and easy to install, as long as the distribution is not providing a package on their own which is frequently updated.
The ownCloud Desktop App gets shipped as AppImage because it is effortless to create, independent from the underlying distro and easy to install, as long as the distribution is not providing a package on their own which is frequently updated.

Copy link
Member

Choose a reason for hiding this comment

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

Where is this coming from? We ever since name it 'ownCloud Desktop Client' ... 😕

Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this coming from? We ever since name it 'ownCloud Desktop Client' ... 😕

This was decided about +2y ago by mstingl. Consequently I named this is all docs + versions. We now have this harmonized naming for the Desktop App, iOS App and Android App.

Copy link
Member

Choose a reason for hiding this comment

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

Well but here we are in the client repo and talk about dev facing documentation.

Copy link
Contributor

Choose a reason for hiding this comment

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

It's about consistency which is independent of where we are. just my 2c

Copy link
Member

Choose a reason for hiding this comment

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

It is indeed named that way on the web page as well .... https://owncloud.com/desktop-app/

THX

Copy link
Member

Choose a reason for hiding this comment

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

... but is anybody talking about apps in the context of desktop? 😕

@mmattel
Copy link
Contributor

mmattel commented Oct 18, 2024

@DeepDiver1975 I have added some suggestable commits and named the SW correctly: Desktop App

Co-authored-by: Martin <github@diemattels.at>
@DeepDiver1975 DeepDiver1975 merged commit 2e71f7b into master Oct 21, 2024
2 of 6 checks passed
@DeepDiver1975 DeepDiver1975 deleted the add_shipping_md branch October 21, 2024 14:21
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.

4 participants