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

Create iPuissance_4D #2098

Merged
merged 1 commit into from
Jul 31, 2020
Merged

Create iPuissance_4D #2098

merged 1 commit into from
Jul 31, 2020

Conversation

wernight
Copy link
Contributor

@wernight wernight force-pushed the patch-1 branch 2 times, most recently from 8ad03a5 to c3c2dd5 Compare July 28, 2020 14:29
@wernight
Copy link
Contributor Author

wernight commented Jul 28, 2020

I wonder if it works for OpenGL / SDL 1.2 application:

NUMBER_OF_WINDOWS: 0
ERROR: Could not find a single window on screen :-(

However from the log (Entered SplashScreen state.) it did create the window it seems.

Is there anything I can do to get an idea of the issue here?

@probonopd
Copy link
Member

Thanks for submitting.

Probably the test fails due to the test server being headless (no GPU)? See the worker.sh script to understand how the test works.

@wernight
Copy link
Contributor Author

I did this locally:

$ Xvfb :99 -screen 0 800x600x24 &
$ DISPLAY=:99 ./iPuissance_4D-x86_64.AooImage
$ DISPLAY=:99 xwininfo -tree -root | grep 0x | grep '": ("' | sed -e 's/^[[:space:]]*//'
0x20000e "iPuissance 4D": ("iPuissance4D" "iPuissance4D")  800x600+0+0  +0+0
$ DISPLAY=:99 xwininfo -tree -root | grep 0x | grep '": ("' | sed -e 's/^[[:space:]]*//' | wc -l
1
$ DISPLAY=:99 xwd -root -silent | convert xwd:- png:/tmp/screenshot.png

All worked especially the window count. I didn't run on Ubuntu Xenial though.

@wernight
Copy link
Contributor Author

wernight commented Jul 30, 2020

sudo docker run --rm -it -v $PWD:/mnt:ro ubuntu:xenial
apt update
apt-get -qq -y install libasound2-dev alsa-utils alsa-oss libjack0 desktop-file-utils xmlstarlet xterm xvfb icewm x11-utils x11-apps netpbm xdotool libgl1-mesa-dri libgl1-mesa-dev mesa-utils libosmesa6 libsdl1.2-dev fonts-wqy-microhei libfile-mimeinfo-perl 
export DISPLAY=:99
Xvfb $DISPLAY -screen 0 800x600x24 &
cd /tmp
/mnt/iPuissance_4D-x86_64.AppImage --appimage-extract
cd squashfs-root
./AppRun

It outputs:

...
Entered SplashScreen state.
Segmentation fault (core dumped)


Visibly due to libexpat:

Stack trace of thread 24597:
#0 0x00000000005aba42 n/a (/tmp/squashfs-root/usr/bin/iPuissance4D + 0x1aba42)
#1 0x00000000005ac1e8 n/a (/tmp/squashfs-root/usr/bin/iPuissance4D + 0x1ac1e8)
#2 0x00007f983dda793d n/a (/lib/x86_64-linux-gnu/libexpat.so.1.6.0 + 0x893d)


The linuxdeploy tool didn't add `libexpat.so` to the AppImage.

What I find more weird is that installing `libexpat1` which creates `/lib/x86_64-linux-gnu/libexpat.so.1.6.0`, does not fix the issue.

@probonopd
Copy link
Member

The linuxdeploy tool didn't add libexpat.so to the AppImage.

Is libexpat.so opened using dl_open()?
If you suspect a bug in linuxdeploy, then please post it over at https://github.com/linuxdeploy/linuxdeploy, thanks.

@wernight
Copy link
Contributor Author

I added libpcre as present in my build system and it still fails with the same error.

Tried the same on debian:jessie, it gives again the same error.
Tried the same on ubuntu:bionic, it gives a similar error but against /lib/x86_64-linux-gnu/libexpat.so.1.6.7.

Note that I build on debian:jessie (oldoldstable).

It seems as if the error is due in some other dependency. I could try to remove the dependency to XML (which was a horrible idea anyway) but that would require quite some work.

@probonopd
Copy link
Member

Actually libexpat.so.1 is on the excludelist because we think that every target system (distribution) ships it by default.

https://github.com/AppImage/pkg2appimage/blob/d8117501942f2b4780bac0de551fc431d5901562/excludelist#L128

On which system is this not the case?

@wernight
Copy link
Contributor Author

Tries a test version that doesn't use expat, and coredump shows:

#0  0x00007ff383940438 n/a (/lib/x86_64-linux-gnu/libc-2.23.so + 0x35438)

So it's visibly an issue when running in Docker for some reason.

@wernight
Copy link
Contributor Author

Aha!. No my bad, it is really EXPAT that fails. I'll try to remove it.

@wernight wernight force-pushed the patch-1 branch 3 times, most recently from fa50fb2 to 0e45809 Compare July 31, 2020 16:51
Connect 4 game in 3D with multiplayer support, see
https://ipuissance4d.beroux.com
@probonopd
Copy link
Member

Thank you very much. It worked 👍

@probonopd probonopd merged commit 9b2438e into AppImage:master Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants