Skip to content

Conversation

bakabtw
Copy link

@bakabtw bakabtw commented Aug 18, 2024

Please confirm your submission meets all the criteria

  • Please describe the application briefly.
    HopToDesk is a free remote desktop tool allowing users to share their screen, transfer files, and allow remote control access to their computers and devices using end-to-end encryption.

  • The domain used for the application ID is controlled by the application developers and the application id guidelines are followed.

  • I have read the and followed all the Submission and licence requirements.

  • I have built and tested the submission locally.

  • I am an author/developer/upstream contributor of the project. If not, I contacted upstream developers about this submission. Link:

Copy link
Contributor

Choose a reason for hiding this comment

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

There is an desktop file inside the deb

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 true but it's malformed, and it doesn't pass lint checks. Therefore, I created an updated desktop file

Copy link
Contributor

Choose a reason for hiding this comment

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

malformed how?

Copy link
Author

Choose a reason for hiding this comment

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

It includes new-window action that is not actually supported:

Actions=new-window;

X-Desktop-File-Install-Version=0.23

[Desktop Action new-window]
Name=Open a New Window

I removed this part to avoid alerts from flatpak-builder lint

Copy link
Contributor

Choose a reason for hiding this comment

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

we just run desktop-file-validate on it.

hoptodesk.desktop: error: value "1.42.1" for key "Version" in group "Desktop Entry" is not a known version
hoptodesk.desktop: warning: value "Remote Desktop Software" for key "Comment" in group "Desktop Entry" looks the same as that of key "GenericName"
hoptodesk.desktop: error: required key "Exec" in group "Desktop Action new-window" is not present

Copy link
Author

Choose a reason for hiding this comment

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

Exactly, that why I'm using a self-made desktop file

Copy link
Contributor

Choose a reason for hiding this comment

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

These should be fixed, for now I guess it's fine.

@bbhtt bbhtt added the awaiting-changes Pull request waiting for inputs or changes from author label Aug 18, 2024
@bbhtt
Copy link
Contributor

bbhtt commented Aug 18, 2024

I am an author/developer/upstream contributor of the project. If not, I contacted upstream developers about this submission. Link:

Is this your project? If not did you contact upstream?

@bakabtw
Copy link
Author

bakabtw commented Aug 18, 2024

I am an author/developer/upstream contributor of the project. If not, I contacted upstream developers about this submission. Link:

Is this your project? If not did you contact upstream?

It's not my project but the author asked me to help with Flathub submission

bakabtw and others added 2 commits August 18, 2024 15:14
Co-authored-by: bbhtt <bbhtt.zn0i8@slmail.me>
"buildsystem" : "simple",
"sources" : [
{
"url": "https://www.hoptodesk.com/hoptodesk.deb",
Copy link
Contributor

Choose a reason for hiding this comment

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

the .deb isn't versioned? This will be great to have the package not build at random times.

Copy link
Contributor

Choose a reason for hiding this comment

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

also why isn't it built from source given that licensing isn't a problem. This would also allow supporting aarch64 at the same time.

Co-authored-by: Hubert Figuière <hub@figuiere.net>
bakabtw and others added 4 commits August 18, 2024 19:50
Co-authored-by: Hubert Figuière <hub@figuiere.net>
Co-authored-by: Hubert Figuière <hub@figuiere.net>
bakabtw and others added 4 commits August 19, 2024 21:43
Co-authored-by: bbhtt <bbhtt.zn0i8@slmail.me>
Co-authored-by: bbhtt <bbhtt.zn0i8@slmail.me>
@bakabtw bakabtw requested review from bbhtt and hfiguiere August 19, 2024 15:06
@bbhtt
Copy link
Contributor

bbhtt commented Aug 19, 2024

bot, build com.hoptodesk.HopToDesk

@flathubbot
Copy link

Queued test build for com.hoptodesk.HopToDesk.

@flathubbot
Copy link

Started test build 141081

@flathubbot
Copy link

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

flatpak install --user https://dl.flathub.org/build-repo/124135/com.hoptodesk.HopToDesk.flatpakref

@bbhtt
Copy link
Contributor

bbhtt commented Aug 21, 2024

bot, build com.hoptodesk.HopToDesk

@flathubbot
Copy link

Queued test build for com.hoptodesk.HopToDesk.

@flathubbot
Copy link

Started test build 141472

@flathubbot
Copy link

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

flatpak install --user https://dl.flathub.org/build-repo/124528/com.hoptodesk.HopToDesk.flatpakref

@bbhtt bbhtt added ready Pull request ready for final review and merge and removed awaiting-changes Pull request waiting for inputs or changes from author labels Aug 21, 2024
@barthalion
Copy link
Member

bot, build com.hoptodesk.HopToDesk

@flathubbot
Copy link

Queued test build for com.hoptodesk.HopToDesk.

@flathubbot
Copy link

Started test build 141608

@flathubbot
Copy link

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

flatpak install --user https://dl.flathub.org/build-repo/124665/com.hoptodesk.HopToDesk.flatpakref

@barthalion
Copy link
Member

/merge

@flathubbot
Copy link

A repository for this submission has been created: https://github.com/flathub/com.hoptodesk.HopToDesk

You will receive an invitation to be a collaborator which will grant you write access to the repository above. The invite can be also viewed here.

If you've never maintained an app on Flathub before, common questions are answered in the app maintenance guide. If you're the original developer (or an authorized party), verify your app next to let users know it's coming from you.

Please follow the Flathub blog for the latest announcements.

Thanks!

@flathubbot flathubbot closed this Aug 22, 2024
@bbhtt bbhtt mentioned this pull request Apr 28, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready Pull request ready for final review and merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants