search

flathub / org.frescobaldi.Frescobaldi

https://flathub.org/apps/details/org.frescobaldi.Frescobaldi
1 stars 3 forks source link

Use --filesystem=home for filesystem permissions #40

Closed jeanas closed 10 months ago

jeanas commented 10 months ago

Refs: https://github.com/frescobaldi/frescobaldi/issues/1630

flathubbot commented 10 months ago

Started test build 68107

flathubbot commented 10 months ago

Started test build 68110

flathubbot commented 10 months ago

Build 68107 failed

flathubbot commented 10 months ago

Started test build 68161

jeanas commented 10 months ago

Looks like the CI didn't appreciate that I force-pushed the branch while a test run was already in progress...

hfiguiere commented 10 months ago

it really doesn't care.

jeanas commented 10 months ago

Are you sure? The first error was

git reset --hard 153033f0e2bec254a025296d0d9702ff8a1facdc --
 in dir /srv/buildbot/worker/build-x86_64-3/build (timeout 1200 secs)
 watching logfiles {}
 argv: [b'git', b'reset', b'--hard', b'153033f0e2bec254a025296d0d9702ff8a1facdc', b'--']
 using PTY: False
fatal: Could not parse object '153033f0e2bec254a025296d0d9702ff8a1facdc'.

and the second

Traceback (most recent call last):
--- <exception caught here> ---
  File "/var/lib/buildbot-master/sandbox/lib/python3.10/site-packages/buildbot/process/buildstep.py", line 612, in startStep
    yield defer.gatherResults(dl)
twisted.internet.defer.FirstError: FirstError[#2, [Failure instance: Traceback (failure with no frames): <class 'twisted.internet.defer.FirstError'>: FirstError[#0, [Failure instance: Traceback (failure with no frames): <class 'twisted.internet.defer.FirstError'>: FirstError[#1, [Failure instance: Traceback: <class 'KeyError'>: 50765
flathubbot commented 10 months ago

Build 68161 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/50812/org.frescobaldi.Frescobaldi.flatpakref
jeanas commented 10 months ago

The build succeeded this time, and the result is as expected in a quick test that I just did.

@fedelibre From my point of view, this is ready to be merged.