Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.debian.bugs.dist > #1214149 > unrolled thread
| Started by | Santiago Vila <sanvila@debian.org> |
|---|---|
| First post | 2024-09-24 19:50 +0200 |
| Last post | 2024-10-01 11:30 +0200 |
| Articles | 6 — 2 participants |
Back to article view | Back to linux.debian.bugs.dist
Bug#1082679: freedoom: FTBFS: dh_auto_build fails Santiago Vila <sanvila@debian.org> - 2024-09-24 19:50 +0200
Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG Simon McVittie <smcv@debian.org> - 2024-09-30 13:40 +0200
Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG Santiago Vila <sanvila@debian.org> - 2024-09-30 14:50 +0200
Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG Santiago Vila <sanvila@debian.org> - 2024-09-30 15:30 +0200
Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG Simon McVittie <smcv@debian.org> - 2024-09-30 16:30 +0200
Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG Santiago Vila <sanvila@debian.org> - 2024-10-01 11:30 +0200
| From | Santiago Vila <sanvila@debian.org> |
|---|---|
| Date | 2024-09-24 19:50 +0200 |
| Subject | Bug#1082679: freedoom: FTBFS: dh_auto_build fails |
| Message-ID | <JqhPI-eyyA-19@gated-at.bofh.it> |
Package: src:freedoom
Version: 0.13.0-1
Severity: serious
Tags: ftbfs
Dear maintainer:
During a rebuild of all packages in unstable, your package failed to build:
--------------------------------------------------------------------------------
[...]
debian/rules binary
dh binary
dh_update_autotools_config
dh_autoreconf
dh_auto_configure
debian/rules override_dh_auto_build
make[1]: Entering directory '/<<PKGBUILDDIR>>'
dh_auto_build -- \
ADOCOPTS="" \
ASCIIDOC="asciidoctor" \
ASCIIDOC_MAN="asciidoctor -b manpage" \
VERSION=0.13.0-1 \
SHELL="bash -Eeuo pipefail"
make -j2 "INSTALL=install --strip-program=true" ADOCOPTS= ASCIIDOC=asciidoctor "ASCIIDOC_MAN=asciidoctor -b manpage" VERSION=0.13.0-1 "SHELL=bash -Eeuo pipefail"
make[2]: Entering directory '/<<PKGBUILDDIR>>'
make -C lumps/dehacked # graphics/text depends on generated dehacked files
make[3]: Entering directory '/<<PKGBUILDDIR>>/lumps/dehacked'
../../scripts/simplecpp -DFREEDM < dehacked.txt > ../fdm_deh.lmp
deutex does not support PNG. Try building deutex with the PNG
libraries (libpng and libpng-devel or similar packages) installed.
deutex can be downloaded from https://github.com/Doom-Utils/deutex.
The full path to duetex can be specified by passing
DEUTEX=/the/path/to/deutex to make when building Freedoom.
make[2]: *** [Makefile:56: deutex-check] Error 1
make[2]: *** Waiting for unfinished jobs....
../../scripts/simplecpp -DPHASE1 < dehacked.txt > ../p1_deh.lmp
/<<PKGBUILDDIR>>/lumps/dehacked/../../scripts/simplecpp:37: SyntaxWarning: invalid escape sequence '\#'
command_re = re.compile("\#(\w+)(\s+(.*))?")
/<<PKGBUILDDIR>>/lumps/dehacked/../../scripts/simplecpp:38: SyntaxWarning: invalid escape sequence '\s'
include_re = re.compile('\s*"(.*)"\s*')
/<<PKGBUILDDIR>>/lumps/dehacked/../../scripts/simplecpp:37: SyntaxWarning: invalid escape sequence '\#'
command_re = re.compile("\#(\w+)(\s+(.*))?")
/<<PKGBUILDDIR>>/lumps/dehacked/../../scripts/simplecpp:38: SyntaxWarning: invalid escape sequence '\s'
include_re = re.compile('\s*"(.*)"\s*')
../../scripts/simplecpp -DPHASE2 < dehacked.txt > ../p2_deh.lmp
/<<PKGBUILDDIR>>/lumps/dehacked/../../scripts/simplecpp:37: SyntaxWarning: invalid escape sequence '\#'
command_re = re.compile("\#(\w+)(\s+(.*))?")
/<<PKGBUILDDIR>>/lumps/dehacked/../../scripts/simplecpp:38: SyntaxWarning: invalid escape sequence '\s'
include_re = re.compile('\s*"(.*)"\s*')
make[3]: Leaving directory '/<<PKGBUILDDIR>>/lumps/dehacked'
make VERSION=0.13.0-1 -C graphics/text
make[3]: Entering directory '/<<PKGBUILDDIR>>/graphics/text'
./textgen
./tint.py ../interpic.png '#57b9b0' helpbg.png
/<<PKGBUILDDIR>>/graphics/text/config.py:106: SyntaxWarning: invalid escape sequence '\w'
return re.sub("^\w*\d:\s*", "", whichbex[defn])
/<<PKGBUILDDIR>>/graphics/text/config.py:255: SyntaxWarning: invalid escape sequence '\w'
levelname = re.sub("^\w*\d:\s*", "", bexdata[bexname])
./smtextgen wikilrs_horiz.png 49x7 \
0,0 killers
./smtextgen wivctms.png 54x7 \
0,0 victims
cp wivctms.png ../
./create_caption ../titlepic/freedm_titlepic.png fdmtitle.png
cp fdmtitle.png ../
./rotate wikilrs_horiz.png 270 wikilrs.png
cp wikilrs.png ../
cp m_detail.png m_disopt.png m_disp.png m_endgam.png m_epi1.png m_epi2.png m_epi3.png m_epi4.png m_episod.png m_gdhigh.png m_gdlow.png m_hurt.png m_jkill.png m_lgttl.png m_loadg.png m_messg.png m_msens.png m_msgoff.png m_msgon.png m_musvol.png m_newg.png m_ngame.png m_nmare.png m_option.png m_optttl.png m_quitg.png m_rdthis.png m_rough.png m_saveg.png m_scrnsz.png m_sfxvol.png m_sgttl.png m_skill.png m_svol.png m_about.png m_accel.png m_auto.png m_chat.png m_chatm.png m_compat.png m_demos.png m_dmflag.png m_enem.png m_etcopt.png m_feat.png m_generl.png m_gset.png m_horsen.png m_hud.png m_joyset.png m_keybnd.png m_ldsv.png m_loksen.png m_menus.png m_mess.png m_mouse.png m_multi.png m_player.png m_serial.png m_setup.png m_sound.png m_stat.png m_status.png m_tcpip.png m_versen.png m_video.png m_wad.png m_wadopt.png m_weap.png m_crispy.png prboom.png t_phase1.png t_phase2.png m_ultra.png wibp1.png wibp2.png wibp3.png wibp4.png wicolon.png wienter.png wif.png wifrgs.png wipcnt.png wiminus.png wimstar.png wimstt.png wiostf.png wiosti.png wiostk.png wiosts.png wiscrt2.png wisucks.png witime.png wip1.png wip2.png wip3.png wip4.png wipar.png winum0.png winum1.png winum2.png winum3.png winum4.png winum5.png winum6.png winum7.png winum8.png winum9.png dmwilv00.png dmwilv01.png dmwilv02.png dmwilv03.png dmwilv04.png dmwilv05.png dmwilv06.png dmwilv07.png dmwilv08.png dmwilv09.png dmwilv10.png dmwilv11.png dmwilv12.png dmwilv13.png dmwilv14.png dmwilv15.png dmwilv16.png dmwilv17.png dmwilv18.png dmwilv19.png dmwilv20.png dmwilv21.png dmwilv22.png dmwilv23.png dmwilv24.png dmwilv25.png dmwilv26.png dmwilv27.png dmwilv28.png dmwilv29.png dmwilv30.png dmwilv31.png cwilv00.png cwilv01.png cwilv02.png cwilv03.png cwilv04.png cwilv05.png cwilv06.png cwilv07.png cwilv08.png cwilv09.png cwilv10.png cwilv11.png cwilv12.png cwilv13.png cwilv14.png cwilv15.png cwilv16.png cwilv17.png cwilv18.png cwilv19.png cwilv20.png cwilv21.png cwilv22.png cwilv23.png cwilv24.png cwilv25.png cwilv26.png cwilv27.png cwilv28.png cwilv29.png cwilv30.png cwilv31.png wilv00.png wilv01.png wilv02.png wilv03.png wilv04.png wilv05.png wilv06.png wilv07.png wilv08.png wilv10.png wilv11.png wilv12.png wilv13.png wilv14.png wilv15.png wilv16.png wilv17.png wilv18.png wilv20.png wilv21.png wilv22.png wilv23.png wilv24.png wilv25.png wilv26.png wilv27.png wilv28.png wilv30.png wilv31.png wilv32.png wilv33.png wilv34.png wilv35.png wilv36.png wilv37.png wilv38.png ../
./smtextgen help.png 320x200 \
-background "helpbg.png" \
150,2 "file:helpttl.png" \
10,20 "Weapons" \
20,33 "file:../../sprites/csawa0.png" \
90,25 "file:../../sprites/shota0.png" \
90,40 "file:../../sprites/sgn2a0.png" \
154,15 "file:../../sprites/mguna0.png" \
161,35 "file:../../sprites/launa0.png" \
235,18 "file:../../sprites/plasa0.png" \
235,33 "file:../../sprites/bfuga0.png" \
10,62 "Bullets" \
80,57 "file:../../sprites/ammoa0.png" \
68,62 "file:../../sprites/clipa0.png" \
113,62 "Shells" \
180,57 "file:../../sprites/sboxa0.png" \
163,62 "file:../../sprites/shela0.png" \
252,60 "Backpack" \
272,69 "file:../../sprites/bpaka0.png" \
10,85 "Missiles" \
90,75 "file:../../sprites/broka0.png" \
76,75 "file:../../sprites/rocka0.png" \
158,85 "Energy" \
225,76 "file:../../sprites/celpa0.png" \
206,84 "file:../../sprites/cella0.png" \
10,110 "Health" \
63,106 "file:../../sprites/bon1a0.png" \
100,101 "file:../../sprites/media0.png" \
79,113 "file:../../sprites/stima0.png" \
132,102 "file:../../sprites/pstra0.png" \
180,110 "Armor" \
276,105 "file:../../sprites/arm2b0.png" \
240,102 "file:../../sprites/arm1b0.png" \
225,109 "file:../../sprites/bon2b0.png" \
145,134 "Map" \
175,122 "file:../../sprites/pmapa0.png" \
10,134 "Big Boost" \
84,127 "file:../../sprites/soula0.png" \
110,127 "file:../../sprites/megaa0.png" \
10,155 "Light" \
48,152 "file:../../sprites/pvisa0.png" \
34,177 "Cloak" \
80,161 "file:../../sprites/pinsa0.png" \
119,177 "Invuln." \
174,166 "file:../../sprites/pinva0.png" \
265,137 "Keys" \
254,149 "file:../../sprites/rkeya0.png" \
271,149 "file:../../sprites/ykeya0.png" \
288,149 "file:../../sprites/bkeya0.png" \
255,173 "file:../../sprites/rskua0.png" \
273,173 "file:../../sprites/yskua0.png" \
291,173 "file:../../sprites/bskua0.png" \
130,155 "Rescue suit" \
215,137 "file:../../sprites/suita0.png" \
2,192 "https://freedoom.github.io/" \
214,192 "<Esc> to return"
./smtextgen credit.png 320x200 \
-background "helpbg.png" \
120,5 "file:freettl.png" \
10,30 "include:credit.txt"
cp help.png ../
./create_caption ../titlepic/titlepic.png ../m_doom.png ../t_phase1.png fd1title.png
cp credit.png ../
./create_caption ../titlepic/titlepic.png ../m_doom.png ../t_phase2.png fd2title.png
cp fd1title.png ../
cp fd2title.png ../
make[3]: Leaving directory '/<<PKGBUILDDIR>>/graphics/text'
make -C lumps/playpal
make[3]: Entering directory '/<<PKGBUILDDIR>>/lumps/playpal'
./playpal playpal-base.lmp > playpal.lmp
cp playpal.lmp ../
make[3]: Leaving directory '/<<PKGBUILDDIR>>/lumps/playpal'
make -C lumps/colormap
make[3]: Entering directory '/<<PKGBUILDDIR>>/lumps/colormap'
./colormap ../playpal/playpal-base.lmp > colormap.lmp
cp colormap.lmp ../
make[3]: Leaving directory '/<<PKGBUILDDIR>>/lumps/colormap'
make -C lumps/genmidi
make[3]: Entering directory '/<<PKGBUILDDIR>>/lumps/genmidi'
./mkgenmidi genmidi.lmp
cp genmidi.lmp ../
make[3]: Leaving directory '/<<PKGBUILDDIR>>/lumps/genmidi'
make -C lumps/dmxgus
make[3]: Entering directory '/<<PKGBUILDDIR>>/lumps/dmxgus'
./gen-ultramid ultramid.ini
cp ultramid.ini ../dmxgus.lmp
make[3]: Leaving directory '/<<PKGBUILDDIR>>/lumps/dmxgus'
make -C lumps/textures
make[3]: Entering directory '/<<PKGBUILDDIR>>/lumps/textures'
../../scripts/simplecpp -DDOOM1_VERSIONS < textures.cfg | \
./build-textures -compat_texture1=doom1/texture1.txt \
-compat_texture2=doom1/texture2.txt \
-compat_pnames=doom1/pnames.txt \
-output_texture1=phase1/texture1.lmp \
-output_texture2=phase1/texture2.lmp \
-output_pnames=phase1/pnames.lmp \
-output_pnames_txt=phase1/pnames.txt \
-sprites_dir=../../sprites
../../scripts/simplecpp < textures.cfg | \
./build-textures -compat_texture1=doom2/texture1.txt \
-compat_pnames=doom2/pnames.txt \
-output_texture1=phase2/texture1.lmp \
-output_pnames=phase2/pnames.lmp \
-output_pnames_txt=phase2/pnames.txt \
-sprites_dir=../../sprites
/<<PKGBUILDDIR>>/lumps/textures/../../scripts/simplecpp:37: SyntaxWarning: invalid escape sequence '\#'
command_re = re.compile("\#(\w+)(\s+(.*))?")
/<<PKGBUILDDIR>>/lumps/textures/../../scripts/simplecpp:38: SyntaxWarning: invalid escape sequence '\s'
include_re = re.compile('\s*"(.*)"\s*')
/<<PKGBUILDDIR>>/lumps/textures/../../scripts/simplecpp:37: SyntaxWarning: invalid escape sequence '\#'
command_re = re.compile("\#(\w+)(\s+(.*))?")
/<<PKGBUILDDIR>>/lumps/textures/../../scripts/simplecpp:38: SyntaxWarning: invalid escape sequence '\s'
include_re = re.compile('\s*"(.*)"\s*')
cp phase2/texture1.lmp ../fd2txtr1.lmp
cp phase1/texture1.lmp ../fd1txtr1.lmp
cp phase2/pnames.lmp ../fd2pname.lmp
cp phase1/texture2.lmp ../fd1txtr2.lmp
cp phase1/pnames.lmp ../fd1pname.lmp
../../scripts/simplecpp -DFREEDM < textures.cfg | \
./build-textures -compat_texture1=doom2/texture1.txt \
-compat_pnames=doom2/pnames.txt \
-output_texture1=freedm/texture1.lmp \
-output_pnames=freedm/pnames.lmp \
-output_pnames_txt=freedm/pnames.txt \
-sprites_dir=../../sprites
/<<PKGBUILDDIR>>/lumps/textures/../../scripts/simplecpp:37: SyntaxWarning: invalid escape sequence '\#'
command_re = re.compile("\#(\w+)(\s+(.*))?")
/<<PKGBUILDDIR>>/lumps/textures/../../scripts/simplecpp:38: SyntaxWarning: invalid escape sequence '\s'
include_re = re.compile('\s*"(.*)"\s*')
cp freedm/texture1.lmp ../fdmtxtr1.lmp
cp freedm/pnames.lmp ../fdmpname.lmp
make[3]: Leaving directory '/<<PKGBUILDDIR>>/lumps/textures'
make -C bootstrap
make[3]: Entering directory '/<<PKGBUILDDIR>>/bootstrap'
./bootstrap < ../lumps/playpal/playpal-base.lmp > doom2.wad
make[3]: Leaving directory '/<<PKGBUILDDIR>>/bootstrap'
make[2]: Leaving directory '/<<PKGBUILDDIR>>'
dh_auto_build: error: make -j2 "INSTALL=install --strip-program=true" ADOCOPTS= ASCIIDOC=asciidoctor "ASCIIDOC_MAN=asciidoctor -b manpage" VERSION=0.13.0-1 "SHELL=bash -Eeuo pipefail" returned exit code 2
make[1]: *** [debian/rules:14: override_dh_auto_build] Error 25
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
make: *** [debian/rules:11: binary] Error 2
dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
--------------------------------------------------------------------------------
The above is just how the build ends and not necessarily the most relevant part.
If required, the full build log is available here:
https://people.debian.org/~sanvila/build-logs/202409/
About the archive rebuild: The build was made on virtual machines from AWS,
using sbuild and a reduced chroot with only build-essential packages.
If you could not reproduce the bug please contact me privately, as I
am willing to provide ssh access to a virtual machine where the bug is
fully reproducible.
If this is really a bug in one of the build-depends, please use
reassign and affects, so that this is still visible in the BTS web
page for this package.
Thanks.
[toc] | [next] | [standalone]
| From | Simon McVittie <smcv@debian.org> |
|---|---|
| Date | 2024-09-30 13:40 +0200 |
| Subject | Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG |
| Message-ID | <JsmUW-fU7e-13@gated-at.bofh.it> |
| In reply to | #1214149 |
On Mon, 30 Sep 2024 at 13:06:31 +0200, Santiago Vila wrote:
> - When I build with 1 CPU, I have never found the build problem so far.
> - When I build with 2 CPUs, I got a failure rate of 66%
> (16 failures over 24 tries so far).
When I tried this, it was consistently failing if and only if I use
the .dsc from the archive, and consistently succeeding with a rebuilt
.dsc. All of these builds have been in a VM with 3 VCPUs, on a host
system with 4 cores (SMT disabled, so 1 thread per core).
I don't know *why* rebuilding the .dsc avoids this failure mode, but it
appears that it does. This means that I can't usefully try workarounds
or add extra debugging instrumentation, because any time I do that, the
.dsc is rebuilt, the problem vanishes, and the debugging instrumentation
doesn't trigger.
In particular I tried modifying the failing target to add the obvious
debug info for why upstream's check was failing:
```diff
deutex-check:
+ $(DEUTEX) -h || echo exit status $$?
+ $(DEUTEX) -h | grep -w PNG || echo exit status $$?
@$(DEUTEX) -h | grep -qw PNG || { \
```
but that didn't exhibit the failure (deutex -h showed the expected output
and exited with status 0, deutex -h | grep -w PNG showed non-empty output
and exited with status 0, and upstream's check succeeded).
Are you able to send a modified .dsc (rebuilt with no changes, or with
only changelog changes) to your rebuild infrastructure? If yes, please
try with a rebuilt .dsc (with 2 or more CPUs, since that seems to be
your worst-case scenario for this particular package).
> I would try how it behaves when using --no-parallel.
It did build successfully with `dh $@ --no-parallel`, but that doesn't
necessarily mean anything, because adding that flag required rebuilding
the .dsc, and my experience has been that rebuilding the .dsc is itself
enough to work around this for some as yet unknown reason.
smcv
[toc] | [prev] | [next] | [standalone]
| From | Santiago Vila <sanvila@debian.org> |
|---|---|
| Date | 2024-09-30 14:50 +0200 |
| Subject | Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG |
| Message-ID | <Jso0F-fUMq-1@gated-at.bofh.it> |
| In reply to | #1214707 |
Ok, I performed the following additional tests. Built 200 times on five different VMs with 1 CPU. 0/200 failures = 0% Built 200 times on five different VMs with 2 CPUs. 183/200 failures = 91.5% My autobuilder framework does not let me feed my own .dsc, but I recreated the .dsc and tried 10 times using a for loop. The result was 5/10 failures, or 50%. So, I don't think that making a dummy upload will fix this. I'm going to try --no-parallel next and will let you know how it goes. Thanks.
[toc] | [prev] | [next] | [standalone]
| From | Santiago Vila <sanvila@debian.org> |
|---|---|
| Date | 2024-09-30 15:30 +0200 |
| Subject | Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG |
| Message-ID | <JsoDn-fVop-1@gated-at.bofh.it> |
| In reply to | #1214707 |
Hmm, I forgot to Cc: the bug address itself for this message. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1082679;msg=29 To summarize: --no-parallel fixes the problem for me, and I believe it's a good enough fix. Thanks.
[toc] | [prev] | [next] | [standalone]
| From | Simon McVittie <smcv@debian.org> |
|---|---|
| Date | 2024-09-30 16:30 +0200 |
| Subject | Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG |
| Message-ID | <Jspzr-fWcW-1@gated-at.bofh.it> |
| In reply to | #1214149 |
On Mon, 30 Sep 2024 at 15:46:24 +0200, Fabian Greffrath wrote:
> Maybe the `deutex -h` output to stdout get spoiled by other build steps
> running in parralel, so that it doesn't include "PNG" as an isolated word
> anymore?
That shouldn't be the case, assuming the shell is working correctly:
it's the output of `deutex -h` that is redirected to a pipe (to grep),
not the output of the build system as a whole.
smcv
[toc] | [prev] | [next] | [standalone]
| From | Santiago Vila <sanvila@debian.org> |
|---|---|
| Date | 2024-10-01 11:30 +0200 |
| Subject | Bug#1082679: freedoom: FTBFS with .dsc from archive: mis-detects that deutex does not support PNG |
| Message-ID | <JsHmG-gaYY-9@gated-at.bofh.it> |
| In reply to | #1214149 |
retitle 1082679 FTBFS: mis-detects that deutex does not support PNG thanks El 1/10/24 a las 7:52, Fabian Greffrath escribió: > Am Dienstag, dem 01.10.2024 um 03:26 +0200 schrieb Santiago Vila: >> b) The SHELL="bash -Eeuo pipefail" definition in debian/rules. > > I think I can just remove these lines, upstream did the same some years > ago: > > https://github.com/freedoom/freedoom/commit/77c53e11adfb1e5124e363d68518527c4fcfdf1a I actually think that's better. Note: I'm dropping the "dsc from the archive" part from the bug title, as this was not really a case of "tainted dsc", just a bug difficult to track. Thanks.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.debian.bugs.dist
csiph-web