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

Replace FF-Multi-Converter with FF-Converter fork #2513

Closed
4 tasks done
l-koehler opened this issue Jan 5, 2024 · 9 comments
Closed
4 tasks done

Replace FF-Multi-Converter with FF-Converter fork #2513

l-koehler opened this issue Jan 5, 2024 · 9 comments

Comments

@l-koehler
Copy link

What is the name of the app?

FF-Converter

(Optional) Where is the app hosted?

https://github.com/l-koehler/FF-converter/

About the app

The app is a fork of lstam/FF-Multi-Converter, which is no longer being developed.
It can convert most common file formats into other formats.
It has the same features as the original, plus some extra functionality (pandoc and some other tools available for conversion, "All Formats" tab).
As far as I know, it should not have any major issues that aren't also present in the original, except that I did not translate all new features into each supported language (only english, german and french are complete).

Confirmations

Copy link
Contributor

github-actions bot commented Jan 5, 2024

Hello there 👋
Thanks for submitting your first issue to the Pi-Apps project! We'll try to get back to you as soon as possible.
In the meantime, we encourage you join our Discord server, where you can ask any questions you might have.

Please respond as soon as possible if a Pi-Apps maintainer requests more information from you. Stale issues will be closed after a lengthy period of time with no response.

@theofficialgman
Copy link
Collaborator

regarding https://github.com/l-koehler/FF-converter?tab=readme-ov-file#troubleshooting that is a known issue with pipx, it doesn't have the ability to install .desktop files and was deemed "out of the project scope" (see pypa/pipx#717)

it is easy to copy the file out of the installed directory though and place it in the proper location since it does reside in the pipx installed directory similar to what we do for Quartz

# add .desktop symlinks since pipx does not create them https://github.com/pypa/pipx/issues/717
sudo mkdir -p /usr/local/share/applications /usr/local/share/icons
sudo ln -sf /usr/local/pipx/venvs/quartz-browser/share/applications/quartz-browser.desktop /usr/local/share/applications/quartz-browser.desktop
sudo ln -sf /usr/local/pipx/venvs/quartz-browser/share/icons/quartz-browser.png /usr/local/share/icons/quartz-browser.png

l-koehler added a commit to l-koehler/FF-converter that referenced this issue Jan 6, 2024
@l-koehler
Copy link
Author

thanks!
added that to the install instructions, as the previous workaround didn't work with the icon.

@theofficialgman
Copy link
Collaborator

@Botspot what are your thoughts on this? should we deprecate FF Multi Converter and instruct users in the popup to install FF Converter or should we simply change the scripts to install FF Converter while keeping the name?

@Botspot
Copy link
Owner

Botspot commented Jan 6, 2024

Let's transition them and keep the name the same.

@l-koehler
Copy link
Author

@theofficialgman
thanks for the update!
small problem: the install script is not automatically installing all optional dependencies.
the fork can also use pandoc, ar, zip and squashfs-tools. (tar too, but that should be preinstalled).
also, maybe it should only fail if python3 or pyqt5 fail to install, not if a optional dependency fails.
i would suggest to replace line 12 with

install_packages python3 python3-pyqt5 python3-distutils || exit 1
install_packages tar ar squashfs-tools pandoc ffmpeg imagemagick unoconv || echo "Could not install all optional dependencies"

(not sure about what pi-apps uses to show non-fatal errors, might need to replace the echo)

@theofficialgman
Copy link
Collaborator

@l-koehler we can install all packages that are available. there isn't any downside for having all the dependencies installed even if they are optional. with them installed the software has more features.

what is the ar that you are referring to? that isn't a package name and the binary under that name is from binutils and is the GNU assembler which I don't think you meant to write.

@l-koehler
Copy link
Author

l-koehler commented Jan 6, 2024

Part of binutils, shouldn't need install on most distributions.

A utility for creating, modifying and extracting from archives.

Don't know why I included it, i guess it was missing for me once.

@theofficialgman
Copy link
Collaborator

The dependencies suggested have been added

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

No branches or pull requests

3 participants