-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
Comments
Replacing
|
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. |
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. |
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. |
There's also 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 |
Looks like this might be a good approach (another Qt-based app): |
Yes, the contious build supports that now. |
@probonopd - thanks - I'll take a look tomorrow. |
Thank you @ghutchis - will the AppImage be available for download on GitHub Releases? |
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. 😄 |
@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.
The text was updated successfully, but these errors were encountered: