Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

Automatically build key portions of AppImageKit #13

Merged
merged 2 commits into from
Apr 3, 2014

Conversation

olekw
Copy link
Contributor

@olekw olekw commented Mar 31, 2014

  • Build only the portion necessary for creating Ember AppImages
  • Use the resulting AppImageKit components to create the Ember AppImage
  • This will remove the requirement to install/compile AppImageKit manually
  • Minor cleanup/adjustment of existing AppImage code

@olekw
Copy link
Contributor Author

olekw commented Mar 31, 2014

I was going to write up the new process for building an AppImage to post on the wiki but it's not simple, and I'm lazy... so instead I modified hammer to take care of all the AppImage stuff internally. ;) Shell scripts are more fun to write than wiki pages and this will make the wiki page MUCH easier to write!

@olekw
Copy link
Contributor Author

olekw commented Mar 31, 2014

Oh, almost forgot, there are two additional package dependencies with this change: xorriso and python-xdg. Don't think either should be a problem for any major GNU/Linux distro.

echo " ogre - 3D rendering engine"
echo " Hint: build ogre first then cegui"
echo " cg - interactive effects toolkit"
echo " AppImageKit - application packaging system"
Copy link
Member

Choose a reason for hiding this comment

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

I would prefer "appimagekit" to fit with the other options which are all lower case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, crap. That's a typo. The script already expects it to be lower case. I'll fix the "documentation"in the morning. It's past my bedtime now. ;-)

On March 31, 2014 1:45:17 AM CDT, Erik Ogenvik notifications@github.com wrote:

@@ -169,12 +169,13 @@ function show_help()
echo ""
echo "Usage: hammer.sh install-deps "
echo "Dependencies Available:"

  • echo " all - install all dependencies listed below"
  • echo " cegui - a free library providing windowing and
    widgets for "
  • echo " graphics APIs / engines"
  • echo " ogre - 3D rendering engine"
  • echo " cg - interactive effects toolkit"
  • echo "Hint: build ogre first then cegui"
  • echo " all - install all dependencies listed below"
  • echo " cegui - a free library providing windowing and
    widgets for"
  • echo " graphics APIs / engines"
  • echo " ogre - 3D rendering engine"
  • echo " Hint: build ogre first then cegui"
  • echo " cg - interactive effects toolkit"
  • echo " AppImageKit - application packaging system"

I would prefer "appimagekit" to fit with the other options which are
all lower case.


Reply to this email directly or view it on GitHub:
https://github.com/worldforge/hammer/pull/13/files#r11105058

Sent from my Android device with K-9 Mail. Please excuse my brevity.

- Build only the portion necessary for creating Ember AppImages
- Use the resulting AppImageKit components to create the Ember AppImage
- This will remove the requirement to install/compile AppImageKit manually
- Minor cleanup/adjustment of existing AppImage code
@olekw
Copy link
Contributor Author

olekw commented Apr 1, 2014

Ok, help screen updated.

@erikogenvik
Copy link
Member

Running "./hammer.sh release_ember dev image" results in an error, but none is reported. The script finishes with this output:
"Build complete.
Creating AppImage.
Installing core AppImageKit functionality...
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 3697 100 3697 0 0 2460 0 0:00:01 0:00:01 --:--:-- 3138
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 4973 100 4973 0 0 10407 0 --:--:-- --:--:-- --:--:-- 11484
Configuring...
Building...
Installing...
Done."

However, when looking at the logs AppDir.log says:
cp: cannot stat ‘/home/erik/code/worldforge-git/hammer/../AppImageKit/AppRun’: No such file or directory

and AppImage says:
python: can't open file '/home/erik/code/worldforge-git/hammer/../AppImageKit/AppImageAssistant.AppDir/package': [Errno 2] No such file or directory

@DavidJFelix
Copy link
Contributor

Possibly missing /AppImageAssistant.AppDir/ from the path on line 915 of hammer.sh? I can't account for why it was removed in the changes. Just a guess.

@olekw
Copy link
Contributor Author

olekw commented Apr 2, 2014

Erik, that path is from an older version of hammer. Could you try removing your hammer directory and re-cloning from the updated hammer? Let me know if that fixes the problem.

@olekw
Copy link
Contributor Author

olekw commented Apr 2, 2014

David, the paths are different because this commit allows hammer to build the portions of AppImageKit that it needs, instead of using a separately compiled AppImageKit.

- When creating an AppImage, certain libraries must be removed
- The script will currently exit when it fails to delete such a library
- The script should not exit if such a library was never present
@olekw
Copy link
Contributor Author

olekw commented Apr 3, 2014

Ok, just pushed that commit.

@probonopd
Copy link

Are you providing an AppImage for download and if so, where?
If not, why not?

@erikogenvik
Copy link
Member

We do provide AppImages of the latest release at https://sourceforge.net/projects/worldforge/files/Ember%20%28OGRE%20client%29/0.7.2/

We don't currently provide AppImages of the development version, mainly because we don't have a CI server setup.

@probonopd
Copy link

probonopd commented Jun 7, 2016

Thanks for the clarification, so I assume there is no support needed for generating the AppImage :-)

@erikogenvik
Copy link
Member

We would be very happy for any help with working on the AppImage generation code. I haven't used it in a while, so I don't know how up to date it is. Presumably there's been developments in AppImage which we haven't kept us up to date with.
If you have more information, or are able to help out, it would be greatly appreciated.

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

Successfully merging this pull request may close these issues.

None yet

4 participants