/srv/irclogs.ubuntu.com/2022/07/26/#ubuntu-desktop.txt

didrocksgood morning05:37
oSoMoNgood morning desktoppers06:08
didrockssalut oSoMoN 06:12
oSoMoNsalut didrocks 06:12
KGB-0system-config-printer ubuntu/jammy b18f13b Gunnar Hjalmarsson * pushed 9 commits (first 5 follow) * https://deb.li/G1H106:23
KGB-0system-config-printer ubuntu/jammy 8a7ad9a Jeremy Bicha debian/patches/ series Allow-installing-non-free-drivers-from-OpenPrinting.patch Hide-Printers-app-in-GNOME-and-KDE.patch Rename-app-to-Printers.patch * Add remaining Ubuntu patches * https://deb.li/35g5N06:23
KGB-0system-config-printer ubuntu/jammy 761a928 Jeremy Bicha debian/ control control.in gbp.conf * Update Maintainer and Vcs fields and debian/gbp.conf * https://deb.li/P3Qc06:24
KGB-0system-config-printer ubuntu/jammy cae8ee3 Jeremy Bicha debian/changelog * releasing package system-config-printer version 1.5.15-1ubuntu1 * https://deb.li/3JP9y06:24
KGB-0system-config-printer ubuntu/jammy e46e53e Jeremy Bicha debian/ (5 files in 2 dirs) * Merge tag 'debian/1.5.15-2' into ubuntu/master * https://deb.li/WYns06:24
KGB-0system-config-printer ubuntu/jammy eab997d Jeremy Bicha debian/changelog * releasing package system-config-printer version 1.5.15-2ubuntu1 * https://deb.li/3FifU06:24
KGB-0system-config-printer ubuntu/jammy 6e378ba Gunnar Hjalmarsson debian/rules * Fix syntax error in d/rules * https://deb.li/iUqXM06:40
lissyxoSoMoN, good, the cron is ready :) https://treeherder.mozilla.org/jobs?repo=mozilla-central&searchStr=symbols&selectedTaskRun=JuLzffjpSCeRHe-RczPocA.007:04
dufluHi didrocks, oSoMoN, lissyx 07:14
=== duflu_ is now known as duflu
didrockshey duflu 07:50
ricotzgood morning desktopers :)08:07
alocermorning all08:13
dufluHi ricotz and alocer 08:16
seb128goood morning desltopers08:25
seb128hey duflu ricotz alocer 08:25
dufluHi seb128 08:26
didrockshey alocer, seb128 08:27
seb128hey duflu, how is it going?08:33
dufluseb128, too much drama for one day but I am getting on top of it08:34
dufluHow are you?08:35
ricotzduflu, seb128, didrocks, alocer, hello08:36
seb128lut didrocks 08:37
seb128seems like IRC timeouted again :/08:37
lissyxoSoMoN, finger crossed.09:45
oSoMoNlissyx, I'm seeing the same build failures for arm64 dbgsym-enabled builds on Launchpad :/ So if the patch to reduce the debug info is working for you, we'll probably need to apply it conditionnally depending on the architecture09:48
lissyxwhat are the limitations on launchpad?09:51
lissyxmaybe you can raise some more easily?09:52
lissyxoSoMoN, build seems to be successfully finishing for me09:58
lissyxoSoMoN, I'm not so used to quilt, how can we apply arch-based?09:59
oSoMoNlissyx, quilt doesn't allow to apply patches conditionnally, so the condition has to be written in the patch itself10:01
oSoMoNlissyx, or, put the patch in a different place, and apply it conditionnally from snapcraft.yaml10:01
lissyxoSoMoN, what solution do you prefer?10:02
lissyxgiven that for now we would need to condition on GitHub Actions running, or on ARM64 builds, I tend to think we might want to deal with it at snapcraft.yaml level?10:03
lissyxI'm also verifying if it it safe to apply always ...10:03
lissyx(I was suggested that because it is what opensuse does, and according to their spec file, it seems they do unconditionnally?10:04
oSoMoNyes, doing it conditionally at the snapcraft.yaml level sounds fine10:07
oSoMoNwe already do quite a lot of conditional work there anyway10:07
lissyxI've failed to be able to pass env var to snapcraft, do you know how I could detect a GitHub Actions build VS a Launchpad build?10:08
=== guiverc2 is now known as guiverc
oSoMoNlissyx, according to https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#file-systems , there's $GITHUB_WORKSPACE and $GITHUB_EVENT_PATH that we could use to detect a github action run10:12
lissyxok10:12
lissyxlunch time10:12
lissyxoSoMoN, but do we get those inside snapcraft.yaml?11:25
lissyxoSoMoN, updated my patch11:30
lissyxoSoMoN, testing: https://github.com/lissyx/firefox-snap/actions/runs/273915238911:32
oSoMoNlissyx, I would think that snapcraft doesn't strip the environment (but I might be wrong), so we should be seeing $GITHUB_WORKSPACE inside the build scriptlets11:55
lissyxoSoMoN, /bin/bash: line 53: GITHUB_WORKSPACE: unbound variable12:05
lissyxI'm going to rely on the symbols upload token file ...12:08
ricotzoSoMoN, is it possible that https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1951210 regressed?12:17
ubottuLaunchpad bug 1951210 in libreoffice (Ubuntu Jammy) "libreoffice help doesn't open in firefox (404 error on file:///tmp/lu417531j7po.tmp/NewHelp0.html)" [Medium, Fix Released]12:17
ricotzoSoMoN, this isn't working anymore in a clean VM with snapd 2.56.2 and firefox 102.1.0 snap12:21
lissyxoSoMoN, weird12:55
lissyxoSoMoN, https://github.com/lissyx/firefox-snap/runs/7519781080?check_suite_focus=true#step:5:2143512:55
lissyxoSoMoN, like it skipped the build ?12:55
oSoMoNlissyx, it looks like it stopped the build script after "patch -p1 patches/mozilla-reduce-rust-debuginfo.patch"13:11
oSoMoNmaybe the patch didn't apply cleanly?13:11
lissyxthere was no error reported13:12
lissyxoSoMoN, meh13:59
lissyxoSoMoN, "patch -p1 ..."13:59
lissyxlike, it misses "<" ...13:59
oSoMoNheh14:01
lissyxoSoMoN, so the file is there: https://github.com/lissyx/firefox-snap/blob/debug-symbols_only_TESTS/patches/mozilla-reduce-rust-debuginfo.patch14:44
lissyxbut it's not finding it? https://github.com/lissyx/firefox-snap/runs/7521824157?check_suite_focus=true#step:5:2135714:45
lissyxrelative vs abslute path maybe?14:46
oSoMoNlissyx, you'll need to prefix the path to the patch with $SNAPCRAFT_PROJECT_DIR/14:46
oSoMoNbecause the build script is running relative to a different directory, not the project's root14:46
lissyxyeah14:54
oSoMoNricotz, it looks like bug #1951210 regressed indeed14:56
ubottuBug 1951210 in libreoffice (Ubuntu Jammy) "libreoffice help doesn't open in firefox (404 error on file:///tmp/lu417531j7po.tmp/NewHelp0.html)" [Medium, Fix Released] https://launchpad.net/bugs/195121014:56
oSoMoNa possible suspect: https://github.com/snapcore/snapd/commit/5abb3aab6b74db5a9e9920c537d673806603349a15:03
ubottuCommit 5abb3aa in snapcore/snapd "interfaces/system-packages-doc: allow read-only access to /usr/share/gtk-doc"15:03
oSoMoNjbicha, could it be that the changes you made to interfaces/builtin/system_packages_doc.go in that commit regressed RO access to /usr/share/libreoffice/help ?15:07
oSoMoNthe removal of 'apparmor.GenWritableProfile(emit, "/usr/share/libreoffice/help", 3)' looks suspicious to me15:07
oSoMoNI've verified that /usr/share/libreoffice is mounted in the snap's namespace, but not its subdirectories15:09
lissyxoSoMoN, 1h28m still building, we might be on something :)16:16
lissyxoSoMoN, yallah! https://github.com/lissyx/firefox-snap/runs/7522784969?check_suite_focus=true#step:5:3387418:07
lissyxoSoMoN, wrong token, so it's all good18:08
lissyxoSoMoN, I'm going to change the URL back to the prod server on my other PR18:08
ricotzoSoMoN, thanks for confirming19:59

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!