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

Performs a secondary cleanup attempt in isolate-test.sh in case the first one fails #1533

Closed
wants to merge 1 commit into from

Conversation

xsdg
Copy link
Contributor

@xsdg xsdg commented Oct 29, 2024

This works around a race condiditon that occurs reliably when the xdg-desktop-portal package is installed.

When this triggers, the output looks like:

$.../scripts/isolate-test.sh .../scripts/image-test.py .../build/src/geeqie .../build/test-images.p/images/JPG_MINOLTA_RD175.JPG
Variables in isolated environment:
G_DEBUG=fatal-critical
HOME=/tmp/tmp.uCSyUTWVkw
XDG_CONFIG_HOME=/tmp/tmp.uCSyUTWVkw/.config
XDG_RUNTIME_DIR=/tmp/tmp.uCSyUTWVkw/.runtime
DBUS_SESSION_BUS_ADDRESS=unix:path=/tmp/dbus-Nou8YYvYti,guid=9de5a10c77563265a7e1c9b167214161

[...]
Warning: Failed to convert Exif.Photo.DateTimeOriginal to Xmp.photoshop.DateCreated, unable to parse '19ff:1f:3f 3f:7f:00'
Warning: Failed to convert Exif.Photo.DateTimeDigitized to Xmp.xmp.CreateDate, unable to parse '19ff:1f:3f 3f:7f:00'
rm: cannot remove '/tmp/tmp.uCSyUTWVkw/.runtime/doc': Is a directory

First cleanup attempt failed; sleeping and retrying...
removed directory '/tmp/tmp.uCSyUTWVkw/.runtime/doc'
removed directory '/tmp/tmp.uCSyUTWVkw/.runtime'
removed directory '/tmp/tmp.uCSyUTWVkw'

Fixes #1532

This works around a race condiditon that occurs reliably when the `xdg-desktop-portal` package is installed.

When this triggers, the output looks like:
```
$.../scripts/isolate-test.sh .../scripts/image-test.py .../build/src/geeqie .../build/test-images.p/images/JPG_MINOLTA_RD175.JPG
Variables in isolated environment:
G_DEBUG=fatal-critical
HOME=/tmp/tmp.uCSyUTWVkw
XDG_CONFIG_HOME=/tmp/tmp.uCSyUTWVkw/.config
XDG_RUNTIME_DIR=/tmp/tmp.uCSyUTWVkw/.runtime
DBUS_SESSION_BUS_ADDRESS=unix:path=/tmp/dbus-Nou8YYvYti,guid=9de5a10c77563265a7e1c9b167214161

[...]
Warning: Failed to convert Exif.Photo.DateTimeOriginal to Xmp.photoshop.DateCreated, unable to parse '19ff:1f:3f 3f:7f:00'
Warning: Failed to convert Exif.Photo.DateTimeDigitized to Xmp.xmp.CreateDate, unable to parse '19ff:1f:3f 3f:7f:00'
rm: cannot remove '/tmp/tmp.uCSyUTWVkw/.runtime/doc': Is a directory

First cleanup attempt failed; sleeping and retrying...
removed directory '/tmp/tmp.uCSyUTWVkw/.runtime/doc'
removed directory '/tmp/tmp.uCSyUTWVkw/.runtime'
removed directory '/tmp/tmp.uCSyUTWVkw'
```
@caclark
Copy link
Collaborator

caclark commented Nov 8, 2024

Implemented in commit 9e5a7eb

@caclark caclark closed this Nov 8, 2024
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.

Geeqie image tests fail with xdg-desktop-portal package installed
2 participants