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

Adapt AppImage from GitHub build #301

Closed
ghutchis opened this issue Jul 6, 2018 · 10 comments · Fixed by #748
Closed

Adapt AppImage from GitHub build #301

ghutchis opened this issue Jul 6, 2018 · 10 comments · Fixed by #748

Comments

@ghutchis
Copy link
Member

ghutchis commented Jul 6, 2018

@probonopd contributed an AppImage upload feature for Avogadro using a Travis build:
cryos/avogadro#907

Assuming issues with Open Babel environment variables can be worked out, this seems like an excellent use for the Travis builds to provide pre-releases.

@probonopd
Copy link

probonopd commented Jul 6, 2018

Replacing AppRun with something like this should do the trick:

#!/bin/bash

# The purpose of this custom AppRun script is
# to export Open Babel environment variables
# before launching the application

HERE="$(dirname "$(readlink -f "${0}")")"

export BABEL_DATADIR=$(readlink -f "${HERE}/usr/share/openbabel/*/")
export BABEL_LIBDIR=$(readlink -f "${HERE}/usr/lib/openbabel/*/")

exec "${HERE}/usr/bin/avogadro" "$@"

@ghutchis
Copy link
Member Author

ghutchis commented Jul 6, 2018

Thanks - I don't have a Linux box handy, but I'll look at merging this, updating the script and we'll see if others can test.

@ghutchis ghutchis added the build label Jul 11, 2018
@fusion809
Copy link

fusion809 commented Jul 16, 2018

I'd be happy to test it out, I've got a heap of Linux installs I can try any AppImages ya build on. Virtual machines could be handy way for ya to test 'em out.

Bodhi Linux might be a good distro to use in a virtual machine to test it out, as its installer is simple and its installation is pretty quick, but most importantly it is minimal. Minimalism might be handy as it means if your AppImage can run under it, it's almost guaranteed it'll also work on other distros as they tend to have more libraries and programs pre-installed.

MX Linux is another suggestion for a distro, its installer is even simpler and faster to use (which I'm guessing would be handy as I'd imagine you're busy and don't have time to waste installing a distro). It isn't as minimal as Bodhi, but still.

@cryos
Copy link
Member

cryos commented Jul 16, 2018

For Avogadro 2 we try to figure out the correct environment variables to help Open Babel find its plugins, etc. See avogadro/qtplugins/openbabel/obprocess.cpp with BABEL_DATADIR and BABEL_LIBDIR, we also did that in avogadro/src/main.cpp in Avogadro 1.x. I remember Open Babel kept changing things, and that would occasionally break. Maybe we just need to fix that as it is used on Windows, macOS, and Linux (at least in theory), and should remove the need for a wrapper script.

@ghutchis
Copy link
Member Author

There's also linuxdeployqt here: https://github.com/probonopd/linuxdeployqt

At the moment, it requires Ubuntu 16.04, which had problems compiling everything. Hopefully the next release will support 18.04 LTS and we can build using the GitHub actions.

Another possibility now that it's on Flatpak, is flatpak2appdir - https://github.com/sudo-give-me-coffee/flatpak2appdir

@ghutchis ghutchis changed the title Adapt AppImage from Travis build Adapt AppImage from GitHub build Apr 22, 2021
@ghutchis
Copy link
Member Author

ghutchis commented May 9, 2021

Looks like this might be a good approach (another Qt-based app):
crow-translate/crow-translate@4f8a1f4

@probonopd
Copy link

At the moment, it requires Ubuntu 16.04, which had problems compiling everything. Hopefully the next release will support 18.04 LTS and we can build using the GitHub actions.

Yes, the contious build supports that now.

@ghutchis
Copy link
Member Author

@probonopd - thanks - I'll take a look tomorrow.

@ghutchis ghutchis linked a pull request Sep 7, 2021 that will close this issue
@probonopd
Copy link

Thank you @ghutchis - will the AppImage be available for download on GitHub Releases?

@ghutchis
Copy link
Member Author

ghutchis commented Sep 7, 2021

Yes, once we have our next release (probably a 1.95.2) it will show up on GitHub releases and I can add it to the hub.

For now, I'm testing to make sure it's not missing anything critical. 😄

@ghutchis ghutchis moved this to Done in Avogadro 2.0 Nov 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants